[11] RFR(S): 8199777: Deprecate -XX:+AggressiveOpts (original) (raw)
Tobias Hartmann tobias.hartmann at oracle.com
Thu Mar 22 07:23:12 UTC 2018
- Previous message: [11] RFR(S): 8199777: Deprecate -XX:+AggressiveOpts
- Next message: [11] RFR(S): 8199777: Deprecate -XX:+AggressiveOpts
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Thanks Vladimir and David!
Best regards, Tobias
On 21.03.2018 23:11, David Holmes wrote:
+1 (in case there was any doubt)
David On 22/03/2018 3:59 AM, Vladimir Kozlov wrote: 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
- Previous message: [11] RFR(S): 8199777: Deprecate -XX:+AggressiveOpts
- Next message: [11] RFR(S): 8199777: Deprecate -XX:+AggressiveOpts
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]