Make all evaluator created type public
[mono.git] / mcs / mcs / constant.cs
index 0b11bcadadf89646b7483ca4441b68390de9b358..81d2c9604c33482226971554f6c8f736937bc64d 100644 (file)
@@ -5,14 +5,19 @@
 //   Miguel de Icaza (miguel@ximian.com)
 //   Marek Safar (marek.safar@seznam.cz)
 //
-// (C) 2001 Ximian, Inc.
-//
+// Copyright 2001-2003 Ximian, Inc.
+// Copyright 2003-2008 Novell, Inc.
 //
 
-namespace Mono.CSharp {
+using System;
 
-       using System;
-       using System.Reflection.Emit;
+#if STATIC
+using IKVM.Reflection.Emit;
+#else
+using System.Reflection.Emit;
+#endif
+
+namespace Mono.CSharp {
 
        /// <summary>
        ///   Base class for constants and literals.
@@ -24,36 +29,9 @@ namespace Mono.CSharp {
                        this.loc = loc;
                }
 
-               /// <remarks>
-               ///   This is different from ToString in that ToString
-               ///   is supposed to be there for debugging purposes,
-               ///   and is not guaranteed to be useful for anything else,
-               ///   AsString() will provide something that can be used
-               ///   for round-tripping C# code.  Maybe it can be used
-               ///   for IL assembly as well.
-               /// </remarks>
-               public abstract string AsString ();
-
                override public string ToString ()
                {
-                       return this.GetType ().Name + " (" + AsString () + ")";
-               }
-
-               public override bool GetAttributableValue (Type valueType, out object value)
-               {
-                       if (valueType == TypeManager.object_type) {
-                               value = GetTypedValue ();
-                               return true;
-                       }
-
-                       Constant c = ImplicitConversionRequired (valueType, loc);
-                       if (c == null) {
-                               value = null;
-                               return false;
-                       }
-
-                       value = c.GetTypedValue ();
-                       return true;
+                       return this.GetType ().Name + " (" + GetValueAsLiteral () + ")";
                }
 
                /// <summary>
@@ -62,28 +40,40 @@ namespace Mono.CSharp {
                /// </summary>
                public abstract object GetValue ();
 
+               public abstract string GetValueAsLiteral ();
+
+#if !STATIC
+               //
+               // Returns an object value which is typed to contant type
+               //
                public virtual object GetTypedValue ()
                {
                        return GetValue ();
                }
+#endif
 
-               /// <summary>
-               ///   Constants are always born in a fully resolved state
-               /// </summary>
-               public override Expression DoResolve (EmitContext ec)
+               public override void Error_ValueCannotBeConverted (ResolveContext ec, Location loc, TypeSpec target, bool expl)
                {
-                       return this;
+                       if (!expl && IsLiteral && 
+                               (TypeManager.IsPrimitiveType (target) || type == TypeManager.decimal_type) &&
+                               (TypeManager.IsPrimitiveType (type) || type == TypeManager.decimal_type)) {
+                               ec.Report.Error (31, loc, "Constant value `{0}' cannot be converted to a `{1}'",
+                                       GetValueAsLiteral (), TypeManager.CSharpName (target));
+                       } else {
+                               base.Error_ValueCannotBeConverted (ec, loc, target, expl);
+                       }
                }
 
-               public Constant ImplicitConversionRequired (Type type, Location loc)
+               public Constant ImplicitConversionRequired (ResolveContext ec, TypeSpec type, Location loc)
                {
-                       Constant c = ConvertImplicitly (type);
+                       Constant c = ConvertImplicitly (ec, type);
                        if (c == null)
-                               Error_ValueCannotBeConverted (null, loc, type, false);
+                               Error_ValueCannotBeConverted (ec, loc, type, false);
+
                        return c;
                }
 
-               public virtual Constant ConvertImplicitly (Type type)
+               public virtual Constant ConvertImplicitly (ResolveContext rc, TypeSpec type)
                {
                        if (this.type == type)
                                return this;
@@ -98,16 +88,22 @@ namespace Mono.CSharp {
                                // We should always catch the error before this is ever
                                // reached, by calling Convert.ImplicitStandardConversionExists
                                //
-                               throw new Exception (
-                                       String.Format ("LookupConstantValue: This should never be reached {0} {1}", Type, type));
+                               throw new InternalErrorException ("Missing constant conversion between `{0}' and `{1}'",
+                                 TypeManager.CSharpName (Type), TypeManager.CSharpName (type));
                        }
 
-                       return CreateConstant (type, constant_value, loc);
+                       return CreateConstant (rc, type, constant_value, loc);
+               }
+
+               //
+               //  Returns a constant instance based on Type
+               //
+               public static Constant CreateConstant (ResolveContext rc, TypeSpec t, object v, Location loc)
+               {
+                       return CreateConstantFromValue (t, v, loc).Resolve (rc);
                }
 
-               ///  Returns a constant instance based on Type
-               ///  The returned value is already resolved.
-               public static Constant CreateConstant (Type t, object v, Location loc)
+               public static Constant CreateConstantFromValue (TypeSpec t, object v, Location loc)
                {
                        if (t == TypeManager.int32_type)
                                return new IntConstant ((int) v, loc);
@@ -138,67 +134,90 @@ namespace Mono.CSharp {
                        if (t == TypeManager.decimal_type)
                                return new DecimalConstant ((decimal) v, loc);
                        if (TypeManager.IsEnumType (t)) {
-                               Type real_type = TypeManager.TypeToCoreType (v.GetType ());
-                               if (real_type == t)
-                                       real_type = System.Enum.GetUnderlyingType (real_type);
-                               return new EnumConstant (CreateConstant (real_type, v, loc), t);
-                       } 
-                       if (v == null && !TypeManager.IsValueType (t))
-                               return new EmptyConstantCast (new NullConstant (loc), t);
+                               var real_type = EnumSpec.GetUnderlyingType (t);
+                               return new EnumConstant (CreateConstantFromValue (real_type, v, loc).Resolve (null), t);
+                       }
+                       if (v == null) {
+                               if (TypeManager.IsNullableType (t))
+                                       return Nullable.LiftedNull.Create (t, loc);
 
-                       throw new Exception ("Unknown type for constant (" + t +
-                                       "), details: " + v);
+                               if (TypeManager.IsReferenceType (t))
+                                       return new NullConstant (t, loc);
+                       }
+
+                       throw new InternalErrorException ("Constant value `{0}' has unexpected underlying type `{1}'",
+                               v, TypeManager.CSharpName (t));
                }
+
+               public override Expression CreateExpressionTree (ResolveContext ec)
+               {
+                       Arguments args = new Arguments (2);
+                       args.Add (new Argument (this));
+                       args.Add (new Argument (new TypeOf (new TypeExpression (type, loc), loc)));
+
+                       return CreateExpressionFactoryCall (ec, "Constant", args);
+               }
+
+
                /// <summary>
                /// Maybe ConvertTo name is better. It tries to convert `this' constant to target_type.
                /// It throws OverflowException 
                /// </summary>
                // DON'T CALL THIS METHOD DIRECTLY AS IT DOES NOT HANDLE ENUMS
-               public abstract Constant ConvertExplicitly (bool inCheckedContext, Type target_type);
+               public abstract Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type);
 
                /// <summary>
                ///   Attempts to do a compile-time folding of a constant cast.
                /// </summary>
-               public Constant TryReduce (EmitContext ec, Type target_type, Location loc)
+               public Constant TryReduce (ResolveContext ec, TypeSpec target_type, Location loc)
                {
                        try {
                                return TryReduce (ec, target_type);
                        }
                        catch (OverflowException) {
-                               Report.Error (221, loc, "Constant value `{0}' cannot be converted to a `{1}' (use `unchecked' syntax to override)",
-                                       GetValue ().ToString (), TypeManager.CSharpName (target_type));
-                               return null;
+                               if (ec.ConstantCheckState && Type.BuildinType != BuildinTypeSpec.Type.Decimal) {
+                                       ec.Report.Error (221, loc,
+                                               "Constant value `{0}' cannot be converted to a `{1}' (use `unchecked' syntax to override)",
+                                               GetValueAsLiteral (), target_type.GetSignatureForError ());
+                               } else {
+                                       Error_ValueCannotBeConverted (ec, loc, target_type, false);
+                               }
+
+                               return New.Constantify (target_type, loc).Resolve (ec);
                        }
                }
 
-               Constant TryReduce (EmitContext ec, Type target_type)
+               Constant TryReduce (ResolveContext ec, TypeSpec target_type)
                {
                        if (Type == target_type)
                                return this;
 
+                       Constant c;
                        if (TypeManager.IsEnumType (target_type)) {
-                               Constant c = TryReduce (ec, TypeManager.EnumToUnderlying (target_type));
+                               c = TryReduce (ec, EnumSpec.GetUnderlyingType (target_type));
                                if (c == null)
                                        return null;
 
-                               return new EnumConstant (c, target_type);
+                               return new EnumConstant (c, target_type).Resolve (ec);
                        }
 
-                       return ConvertExplicitly (ec.ConstantCheckState, target_type);
+                       c = ConvertExplicitly (ec.ConstantCheckState, target_type);
+                       if (c != null)
+                               c = c.Resolve (ec);
+
+                       return c;
                }
 
-               public abstract Constant Increment ();
-               
                /// <summary>
                /// Need to pass type as the constant can require a boxing
                /// and in such case no optimization is possible
                /// </summary>
-               public bool IsDefaultInitializer (Type type)
+               public bool IsDefaultInitializer (TypeSpec type)
                {
                        if (type == Type)
                                return IsDefaultValue;
 
-                       return Type == TypeManager.null_type;
+                       return this is NullLiteral;
                }
 
                public abstract bool IsDefaultValue {
@@ -209,6 +228,17 @@ namespace Mono.CSharp {
                        get;
                }
 
+               //
+               // When constant is declared as literal
+               //
+               public virtual bool IsLiteral {
+                       get { return false; }
+               }
+               
+               public virtual bool IsOneInteger {
+                       get { return false; }
+               }               
+
                //
                // Returns true iff 1) the stack type of this is one of Object, 
                // int32, int64 and 2) this == 0 or this == null.
@@ -216,15 +246,62 @@ namespace Mono.CSharp {
                public virtual bool IsZeroInteger {
                        get { return false; }
                }
+
+               public override void EmitSideEffect (EmitContext ec)
+               {
+                       // do nothing
+               }
+
+               public sealed override Expression Clone (CloneContext clonectx)
+               {
+                       // No cloning is not needed for constants
+                       return this;
+               }
+
+               protected override void CloneTo (CloneContext clonectx, Expression target)
+               {
+                       throw new NotSupportedException ("should not be reached");
+               }
+
+               public override System.Linq.Expressions.Expression MakeExpression (BuilderContext ctx)
+               {
+#if STATIC
+                       return base.MakeExpression (ctx);
+#else
+                       return System.Linq.Expressions.Expression.Constant (GetTypedValue (), type.GetMetaInfo ());
+#endif
+               }
+
+               public new Constant Resolve (ResolveContext rc)
+               {
+                       if (eclass != ExprClass.Unresolved)
+                               return this;
+
+                       // Resolved constant has to be still a constant
+                       Constant c = (Constant) DoResolve (rc);
+                       if (c == null)
+                               return null;
+
+                       if ((c.eclass & ExprClass.Value) == 0) {
+                               c.Error_UnexpectedKind (rc, ResolveFlags.VariableOrValue, loc);
+                               return null;
+                       }
+
+                       if (c.type == null)
+                               throw new InternalErrorException ("Expression `{0}' did not set its type after Resolve", c.GetType ());
+
+                       return c;
+               }
        }
 
-       public abstract class IntegralConstant : Constant {
+       public abstract class IntegralConstant : Constant
+       {
                protected IntegralConstant (Location loc) :
                        base (loc)
                {
                }
 
-               public override void Error_ValueCannotBeConverted (EmitContext ec, Location loc, Type target, bool expl)
+               public override void Error_ValueCannotBeConverted (ResolveContext ec, Location loc, TypeSpec target, bool expl)
                {
                        try {
                                ConvertExplicitly (true, target);
@@ -232,10 +309,17 @@ namespace Mono.CSharp {
                        }
                        catch
                        {
-                               Report.Error (31, loc, "Constant value `{0}' cannot be converted to a `{1}'",
+                               ec.Report.Error (31, loc, "Constant value `{0}' cannot be converted to a `{1}'",
                                        GetValue ().ToString (), TypeManager.CSharpName (target));
                        }
                }
+
+               public override string GetValueAsLiteral ()
+               {
+                       return GetValue ().ToString ();
+               }
+               
+               public abstract Constant Increment ();
        }
        
        public class BoolConstant : Constant {
@@ -243,37 +327,40 @@ namespace Mono.CSharp {
                
                public BoolConstant (bool val, Location loc):
                        base (loc)
+               {
+                       Value = val;
+               }
+
+               protected override Expression DoResolve (ResolveContext ec)
                {
                        type = TypeManager.bool_type;
                        eclass = ExprClass.Value;
+                       return this;
+               }
 
-                       Value = val;
+               public override object GetValue ()
+               {
+                       return (object) Value;
                }
 
-               override public string AsString ()
+               public override string GetValueAsLiteral ()
                {
                        return Value ? "true" : "false";
                }
 
-               public override object GetValue ()
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       return (object) Value;
+                       enc.Encode (Value);
                }
-                               
                
                public override void Emit (EmitContext ec)
                {
                        if (Value)
-                               ec.ig.Emit (OpCodes.Ldc_I4_1);
+                               ec.Emit (OpCodes.Ldc_I4_1);
                        else
-                               ec.ig.Emit (OpCodes.Ldc_I4_0);
+                               ec.Emit (OpCodes.Ldc_I4_0);
                }
 
-               public override Constant Increment ()
-               {
-                       throw new NotSupportedException ();
-               }
-       
                public override bool IsDefaultValue {
                        get {
                                return !Value;
@@ -290,7 +377,7 @@ namespace Mono.CSharp {
                        get { return Value == false; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        return null;
                }
@@ -303,19 +390,24 @@ namespace Mono.CSharp {
                public ByteConstant (byte v, Location loc):
                        base (loc)
                {
-                       type = TypeManager.byte_type;
-                       eclass = ExprClass.Value;
                        Value = v;
                }
 
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
+               {
+                       enc.Encode (Value);
+               }
+
                public override void Emit (EmitContext ec)
                {
-                       IntLiteral.EmitInt (ec.ig, Value);
+                       ec.EmitInt (Value);
                }
 
-               public override string AsString ()
+               protected override Expression DoResolve (ResolveContext ec)
                {
-                       return Value.ToString ();
+                       type = TypeManager.byte_type;
+                       eclass = ExprClass.Value;
+                       return this;
                }
 
                public override object GetValue ()
@@ -334,6 +426,12 @@ namespace Mono.CSharp {
                        }
                }
 
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
+
                public override bool IsNegative {
                        get {
                                return false;
@@ -344,10 +442,10 @@ namespace Mono.CSharp {
                        get { return Value == 0; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > SByte.MaxValue)
                                                throw new OverflowException ();
                                }
@@ -384,18 +482,28 @@ namespace Mono.CSharp {
 
                public CharConstant (char v, Location loc):
                        base (loc)
+               {
+                       Value = v;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
                {
                        type = TypeManager.char_type;
                        eclass = ExprClass.Value;
-                       Value = v;
+                       return this;
+               }
+
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
+               {
+                       enc.Encode ((ushort) Value);
                }
 
                public override void Emit (EmitContext ec)
                {
-                       IntLiteral.EmitInt (ec.ig, Value);
+                       ec.EmitInt (Value);
                }
 
-               static public string descape (char c)
+               static string descape (char c)
                {
                        switch (c){
                        case '\a':
@@ -424,21 +532,16 @@ namespace Mono.CSharp {
                        return c.ToString ();
                }
 
-               public override string AsString ()
-               {
-                       return "\"" + descape (Value) + "\"";
-               }
-
                public override object GetValue ()
                {
                        return Value;
                }
 
-               public override Constant Increment ()
+               public override string GetValueAsLiteral ()
                {
-                       return new CharConstant (checked ((char)(Value + 1)), loc);
+                       return "\"" + descape (Value) + "\"";
                }
-               
+
                public override bool IsDefaultValue {
                        get {
                                return Value == 0;
@@ -455,24 +558,24 @@ namespace Mono.CSharp {
                        get { return Value == '\0'; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Byte.MinValue || Value > Byte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > SByte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > Int16.MaxValue)
                                                throw new OverflowException ();
                                }                                       
@@ -503,20 +606,25 @@ namespace Mono.CSharp {
 
                public SByteConstant (sbyte v, Location loc):
                        base (loc)
+               {
+                       Value = v;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
                {
                        type = TypeManager.sbyte_type;
                        eclass = ExprClass.Value;
-                       Value = v;
+                       return this;
                }
 
-               public override void Emit (EmitContext ec)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       IntLiteral.EmitInt (ec.ig, Value);
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.EmitInt (Value);
                }
 
                public override object GetValue ()
@@ -541,33 +649,39 @@ namespace Mono.CSharp {
                        }
                }
                
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
+               
                public override bool IsZeroInteger {
                        get { return Value == 0; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type)
                                return new ShortConstant ((short) Value, Location);
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new UShortConstant ((ushort) Value, Location);
                        } if (target_type == TypeManager.int32_type)
                                  return new IntConstant ((int) Value, Location);
                        if (target_type == TypeManager.uint32_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new UIntConstant ((uint) Value, Location);
                        } if (target_type == TypeManager.int64_type)
                                  return new LongConstant ((long) Value, Location);
                        if (target_type == TypeManager.uint64_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new ULongConstant ((ulong) Value, Location);
                        }
@@ -576,7 +690,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new CharConstant ((char) Value, Location);
                        }
@@ -593,20 +707,25 @@ namespace Mono.CSharp {
 
                public ShortConstant (short v, Location loc):
                        base (loc)
+               {
+                       Value = v;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
                {
                        type = TypeManager.short_type;
                        eclass = ExprClass.Value;
-                       Value = v;
+                       return this;
                }
 
-               public override void Emit (EmitContext ec)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       IntLiteral.EmitInt (ec.ig, Value);
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.EmitInt (Value);
                }
 
                public override object GetValue ()
@@ -634,25 +753,31 @@ namespace Mono.CSharp {
                                return Value < 0;
                        }
                }
+               
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Byte.MinValue || Value > Byte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < SByte.MinValue || Value > SByte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                
                                return new UShortConstant ((ushort) Value, Location);
@@ -660,14 +785,14 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.int32_type)
                                return new IntConstant ((int) Value, Location);
                        if (target_type == TypeManager.uint32_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new UIntConstant ((uint) Value, Location);
                        }
                        if (target_type == TypeManager.int64_type)
                                return new LongConstant ((long) Value, Location);
                        if (target_type == TypeManager.uint64_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new ULongConstant ((ulong) Value, Location);
                        }
@@ -676,7 +801,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Char.MinValue)
                                                throw new OverflowException ();
                                }
@@ -695,20 +820,25 @@ namespace Mono.CSharp {
 
                public UShortConstant (ushort v, Location loc):
                        base (loc)
+               {
+                       Value = v;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
                {
                        type = TypeManager.ushort_type;
                        eclass = ExprClass.Value;
-                       Value = v;
+                       return this;
                }
 
-               public override void Emit (EmitContext ec)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       IntLiteral.EmitInt (ec.ig, Value);
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.EmitInt (Value);
                }
 
                public override object GetValue ()
@@ -732,29 +862,35 @@ namespace Mono.CSharp {
                                return false;
                        }
                }
+               
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
        
                public override bool IsZeroInteger {
                        get { return Value == 0; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > Byte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > SByte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > Int16.MaxValue)
                                                throw new OverflowException ();
                                }
@@ -773,7 +909,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > Char.MaxValue)
                                                throw new OverflowException ();
                                }
@@ -792,71 +928,24 @@ namespace Mono.CSharp {
                public IntConstant (int v, Location loc):
                        base (loc)
                {
-                       type = TypeManager.int32_type;
-                       eclass = ExprClass.Value;
                        Value = v;
                }
 
-               static public void EmitInt (ILGenerator ig, int i)
+               protected override Expression DoResolve (ResolveContext rc)
                {
-                       switch (i){
-                       case -1:
-                               ig.Emit (OpCodes.Ldc_I4_M1);
-                               break;
-                               
-                       case 0:
-                               ig.Emit (OpCodes.Ldc_I4_0);
-                               break;
-                               
-                       case 1:
-                               ig.Emit (OpCodes.Ldc_I4_1);
-                               break;
-                               
-                       case 2:
-                               ig.Emit (OpCodes.Ldc_I4_2);
-                               break;
-                               
-                       case 3:
-                               ig.Emit (OpCodes.Ldc_I4_3);
-                               break;
-                               
-                       case 4:
-                               ig.Emit (OpCodes.Ldc_I4_4);
-                               break;
-                               
-                       case 5:
-                               ig.Emit (OpCodes.Ldc_I4_5);
-                               break;
-                               
-                       case 6:
-                               ig.Emit (OpCodes.Ldc_I4_6);
-                               break;
-                               
-                       case 7:
-                               ig.Emit (OpCodes.Ldc_I4_7);
-                               break;
-                               
-                       case 8:
-                               ig.Emit (OpCodes.Ldc_I4_8);
-                               break;
-
-                       default:
-                               if (i >= -128 && i <= 127){
-                                       ig.Emit (OpCodes.Ldc_I4_S, (sbyte) i);
-                               } else
-                                       ig.Emit (OpCodes.Ldc_I4, i);
-                               break;
-                       }
+                       type = TypeManager.int32_type;
+                       eclass = ExprClass.Value;
+                       return this;
                }
 
-               public override void Emit (EmitContext ec)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       EmitInt (ec.ig, Value);
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.EmitInt (Value);
                }
 
                public override object GetValue ()
@@ -880,43 +969,49 @@ namespace Mono.CSharp {
                                return Value < 0;
                        }
                }
+               
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
 
                public override bool IsZeroInteger {
                        get { return Value == 0; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Byte.MinValue || Value > Byte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < SByte.MinValue || Value > SByte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Int16.MinValue || Value > Int16.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ShortConstant ((short) Value, Location);
                        }
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < UInt16.MinValue || Value > UInt16.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new UShortConstant ((ushort) Value, Location);
                        }
                        if (target_type == TypeManager.uint32_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < UInt32.MinValue)
                                                throw new OverflowException ();
                                }
@@ -925,7 +1020,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.int64_type)
                                return new LongConstant ((long) Value, Location);
                        if (target_type == TypeManager.uint64_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new ULongConstant ((ulong) Value, Location);
                        }
@@ -934,7 +1029,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Char.MinValue || Value > Char.MaxValue)
                                                throw new OverflowException ();
                                }
@@ -946,16 +1041,16 @@ namespace Mono.CSharp {
                        return null;
                }
 
-               public override Constant ConvertImplicitly (Type type)
+               public override Constant ConvertImplicitly (ResolveContext rc, TypeSpec type)
                {
                        if (this.type == type)
                                return this;
 
                        Constant c = TryImplicitIntConversion (type);
                        if (c != null)
-                               return c;
+                               return c.Resolve (rc);
 
-                       return base.ConvertImplicitly (type);
+                       return base.ConvertImplicitly (rc, type);
                }
 
                /// <summary>
@@ -963,7 +1058,7 @@ namespace Mono.CSharp {
                ///   into a different data type using casts (See Implicit Constant
                ///   Expression Conversions)
                /// </summary>
-               Constant TryImplicitIntConversion (Type target_type)
+               Constant TryImplicitIntConversion (TypeSpec target_type)
                {
                        if (target_type == TypeManager.sbyte_type) {
                                if (Value >= SByte.MinValue && Value <= SByte.MaxValue)
@@ -1008,20 +1103,25 @@ namespace Mono.CSharp {
 
                public UIntConstant (uint v, Location loc):
                        base (loc)
+               {
+                       Value = v;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
                {
                        type = TypeManager.uint32_type;
                        eclass = ExprClass.Value;
-                       Value = v;
+                       return this;
                }
 
-               public override void Emit (EmitContext ec)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       IntLiteral.EmitInt (ec.ig, unchecked ((int) Value));
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.EmitInt (unchecked ((int) Value));
                }
 
                public override object GetValue ()
@@ -1045,43 +1145,49 @@ namespace Mono.CSharp {
                                return false;
                        }
                }
+               
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
 
                public override bool IsZeroInteger {
                        get { return Value == 0; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
-                                       if (Value < Char.MinValue || Value > Char.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < 0 || Value > byte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > SByte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > Int16.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ShortConstant ((short) Value, Location);
                        }
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < UInt16.MinValue || Value > UInt16.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new UShortConstant ((ushort) Value, Location);
                        }
                        if (target_type == TypeManager.int32_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value > Int32.MaxValue)
                                                throw new OverflowException ();
                                }
@@ -1096,7 +1202,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Char.MinValue || Value > Char.MaxValue)
                                                throw new OverflowException ();
                                }
@@ -1116,29 +1222,24 @@ namespace Mono.CSharp {
                public LongConstant (long v, Location loc):
                        base (loc)
                {
-                       type = TypeManager.int64_type;
-                       eclass = ExprClass.Value;
                        Value = v;
                }
 
-               public override void Emit (EmitContext ec)
+               protected override Expression DoResolve (ResolveContext rc)
                {
-                       EmitLong (ec.ig, Value);
+                       type = TypeManager.int64_type;
+                       eclass = ExprClass.Value;
+                       return this;
                }
 
-               static public void EmitLong (ILGenerator ig, long l)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       if (l >= int.MinValue && l <= int.MaxValue) {
-                               IntLiteral.EmitInt (ig, unchecked ((int) l));
-                               ig.Emit (OpCodes.Conv_I8);
-                               return;
-                       }
-                       ig.Emit (OpCodes.Ldc_I8, l);
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.EmitLong (Value);
                }
 
                public override object GetValue ()
@@ -1162,57 +1263,63 @@ namespace Mono.CSharp {
                                return Value < 0;
                        }
                }
+               
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
 
                public override bool IsZeroInteger {
                        get { return Value == 0; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Byte.MinValue || Value > Byte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < SByte.MinValue || Value > SByte.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Int16.MinValue || Value > Int16.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new ShortConstant ((short) Value, Location);
                        }
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < UInt16.MinValue || Value > UInt16.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new UShortConstant ((ushort) Value, Location);
                        }
                        if (target_type == TypeManager.int32_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Int32.MinValue || Value > Int32.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new IntConstant ((int) Value, Location);
                        }
                        if (target_type == TypeManager.uint32_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < UInt32.MinValue || Value > UInt32.MaxValue)
                                                throw new OverflowException ();
                                }
                                return new UIntConstant ((uint) Value, Location);
                        }
                        if (target_type == TypeManager.uint64_type) {
-                               if (inCheckedContext && Value < 0)
+                               if (in_checked_context && Value < 0)
                                        throw new OverflowException ();
                                return new ULongConstant ((ulong) Value, Location);
                        }
@@ -1221,7 +1328,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext){
+                               if (in_checked_context){
                                        if (Value < Char.MinValue || Value > Char.MaxValue)
                                                throw new OverflowException ();
                                }
@@ -1233,13 +1340,13 @@ namespace Mono.CSharp {
                        return null;
                }
 
-               public override Constant ConvertImplicitly (Type type)
+               public override Constant ConvertImplicitly (ResolveContext rc, TypeSpec type)
                {
                        if (Value >= 0 && type == TypeManager.uint64_type) {
-                               return new ULongConstant ((ulong) Value, loc);
+                               return new ULongConstant ((ulong) Value, loc).Resolve (rc);
                        }
 
-                       return base.ConvertImplicitly (type);
+                       return base.ConvertImplicitly (rc, type);
                }
        }
 
@@ -1249,21 +1356,24 @@ namespace Mono.CSharp {
                public ULongConstant (ulong v, Location loc):
                        base (loc)
                {
-                       type = TypeManager.uint64_type;
-                       eclass = ExprClass.Value;
                        Value = v;
                }
 
-               public override void Emit (EmitContext ec)
+               protected override Expression DoResolve (ResolveContext rc)
                {
-                       ILGenerator ig = ec.ig;
+                       type = TypeManager.uint64_type;
+                       eclass = ExprClass.Value;
+                       return this;
+               }
 
-                       LongLiteral.EmitLong (ig, unchecked ((long) Value));
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
+               {
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.EmitLong (unchecked ((long) Value));
                }
 
                public override object GetValue ()
@@ -1287,45 +1397,51 @@ namespace Mono.CSharp {
                                return false;
                        }
                }
+               
+               public override bool IsOneInteger {
+                       get {
+                               return Value == 1;
+                       }
+               }               
 
                public override bool IsZeroInteger {
                        get { return Value == 0; }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext && Value > Byte.MaxValue)
+                               if (in_checked_context && Value > Byte.MaxValue)
                                        throw new OverflowException ();
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext && Value > ((ulong) SByte.MaxValue))
+                               if (in_checked_context && Value > ((ulong) SByte.MaxValue))
                                        throw new OverflowException ();
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext && Value > ((ulong) Int16.MaxValue))
+                               if (in_checked_context && Value > ((ulong) Int16.MaxValue))
                                        throw new OverflowException ();
                                return new ShortConstant ((short) Value, Location);
                        }
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext && Value > UInt16.MaxValue)
+                               if (in_checked_context && Value > UInt16.MaxValue)
                                        throw new OverflowException ();
                                return new UShortConstant ((ushort) Value, Location);
                        }
                        if (target_type == TypeManager.int32_type) {
-                               if (inCheckedContext && Value > UInt32.MaxValue)
+                               if (in_checked_context && Value > UInt32.MaxValue)
                                        throw new OverflowException ();
                                return new IntConstant ((int) Value, Location);
                        }
                        if (target_type == TypeManager.uint32_type) {
-                               if  (inCheckedContext && Value > UInt32.MaxValue)
+                               if  (in_checked_context && Value > UInt32.MaxValue)
                                        throw new OverflowException ();
                                return new UIntConstant ((uint) Value, Location);
                        }
                        if (target_type == TypeManager.int64_type) {
-                               if (inCheckedContext && Value > Int64.MaxValue)
+                               if (in_checked_context && Value > Int64.MaxValue)
                                        throw new OverflowException ();
                                return new LongConstant ((long) Value, Location);
                        }
