Re: [9, 8u40] RFR (XXS): 8059880: Get rid of LambdaForm interpretation

2014-11-24 Thread Paul Sandoz
On Nov 19, 2014, at 11:24 AM, Vladimir Ivanov wrote: > Aleksey, Duncan, thanks for the review and the confirmation that it doesn't > break stuff for you. > > Any Reviews, please? :-) > Looks good. > Best regards, > Vladimir Ivanov > > On 11/19/14, 2:23 PM, MacGregor, Duncan (GE Energy Man

Re: [9, 8u40] RFR (XXS): 8059880: Get rid of LambdaForm interpretation

2014-11-19 Thread Vladimir Ivanov
Thanks, Vladimir! Best regards, Vladimir Ivanov On 11/19/14, 9:20 PM, Vladimir Kozlov wrote: As far as I can guess :) this change looks good. Reviewed. Thanks, Vladimir On 11/19/14 2:24 AM, Vladimir Ivanov wrote: Aleksey, Duncan, thanks for the review and the confirmation that it doesn't bre

Re: [9, 8u40] RFR (XXS): 8059880: Get rid of LambdaForm interpretation

2014-11-19 Thread Vladimir Kozlov
As far as I can guess :) this change looks good. Reviewed. Thanks, Vladimir On 11/19/14 2:24 AM, Vladimir Ivanov wrote: Aleksey, Duncan, thanks for the review and the confirmation that it doesn't break stuff for you. Any Reviews, please? :-) Best regards, Vladimir Ivanov On 11/19/14, 2:23 P

Re: [9, 8u40] RFR (XXS): 8059880: Get rid of LambdaForm interpretation

2014-11-19 Thread Vladimir Ivanov
Aleksey, Duncan, thanks for the review and the confirmation that it doesn't break stuff for you. Any Reviews, please? :-) Best regards, Vladimir Ivanov On 11/19/14, 2:23 PM, MacGregor, Duncan (GE Energy Management) wrote: On 18/11/2014 23:33, "Aleksey Shipilev" wrote: On 11/19/2014 12:01 AM

Re: [9, 8u40] RFR (XXS): 8059880: Get rid of LambdaForm interpretation

2014-11-19 Thread MacGregor, Duncan (GE Energy Management)
On 18/11/2014 23:33, "Aleksey Shipilev" wrote: >On 11/19/2014 12:01 AM, Vladimir Ivanov wrote: >> http://cr.openjdk.java.net/~vlivanov/8059880/webrev.00/ >> https://bugs.openjdk.java.net/browse/JDK-8059880 > >Yes, for the love of God, GO FOR IT. Seconded. Startup of our stuff seems fine now with

Re: [9, 8u40] RFR (XXS): 8059880: Get rid of LambdaForm interpretation

2014-11-18 Thread Aleksey Shipilev
On 11/19/2014 12:01 AM, Vladimir Ivanov wrote: > http://cr.openjdk.java.net/~vlivanov/8059880/webrev.00/ > https://bugs.openjdk.java.net/browse/JDK-8059880 Yes, for the love of God, GO FOR IT. -Aleksey. signature.asc Description: OpenPGP digital signature __

[9, 8u40] RFR (XXS): 8059880: Get rid of LambdaForm interpretation

2014-11-18 Thread Vladimir Ivanov
http://cr.openjdk.java.net/~vlivanov/8059880/webrev.00/ https://bugs.openjdk.java.net/browse/JDK-8059880 Once full LambdaForm sharing is on by default, amount of instantiated LambdaForms drastically reduces. On Octane it varies from 1k to 3k depending on the subbenchmark (was up to 300k before