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

[David Holmes]

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