[11] RFR(S): 8199777: Deprecate -XX:+AggressiveOpts

Vladimir Kozlov vladimir.kozlov at oracle.com
Wed Mar 21 17:59:08 UTC 2018


Okay. Looks good.

Thanks,
Vladimir

> On Mar 21, 2018, at 12:38 AM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
> 
> Hi Vladimir and David,
> 
> thanks for the review!
> 
> On 20.03.2018 23:23, Vladimir Kozlov wrote:
>> Actually you don't need to specify EliminateAutoBox in tests because it is true by default:
> 
> Yes, but I agree with David that it's better to leave the flag in to state what the test is supposed
> to be run with. Also, AutoBoxCacheMax is a C2 specific flag as well.
> 
> Here's the new webrev with -XX:+IgnoreUnrecognizedVMOptions added to the tests:
> http://cr.openjdk.java.net/~thartmann/8199777/webrev.01/
> 
> Thanks,
> Tobias



More information about the hotspot-dev mailing list