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

[Vladimir Kozlov]

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