review for 7103784: enable some flags by default

[Tom Rodriguez]

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.

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.

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.
>>