RFR (S) 8032490: Remove -XX:+-UseOldInlining (original) (raw)
Vladimir Kozlov vladimir.kozlov at oracle.com
Fri Jan 24 07:16:02 PST 2014
- Previous message: RFR (S) 8032490: Remove -XX:+-UseOldInlining
- Next message: RFR (S) 8032490: Remove -XX:+-UseOldInlining
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
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.
- Previous message: RFR (S) 8032490: Remove -XX:+-UseOldInlining
- Next message: RFR (S) 8032490: Remove -XX:+-UseOldInlining
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
More information about the hotspot-compiler-dev mailing list