[11] RFR(S): 8199777: Deprecate -XX:+AggressiveOpts (original) (raw)
David Holmes david.holmes at oracle.com
Wed Mar 21 22:11:51 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 ]
+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 ]