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