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

Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Jan 24 07:16:02 PST 2014


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 obsoletejvmflags (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 (obsoletejvmflags) 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