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

[Tobias Hartmann]

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