@@ -1334,7 +1450,7 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext && Value > Char.MaxValue)
+                               if (in_checked_context && Value > Char.MaxValue)
                                        throw new OverflowException ();
                                return new CharConstant ((char) Value, Location);
                        }
@@ -1351,20 +1467,25 @@ namespace Mono.CSharp {
 
                public FloatConstant (float v, Location loc):
                        base (loc)
+               {
+                       Value = v;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
                {
                        type = TypeManager.float_type;
                        eclass = ExprClass.Value;
-                       Value = v;
+                       return this;
                }
 
-               public override void Emit (EmitContext ec)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       ec.ig.Emit (OpCodes.Ldc_R4, Value);
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.Emit (OpCodes.Ldc_R4, Value);
                }
 
                public override object GetValue ()
@@ -1372,9 +1493,9 @@ namespace Mono.CSharp {
                        return Value;
                }
 
-               public override Constant Increment ()
+               public override string GetValueAsLiteral ()
                {
-                       return new FloatConstant (checked(Value + 1), loc);
+                       return Value.ToString ();
                }
 
                public override bool IsDefaultValue {
@@ -1389,60 +1510,60 @@ namespace Mono.CSharp {
                        }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
-                                       if (Value < byte.MinValue || Value > byte.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < byte.MinValue || Value > byte.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
-                                       if (Value <  sbyte.MinValue || Value > sbyte.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < sbyte.MinValue || Value > sbyte.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext){
-                                       if (Value < short.MinValue || Value > short.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < short.MinValue || Value > short.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new ShortConstant ((short) Value, Location);
                        }
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext){
-                                       if (Value < ushort.MinValue || Value > ushort.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < ushort.MinValue || Value > ushort.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new UShortConstant ((ushort) Value, Location);
                        }
                        if (target_type == TypeManager.int32_type) {
-                               if (inCheckedContext){
-                                       if (Value < int.MinValue || Value > int.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < int.MinValue || Value > int.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new IntConstant ((int) Value, Location);
                        }
                        if (target_type == TypeManager.uint32_type) {
-                               if (inCheckedContext){
-                                       if (Value < uint.MinValue || Value > uint.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < uint.MinValue || Value > uint.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new UIntConstant ((uint) Value, Location);
                        }
                        if (target_type == TypeManager.int64_type) {
-                               if (inCheckedContext){
-                                       if (Value < long.MinValue || Value > long.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < long.MinValue || Value > long.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new LongConstant ((long) Value, Location);
                        }
                        if (target_type == TypeManager.uint64_type) {
-                               if (inCheckedContext){
-                                       if (Value < ulong.MinValue || Value > ulong.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < ulong.MinValue || Value > ulong.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new ULongConstant ((ulong) Value, Location);
@@ -1450,8 +1571,8 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.double_type)
                                return new DoubleConstant ((double) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext){
-                                       if (Value < (float) char.MinValue || Value > (float) char.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < (float) char.MinValue || Value > (float) char.MaxValue || float.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new CharConstant ((char) Value, Location);
@@ -1469,20 +1590,25 @@ namespace Mono.CSharp {
 
                public DoubleConstant (double v, Location loc):
                        base (loc)
+               {
+                       Value = v;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
                {
                        type = TypeManager.double_type;
                        eclass = ExprClass.Value;
-                       Value = v;
+                       return this;
                }
 
-               public override void Emit (EmitContext ec)
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       ec.ig.Emit (OpCodes.Ldc_R8, Value);
+                       enc.Encode (Value);
                }
 
-               public override string AsString ()
+               public override void Emit (EmitContext ec)
                {
-                       return Value.ToString ();
+                       ec.Emit (OpCodes.Ldc_R8, Value);
                }
 
                public override object GetValue ()
@@ -1490,9 +1616,9 @@ namespace Mono.CSharp {
                        return Value;
                }
 
-               public override Constant Increment ()
+               public override string GetValueAsLiteral ()
                {
-                       return new DoubleConstant (checked(Value + 1), loc);
+                       return Value.ToString ();
                }
 
                public override bool IsDefaultValue {
@@ -1507,60 +1633,60 @@ namespace Mono.CSharp {
                        }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.byte_type) {
-                               if (inCheckedContext){
-                                       if (Value < Byte.MinValue || Value > Byte.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < Byte.MinValue || Value > Byte.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new ByteConstant ((byte) Value, Location);
                        }
                        if (target_type == TypeManager.sbyte_type) {
-                               if (inCheckedContext){
-                                       if (Value < SByte.MinValue || Value > SByte.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < SByte.MinValue || Value > SByte.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new SByteConstant ((sbyte) Value, Location);
                        }
                        if (target_type == TypeManager.short_type) {
-                               if (inCheckedContext){
-                                       if (Value < short.MinValue || Value > short.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < short.MinValue || Value > short.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new ShortConstant ((short) Value, Location);
                        }
                        if (target_type == TypeManager.ushort_type) {
-                               if (inCheckedContext){
-                                       if (Value < ushort.MinValue || Value > ushort.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < ushort.MinValue || Value > ushort.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new UShortConstant ((ushort) Value, Location);
                        }
                        if (target_type == TypeManager.int32_type) {
-                               if (inCheckedContext){
-                                       if (Value < int.MinValue || Value > int.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < int.MinValue || Value > int.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new IntConstant ((int) Value, Location);
                        }
                        if (target_type == TypeManager.uint32_type) {
-                               if (inCheckedContext){
-                                       if (Value < uint.MinValue || Value > uint.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < uint.MinValue || Value > uint.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new UIntConstant ((uint) Value, Location);
                        }
                        if (target_type == TypeManager.int64_type) {
-                               if (inCheckedContext){
-                                       if (Value < long.MinValue || Value > long.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < long.MinValue || Value > long.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new LongConstant ((long) Value, Location);
                        }
                        if (target_type == TypeManager.uint64_type) {
-                               if (inCheckedContext){
-                                       if (Value < ulong.MinValue || Value > ulong.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < ulong.MinValue || Value > ulong.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new ULongConstant ((ulong) Value, Location);
@@ -1568,8 +1694,8 @@ namespace Mono.CSharp {
                        if (target_type == TypeManager.float_type)
                                return new FloatConstant ((float) Value, Location);
                        if (target_type == TypeManager.char_type) {
-                               if (inCheckedContext){
-                                       if (Value < (double) char.MinValue || Value > (double) char.MaxValue)
+                               if (in_checked_context){
+                                       if (Value < (double) char.MinValue || Value > (double) char.MaxValue || double.IsNaN (Value))
                                                throw new OverflowException ();
                                }
                                return new CharConstant ((char) Value, Location);
@@ -1588,57 +1714,81 @@ namespace Mono.CSharp {
                public DecimalConstant (decimal d, Location loc):
                        base (loc)
                {
-                       type = TypeManager.decimal_type;
-                       eclass = ExprClass.Value;
                        Value = d;
                }
 
-               override public string AsString ()
+               protected override Expression DoResolve (ResolveContext rc)
                {
-                       return Value.ToString ();
+                       type = TypeManager.decimal_type;
+                       eclass = ExprClass.Value;
+                       return this;
                }
 
                public override object GetValue ()
                {
-                       return (object) Value;
+                       return Value;
                }
 
-               public override void Emit (EmitContext ec)
+               public override string GetValueAsLiteral ()
                {
-                       ILGenerator ig = ec.ig;
+                       return Value.ToString () + "M";
+               }
 
-                       int [] words = Decimal.GetBits (Value);
+               public override void Emit (EmitContext ec)
+               {
+                       int [] words = decimal.GetBits (Value);
                        int power = (words [3] >> 16) & 0xff;
 
-                       if (power == 0 && Value <= int.MaxValue && Value >= int.MinValue)
-                       {
-                               IntConstant.EmitInt (ig, (int)Value);
-                               ig.Emit (OpCodes.Newobj, TypeManager.void_decimal_ctor_int_arg);
-                               return;
-                       }
+                       if (power == 0) {
+                               if (Value <= int.MaxValue && Value >= int.MinValue) {
+                                       if (TypeManager.void_decimal_ctor_int_arg == null) {
+                                               TypeManager.void_decimal_ctor_int_arg = TypeManager.GetPredefinedConstructor (
+                                                       TypeManager.decimal_type, loc, TypeManager.int32_type);
 
-                       
-                       //
-                       // FIXME: we could optimize this, and call a better 
-                       // constructor
-                       //
+                                               if (TypeManager.void_decimal_ctor_int_arg == null)
+                                                       return;
+                                       }
+
+                                       ec.EmitInt ((int) Value);
+                                       ec.Emit (OpCodes.Newobj, TypeManager.void_decimal_ctor_int_arg);
+                                       return;
+                               }
+
+                               if (Value <= long.MaxValue && Value >= long.MinValue) {
+                                       if (TypeManager.void_decimal_ctor_long_arg == null) {
+                                               TypeManager.void_decimal_ctor_long_arg = TypeManager.GetPredefinedConstructor (
+                                                       TypeManager.decimal_type, loc, TypeManager.int64_type);
+
+                                               if (TypeManager.void_decimal_ctor_long_arg == null)
+                                                       return;
+                                       }
 
-                       IntConstant.EmitInt (ig, words [0]);
-                       IntConstant.EmitInt (ig, words [1]);
-                       IntConstant.EmitInt (ig, words [2]);
+                                       ec.EmitLong ((long) Value);
+                                       ec.Emit (OpCodes.Newobj, TypeManager.void_decimal_ctor_long_arg);
+                                       return;
+                               }
+                       }
+
+                       ec.EmitInt (words [0]);
+                       ec.EmitInt (words [1]);
+                       ec.EmitInt (words [2]);
 
                        // sign
-                       IntConstant.EmitInt (ig, words [3] >> 31);
+                       ec.EmitInt (words [3] >> 31);
 
                        // power
-                       IntConstant.EmitInt (ig, power);
+                       ec.EmitInt (power);
 
-                       ig.Emit (OpCodes.Newobj, TypeManager.void_decimal_ctor_five_args);
-               }
+                       if (TypeManager.void_decimal_ctor_five_args == null) {
+                               TypeManager.void_decimal_ctor_five_args = TypeManager.GetPredefinedConstructor (
+                                       TypeManager.decimal_type, loc, TypeManager.int32_type, TypeManager.int32_type,
+                                       TypeManager.int32_type, TypeManager.bool_type, TypeManager.byte_type);
 
-               public override Constant Increment ()
-               {
-                       return new DecimalConstant (checked (Value + 1), loc);
+                               if (TypeManager.void_decimal_ctor_five_args == null)
+                                       return;
+                       }
+
+                       ec.Emit (OpCodes.Newobj, TypeManager.void_decimal_ctor_five_args);
                }
 
                public override bool IsDefaultValue {
@@ -1653,7 +1803,7 @@ namespace Mono.CSharp {
                        }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        if (target_type == TypeManager.sbyte_type)
                                return new SByteConstant ((sbyte)Value, loc);
@@ -1689,33 +1839,58 @@ namespace Mono.CSharp {
                public StringConstant (string s, Location loc):
                        base (loc)
                {
-                       type = TypeManager.string_type;
-                       eclass = ExprClass.Value;
                        Value = s;
                }
 
-               // FIXME: Escape the string.
-               override public string AsString ()
+               protected override Expression DoResolve (ResolveContext rc)
                {
-                       return "\"" + Value + "\"";
+                       type = TypeManager.string_type;
+                       eclass = ExprClass.Value;
+                       return this;
                }
 
                public override object GetValue ()
                {
                        return Value;
                }
+
+               public override string GetValueAsLiteral ()
+               {
+                       // FIXME: Escape the string.
+                       return "\"" + Value + "\"";
+               }
                
                public override void Emit (EmitContext ec)
                {
-                       if (Value == null)
-                               ec.ig.Emit (OpCodes.Ldnull);
-                       else
-                               ec.ig.Emit (OpCodes.Ldstr, Value);
+                       if (Value == null) {
+                               ec.Emit (OpCodes.Ldnull);
+                               return;
+                       }
+
+                       //
+                       // Use string.Empty for both literals and constants even if
+                       // it's not allowed at language level
+                       //
+                       if (Value.Length == 0 && RootContext.Optimize && ec.CurrentType != TypeManager.string_type) {
+                               if (TypeManager.string_empty == null)
+                                       TypeManager.string_empty = TypeManager.GetPredefinedField (TypeManager.string_type, "Empty", loc, TypeManager.string_type);
+
+                               if (TypeManager.string_empty != null) {
+                                       ec.Emit (OpCodes.Ldsfld, TypeManager.string_empty);
+                                       return;
+                               }
+                       }
+
+                       ec.Emit (OpCodes.Ldstr, Value);
                }
 
-               public override Constant Increment ()
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
                {
-                       throw new NotSupportedException ();
+                       // cast to object
+                       if (type != targetType)
+                               enc.Encode (type);
+
+                       enc.Encode (Value);
                }
 
                public override bool IsDefaultValue {
@@ -1730,12 +1905,205 @@ namespace Mono.CSharp {
                        }
                }
 
-               public override Constant ConvertExplicitly (bool inCheckedContext, Type target_type)
+               public override bool IsNull {
+                       get {
+                               return IsDefaultValue;
+                       }
+               }
+
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
                {
                        return null;
                }
        }
 
-}
+       //
+       // Null constant can have its own type, think of `default (Foo)'
+       //
+       public class NullConstant : Constant
+       {
+               public NullConstant (TypeSpec type, Location loc)
+                       : base (loc)
+               {
+                       eclass = ExprClass.Value;
+                       this.type = type;
+               }
+
+               public override Expression CreateExpressionTree (ResolveContext ec)
+               {
+                       if (type == InternalType.Null || type == TypeManager.object_type) {
+                               // Optimized version, also avoids referencing literal internal type
+                               Arguments args = new Arguments (1);
+                               args.Add (new Argument (this));
+                               return CreateExpressionFactoryCall (ec, "Constant", args);
+                       }
+
+                       return base.CreateExpressionTree (ec);
+               }
+
+               protected override Expression DoResolve (ResolveContext ec)
+               {
+                       return this;
+               }
+
+               public override void EncodeAttributeValue (IMemberContext rc, AttributeEncoder enc, TypeSpec targetType)
+               {
+                       // Type it as string cast
+                       if (targetType == TypeManager.object_type || targetType == InternalType.Null)
+                               enc.Encode (TypeManager.string_type);
+
+                       var ac = targetType as ArrayContainer;
+                       if (ac != null) {
+                               if (ac.Rank != 1 || ac.Element.IsArray)
+                                       base.EncodeAttributeValue (rc, enc, targetType);
+                               else
+                                       enc.Encode (uint.MaxValue);
+                       } else {
+                               enc.Encode (byte.MaxValue);
+                       }
+               }
+
+               public override void Emit (EmitContext ec)
+               {
+                       ec.Emit (OpCodes.Ldnull);
+
+                       // Only to make verifier happy
+                       if (TypeManager.IsGenericParameter (type))
+                               ec.Emit (OpCodes.Unbox_Any, type);
+               }
+
+               public override string ExprClassName {
+                       get {
+                               return GetSignatureForError ();
+                       }
+               }
+
+               public override Constant ConvertExplicitly (bool inCheckedContext, TypeSpec targetType)
+               {
+                       if (targetType.IsPointer) {
+                               if (IsLiteral || this is NullPointer)
+                                       return new EmptyConstantCast (new NullPointer (loc), targetType);
 
+                               return null;
+                       }
+
+                       // Exlude internal compiler types
+                       if (targetType.Kind == MemberKind.InternalCompilerType && targetType != InternalType.Dynamic && targetType != InternalType.Null)
+                               return null;
+
+                       if (!IsLiteral && !Convert.ImplicitStandardConversionExists (this, targetType))
+                               return null;
+
+                       if (TypeManager.IsReferenceType (targetType))
+                               return new NullConstant (targetType, loc);
+
+                       if (TypeManager.IsNullableType (targetType))
+                               return Nullable.LiftedNull.Create (targetType, loc);
+
+                       return null;
+               }
+
+               public override Constant ConvertImplicitly (ResolveContext rc, TypeSpec targetType)
+               {
+                       return ConvertExplicitly (false, targetType);
+               }
+
+               public override string GetSignatureForError ()
+               {
+                       return "null";
+               }
 
+               public override object GetValue ()
+               {
+                       return null;
+               }
+
+               public override string GetValueAsLiteral ()
+               {
+                       return GetSignatureForError ();
+               }
+
+               public override bool IsDefaultValue {
+                       get { return true; }
+               }
+
+               public override bool IsNegative {
+                       get { return false; }
+               }
+
+               public override bool IsNull {
+                       get { return true; }
+               }
+
+               public override bool IsZeroInteger {
+                       get { return true; }
+               }
+       }
+
+       /// <summary>
+       ///   The value is constant, but when emitted has a side effect.  This is
+       ///   used by BitwiseAnd to ensure that the second expression is invoked
+       ///   regardless of the value of the left side.  
+       /// </summary>
+       public class SideEffectConstant : Constant {
+               public Constant value;
+               Expression side_effect;
+               
+               public SideEffectConstant (Constant value, Expression side_effect, Location loc) : base (loc)
+               {
+                       this.value = value;
+                       while (side_effect is SideEffectConstant)
+                               side_effect = ((SideEffectConstant) side_effect).side_effect;
+                       this.side_effect = side_effect;
+               }
+
+               protected override Expression DoResolve (ResolveContext rc)
+               {
+                       value = value.Resolve (rc);
+
+                       type = value.Type;
+                       eclass = ExprClass.Value;
+                       return this;
+               }
+
+               public override object GetValue ()
+               {
+                       return value.GetValue ();
+               }
+
+               public override string GetValueAsLiteral ()
+               {
+                       return value.GetValueAsLiteral ();
+               }
+
+               public override void Emit (EmitContext ec)
+               {
+                       side_effect.EmitSideEffect (ec);
+                       value.Emit (ec);
+               }
+
+               public override void EmitSideEffect (EmitContext ec)
+               {
+                       side_effect.EmitSideEffect (ec);
+                       value.EmitSideEffect (ec);
+               }
+
+               public override bool IsDefaultValue {
+                       get { return value.IsDefaultValue; }
+               }
+
+               public override bool IsNegative {
+                       get { return value.IsNegative; }
+               }
+
+               public override bool IsZeroInteger {
+                       get { return value.IsZeroInteger; }
+               }
+
+               public override Constant ConvertExplicitly (bool in_checked_context, TypeSpec target_type)
+               {
+                       Constant new_value = value.ConvertExplicitly (in_checked_context, target_type);
+                       return new_value == null ? null : new SideEffectConstant (new_value, side_effect, new_value.Location);
+               }
+       }
+}