void AddressOf (EmitContext ec, AddressOp mode);
}
- /// <summary>
- /// This interface is implemented by variables
- /// </summary>
+ // TODO: Rename to something meaningful, this is flow-analysis interface only
public interface IVariable {
- VariableInfo VariableInfo {
- get;
- }
-
- bool VerifyFixed ();
+ VariableInfo VariableInfo { get; }
+ bool IsFixed { get; }
}
/// <remarks>
return texpr;
}
+
+ public virtual void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ // TODO: It should probably be type = storey.MutateType (type);
+ }
}
/// <summary>
/// </summary>
public abstract class TypeCast : Expression
{
- protected Expression child;
+ protected readonly Expression child;
protected TypeCast (Expression child, Type return_type)
{
return child.GetAttributableValue (value_type, out value);
}
- protected override void CloneTo (CloneContext clonectx, Expression t)
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
{
- TypeCast target = (TypeCast) t;
+ child.MutateHoistedGenericType (storey);
+ }
- target.child = child.Clone (clonectx);
+ protected override void CloneTo (CloneContext clonectx, Expression t)
+ {
+ // Nothing to clone
}
public override bool IsNull {
LoadFromPtr (ig, t);
}
}
+
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ type = storey.MutateType (type);
+ }
}
/// <summary>
}
public class OpcodeCast : TypeCast {
- OpCode op, op2;
- bool second_valid;
+ readonly OpCode op;
public OpcodeCast (Expression child, Type return_type, OpCode op)
: base (child, return_type)
-
- {
- this.op = op;
- second_valid = false;
- }
-
- public OpcodeCast (Expression child, Type return_type, OpCode op, OpCode op2)
- : base (child, return_type)
-
{
this.op = op;
- this.op2 = op2;
- second_valid = true;
}
public override Expression DoResolve (EmitContext ec)
{
base.Emit (ec);
ec.ig.Emit (op);
-
- if (second_valid)
- ec.ig.Emit (op2);
}
public Type UnderlyingType {
/// This kind of cast is used to encapsulate a child and cast it
/// to the class requested
/// </summary>
- public class ClassCast : TypeCast {
+ public sealed class ClassCast : TypeCast {
+ Type child_generic_parameter;
+
public ClassCast (Expression child, Type return_type)
: base (child, return_type)
{
+ if (TypeManager.IsGenericParameter (child.Type))
+ child_generic_parameter = child.Type;
}
public override Expression DoResolve (EmitContext ec)
{
base.Emit (ec);
- if (TypeManager.IsGenericParameter (child.Type))
- ec.ig.Emit (OpCodes.Box, child.Type);
+ if (child_generic_parameter != null)
+ ec.ig.Emit (OpCodes.Box, child_generic_parameter);
#if GMCS_SOURCE
if (type.IsGenericParameter)
#endif
ec.ig.Emit (OpCodes.Castclass, type);
}
+
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ type = storey.MutateType (type);
+ if (child_generic_parameter != null)
+ child_generic_parameter = storey.MutateGenericArgument (child_generic_parameter);
+
+ base.MutateHoistedGenericType (storey);
+ }
}
//
{
return Type.GetHashCode ();
}
+
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ type = storey.MutateType (type);
+ }
}
/// <summary>
Report.Error (831, loc, "An expression tree may not contain a base access");
}
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ if (InstanceExpression != null)
+ InstanceExpression.MutateHoistedGenericType (storey);
+ }
+
// TODO: possible optimalization
// Cache resolved constant result in FieldBuilder <-> expression map
public virtual MemberExpr ResolveMemberAccess (EmitContext ec, Expression left, Location loc,
return base.OverloadResolve (ec, ref arguments, false, loc);
e.ExtensionExpression = ExtensionExpression;
+ e.SetTypeArguments (type_arguments);
return e.ResolveOverloadExtensions (ec, arguments, e.namespace_entry, loc);
}
}
p = Delegate.GetInvokeMethod (null, p).ReturnType;
q = Delegate.GetInvokeMethod (null, q).ReturnType;
+ if (p == TypeManager.void_type && q != TypeManager.void_type)
+ return 2;
+ if (q == TypeManager.void_type && p != TypeManager.void_type)
+ return 1;
} else {
if (argument_type == p)
return 1;
protected virtual void Error_InvalidArguments (EmitContext ec, Location loc, int idx, MethodBase method,
Argument a, ParameterData expected_par, Type paramType)
{
+ ExtensionMethodGroupExpr emg = this as ExtensionMethodGroupExpr;
+
if (a is CollectionElementInitializer.ElementInitializerArgument) {
Report.SymbolRelatedToPreviousError (method);
if ((expected_par.ParameterModifier (idx) & Parameter.Modifier.ISBYREF) != 0) {
TypeManager.CSharpSignature (method));
} else if (delegate_type == null) {
Report.SymbolRelatedToPreviousError (method);
- Report.Error (1502, loc, "The best overloaded method match for `{0}' has some invalid arguments",
- TypeManager.CSharpSignature (method));
+ if (emg != null) {
+ Report.Error (1928, loc,
+ "Type `{0}' does not contain a member `{1}' and the best extension method overload `{2}' has some invalid arguments",
+ emg.ExtensionExpression.GetSignatureForError (),
+ emg.Name, TypeManager.CSharpSignature (method));
+ } else {
+ Report.Error (1502, loc, "The best overloaded method match for `{0}' has some invalid arguments",
+ TypeManager.CSharpSignature (method));
+ }
} else
Report.Error (1594, loc, "Delegate `{0}' has some invalid arguments",
TypeManager.CSharpName (delegate_type));
if (((mod & (Parameter.Modifier.REF | Parameter.Modifier.OUT)) ^
(a.Modifier & (Parameter.Modifier.REF | Parameter.Modifier.OUT))) != 0) {
if ((mod & Parameter.Modifier.ISBYREF) == 0)
- Report.Error (1615, loc, "Argument `{0}' should not be passed with the `{1}' keyword",
+ Report.Error (1615, loc, "Argument `#{0}' does not require `{1}' modifier. Consider removing `{1}' modifier",
index, Parameter.GetModifierSignature (a.Modifier));
else
- Report.Error (1620, loc, "Argument `{0}' must be passed with the `{1}' keyword",
+ Report.Error (1620, loc, "Argument `#{0}' is missing `{1}' modifier",
index, Parameter.GetModifierSignature (mod));
} else {
string p1 = a.GetSignatureForError ();
Report.SymbolRelatedToPreviousError (a.Expr.Type);
Report.SymbolRelatedToPreviousError (paramType);
}
- Report.Error (1503, loc, "Argument {0}: Cannot convert type `{1}' to `{2}'", index, p1, p2);
+
+ if (idx == 0 && emg != null) {
+ Report.Error (1929, loc,
+ "Extension method instance type `{0}' cannot be converted to `{1}'", p1, p2);
+ } else {
+ Report.Error (1503, loc,
+ "Argument `#{0}' cannot convert `{1}' expression to type `{2}'", index, p1, p2);
+ }
}
}
if (a_type != parameter)
return 2;
-
- return 0;
+ } else {
+ if (delegate_type != null ?
+ !Delegate.IsTypeCovariant (argument.Expr, parameter) :
+ !Convert.ImplicitConversionExists (ec, argument.Expr, parameter))
+ return 2;
}
- if (delegate_type != null ?
- !Delegate.IsTypeCovariant (argument.Expr, parameter) :
- !Convert.ImplicitConversionExists (ec, argument.Expr, parameter))
- return 2;
-
if (arg_mod != param_mod)
return 1;
return null;
}
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ base.MutateHoistedGenericType (storey);
+
+ MethodInfo mi = best_candidate as MethodInfo;
+ if (mi != null) {
+ best_candidate = storey.MutateGenericMethod (mi);
+ return;
+ }
+
+ best_candidate = storey.MutateConstructor ((ConstructorInfo) this);
+ }
+
/// <summary>
/// Find the Applicable Function Members (7.4.2.1)
///
}
Report.SetMessageRecorder (prev_recorder);
- if (msg_recorder != null && msg_recorder.PrintMessages ())
+ if (msg_recorder != null && !msg_recorder.IsEmpty) {
+ if (!may_fail)
+ msg_recorder.PrintMessages ();
+
return null;
+ }
int candidate_top = candidates.Count;
/// Fully resolved expression that evaluates to a Field
/// </summary>
public class FieldExpr : MemberExpr, IAssignMethod, IMemoryLocation, IVariable {
- public readonly FieldInfo FieldInfo;
+ public FieldInfo FieldInfo;
+ readonly Type constructed_generic_type;
VariableInfo variable_info;
LocalTemporary temp;
loc = l;
}
+ public FieldExpr (FieldInfo fi, Type genericType, Location l)
+ : this (fi, l)
+ {
+ this.constructed_generic_type = genericType;
+ }
+
public override string Name {
get {
return FieldInfo.Name;
}
}
- AnonymousContainer am = ec.CurrentAnonymousMethod;
- if (am != null){
- if (!FieldInfo.IsStatic){
- if (!am.IsIterator && (ec.TypeContainer is Struct)){
- Report.Error (1673, loc,
- "Anonymous methods inside structs cannot access instance members of `{0}'. Consider copying `{0}' to a local variable outside the anonymous method and using the local instead",
- "this");
- return null;
- }
- }
- }
-
IFixedBuffer fb = AttributeTester.GetFixedBuffer (FieldInfo);
if (fb != null) {
if (!ec.InFixedInitializer && ec.ContainerType.IsValueType) {
// If the instance expression is a local variable or parameter.
IVariable var = InstanceExpression as IVariable;
- if ((var == null) || (var.VariableInfo == null))
+ if (var == null || var.VariableInfo == null)
return this;
VariableInfo vi = var.VariableInfo;
override public Expression DoResolveLValue (EmitContext ec, Expression right_side)
{
IVariable var = InstanceExpression as IVariable;
- if ((var != null) && (var.VariableInfo != null))
+ if (var != null && var.VariableInfo != null)
var.VariableInfo.SetFieldAssigned (ec, FieldInfo.Name);
bool lvalue_instance = !FieldInfo.IsStatic && FieldInfo.DeclaringType.IsValueType;
}
}
- public bool VerifyFixed ()
- {
- IVariable variable = InstanceExpression as IVariable;
- // A variable of the form V.I is fixed when V is a fixed variable of a struct type.
- // We defer the InstanceExpression check after the variable check to avoid a
- // separate null check on InstanceExpression.
- return variable != null && InstanceExpression.Type.IsValueType && variable.VerifyFixed ();
- }
-
public override int GetHashCode ()
{
return FieldInfo.GetHashCode ();
}
+
+ public bool IsFixed {
+ get {
+ IVariable variable = InstanceExpression as IVariable;
+ // A variable of the form V.I is fixed when V is a fixed variable of a struct type.
+ // We defer the InstanceExpression check after the variable check to avoid a
+ // separate null check on InstanceExpression.
+ return variable != null && InstanceExpression.Type.IsValueType && variable.IsFixed;
+ }
+ }
public override bool Equals (object obj)
{
if (FieldInfo.IsStatic){
if (is_volatile)
ig.Emit (OpCodes.Volatile);
-
- ig.Emit (OpCodes.Ldsfld, FieldInfo);
+
+ ig.Emit (OpCodes.Ldsfld, GetConstructedFieldInfo ());
} else {
if (!prepared)
EmitInstance (ec, false);
IFixedBuffer ff = AttributeTester.GetFixedBuffer (FieldInfo);
if (ff != null) {
- ig.Emit (OpCodes.Ldflda, FieldInfo);
+ ig.Emit (OpCodes.Ldflda, GetConstructedFieldInfo ());
ig.Emit (OpCodes.Ldflda, ff.Element);
} else {
if (is_volatile)
ig.Emit (OpCodes.Volatile);
- ig.Emit (OpCodes.Ldfld, FieldInfo);
+ ig.Emit (OpCodes.Ldfld, GetConstructedFieldInfo ());
}
}
}
if (is_static)
- ig.Emit (OpCodes.Stsfld, FieldInfo);
- else
- ig.Emit (OpCodes.Stfld, FieldInfo);
+ ig.Emit (OpCodes.Stsfld, GetConstructedFieldInfo ());
+ else
+ ig.Emit (OpCodes.Stfld, GetConstructedFieldInfo ());
if (temp != null) {
temp.Emit (ec);
temp.Release (ec);
+ temp = null;
}
}
if (FieldInfo.IsStatic){
- ig.Emit (OpCodes.Ldsflda, FieldInfo);
+ ig.Emit (OpCodes.Ldsflda, GetConstructedFieldInfo ());
} else {
if (!prepared)
EmitInstance (ec, false);
- ig.Emit (OpCodes.Ldflda, FieldInfo);
+ ig.Emit (OpCodes.Ldflda, GetConstructedFieldInfo ());
}
}
+
+ FieldInfo GetConstructedFieldInfo ()
+ {
+ if (constructed_generic_type == null)
+ return FieldInfo;
+#if GMCS_SOURCE
+ return TypeBuilder.GetField (constructed_generic_type, FieldInfo);
+#else
+ throw new NotSupportedException ();
+#endif
+ }
+
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ FieldInfo = storey.MutateField (FieldInfo);
+ base.MutateHoistedGenericType (storey);
+ }
}
}
}
+ public override void MutateHoistedGenericType (AnonymousMethodStorey storey)
+ {
+ if (InstanceExpression != null)
+ InstanceExpression.MutateHoistedGenericType (storey);
+
+ type = storey.MutateType (type);
+ getter = storey.MutateGenericMethod (getter);
+ }
+
bool InstanceResolve (EmitContext ec, bool lvalue_instance, bool must_do_cs1540_check)
{
if (is_static) {
string t_name = InstanceExpression.Type.Name;
int t_name_len = t_name.Length;
- return t_name_len > 2 && t_name [t_name_len - 2] == '[' && t_name [t_name_len - 3] != ']';
+ return t_name_len > 2 && t_name [t_name_len - 2] == '[';
}
override public Expression DoResolve (EmitContext ec)
}
}
- public class TemporaryVariable : Expression, IMemoryLocation
+ public class TemporaryVariable : VariableReference
{
LocalInfo li;
- Variable var;
-
+
public TemporaryVariable (Type type, Location loc)
{
this.type = type;
this.loc = loc;
- eclass = ExprClass.Value;
+ eclass = ExprClass.Variable;
}
public override Expression CreateExpressionTree (EmitContext ec)
{
throw new NotSupportedException ("ET");
}
-
+
public override Expression DoResolve (EmitContext ec)
{
if (li != null)
return this;
-
+
TypeExpr te = new TypeExpression (type, loc);
li = ec.CurrentBlock.AddTemporaryVariable (te, loc);
if (!li.Resolve (ec))
return null;
- if (ec.MustCaptureVariable (li)) {
- ScopeInfo scope = li.Block.CreateScopeInfo ();
- var = scope.AddLocal (li);
- type = var.Type;
+ if (ec.MustCaptureVariable (li) && !ec.IsInProbingMode) {
+ AnonymousMethodStorey storey = li.Block.Explicit.CreateAnonymousMethodStorey (ec);
+ storey.CaptureLocalVariable (ec, li);
}
-
- return this;
- }
- public Variable Variable {
- get { return var != null ? var : li.Variable; }
+ return this;
}
public override void Emit (EmitContext ec)
{
- Variable.EmitInstance (ec);
- Variable.Emit (ec);
+ Emit (ec, false);
}
-
- public void EmitLoadAddress (EmitContext ec)
+
+ public void EmitAssign (EmitContext ec, Expression source)
{
- Variable.EmitInstance (ec);
- Variable.EmitAddressOf (ec);
+ EmitAssign (ec, source, false, false);
}
-
- public void Store (EmitContext ec, Expression right_side)
- {
- Variable.EmitInstance (ec);
- right_side.Emit (ec);
- Variable.EmitAssign (ec);
+
+ public override HoistedVariable HoistedVariable {
+ get { return li.HoistedVariableReference; }
}
-
- public void EmitThis (EmitContext ec)
- {
- Variable.EmitInstance (ec);
+
+ public override bool IsFixed {
+ get { return true; }
}
-
- public void EmitStore (EmitContext ec)
- {
- Variable.EmitAssign (ec);
+
+ public override bool IsRef {
+ get { return false; }
}
-
- public void AddressOf (EmitContext ec, AddressOp mode)
- {
- EmitLoadAddress (ec);
+
+ protected override ILocalVariable Variable {
+ get { return li; }
+ }
+
+ public override VariableInfo VariableInfo {
+ get { throw new NotImplementedException (); }
}
}