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

[Aleksey Shipilev]

Thanks!

-Aleksey.

On 01/24/2014 07:16 PM, Vladimir Kozlov wrote:
> Good. I will push it.
> 
> thanks,
> Vladimir
> 
> On 1/24/14 3:29 AM, Aleksey Shipilev wrote:
>> Ok, here's the update:
>>   http://cr.openjdk.java.net/~shade/8032490/webrev.04/
>>
>> (There is also the changeset ready to ease the push)
>>
>> -Aleksey.
>>
>> On 01/24/2014 08:13 AM, Vladimir Kozlov wrote:
>>> 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.
>>>>
>>