[Mono-devel-list] Re: [Mono-patches] r40869 - trunk/mcs/mcs

[Ben Maurer]

Hey,

On Fri, 2005-02-18 at 12:08 -0500, Marek Safar  wrote:
>  
> +				if (RootContext.Optimize) {
> +					Constant c = e as Constant;
> +					if (c != null) {
> +						if (c.IsDefaultValue)
> +							continue;
> +					}
> +				}
> +

I don't see any reason to require /optimize here. This is an
optimization that takes O(1) time, where the constant is *very* small.

Personally, I think /optimize should only be used to enable expensive
optimizations. The goal of the flag is so that the default compile will
take shorter.

-- Ben