review for 7103784: enable some flags by default

[Vladimir Kozlov]

On 10/23/11 12:22 PM, Tom Rodriguez wrote:
>
> On Oct 21, 2011, at 6:41 PM, Vladimir Kozlov wrote:
>
>> What about EliminateAutoBox and OptimizeFill? Should we remove corresponding AggressiveOpts code in arguments.cpp?
>
> I added OptimizeFill to this webrev and removed the code to turn it on from arguments.cpp.

Looks good. Could you, please, also remove EA code from arguments.cpp since it is ON by default for long time already.

>
> I filed a bug a while to turn on EliminateAutoBox but I also wanted to move some code out from under that flag that wasn't strictly part of that functionality.  That's http://cr.openjdk.java.net/~never/6934604.  I'd like to push that after we see how the nightlies look after turning on the others.

I agree to push EAB after this push. Changes looks good.

Thanks,
Vladimir

>
> tom
>
>>
>> Thanks,
>> Vladimir
>>
>> On 10/21/11 6:10 PM, Tom Rodriguez wrote:
>>> http://cr.openjdk.java.net/~never/7103784
>>> 3 lines changed: 0 ins; 0 del; 3 mod; 4396 unchg
>>>
>>> 7103784: enable some flags by default
>>> Reviewed-by:
>>>
>>> Enable UseCodeCacheFlushing, UseRDPCForConstantTableBase and
>>> OptimizeStringConcat by default.  UseCodeCacheFlushing and
>>> OptimizeStringConcat are tested by various settings in our nightlies
>>> so there should only be minor issues.  UseRDPCForConstantTableBase has
>>> been tested and was only off because the performance was unclear but
>>> we believe it performs well on newer hardware.
>>>
>