RFR (S) 8032490: Remove -XX:+-UseOldInlining

[Vladimir Kozlov]

Yes, I think it is reasonable to put it into obsolete_jvm_flags (in 
arguments.cpp) but not CCC. Thank you, Christian for reminding.

Vladimir

On 1/23/14 7:23 PM, Christian Thalinger wrote:
> Since it’s a product flag:
>
> -  product(bool, UseOldInlining, true,                                       \
> -          "Enable the 1.3 inlining strategy")                               \
>
> do we have to put it in the obsolete flags list (obsolete_jvm_flags) or even file a CCC request?  I wouldn’t think so because turning UseOldInlining off didn’t work anyway, as far as I know.
>
> On Jan 23, 2014, at 3:25 PM, Aleksey Shipilev <aleksey.shipilev at oracle.com> wrote:
>
>> On 01/23/2014 08:39 PM, Aleksey Shipilev wrote:
>>> On 01/23/2014 08:35 PM, Vladimir Kozlov wrote:
>>>> On 1/23/14 8:28 AM, Aleksey Shipilev wrote:
>>>>> Right, thanks for spotting that!
>>>>>
>>>>> Updated:
>>>>>    http://cr.openjdk.java.net/~shade/8032490/webrev.01/
>>>>
>>>> Good.
>>
>> Rebased for jdk9/hs-comp:
>>   http://cr.openjdk.java.net/~shade/8032490/webrev.02/
>>
>>
>> -Aleksey.
>