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

Aleksey Shipilev aleksey.shipilev at oracle.com
Fri Jan 24 03:29:58 PST 2014


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



More information about the hotspot-compiler-dev mailing list