Re: [9] RFR (S): 8038261: JSR292: cache and reuse typed array accessors

2014-07-08 Thread Vladimir Ivanov
>>> I'd like to revive review this review thread. Updated version: http://cr.openjdk.java.net/~vlivanov/8038261/webrev.03/ +1 A v. minor point. There is one newly added method InvokerBytecodeGenerator.match that is not used by this patch or the other one for 8037209. I dunno if it will be us

Re: [9] RFR (S): 8038261: JSR292: cache and reuse typed array accessors

2014-07-08 Thread Paul Sandoz
On Jul 8, 2014, at 12:40 PM, Paul Sandoz wrote: > > On Jul 8, 2014, at 12:09 PM, Vladimir Ivanov > wrote: > >> I'd like to revive review this review thread. >> >> Updated version: >> http://cr.openjdk.java.net/~vlivanov/8038261/webrev.03/ >> > > +1 > A v. minor point. There is one newly

Re: [9] RFR (S): 8038261: JSR292: cache and reuse typed array accessors

2014-07-08 Thread Paul Sandoz
On Jul 8, 2014, at 12:09 PM, Vladimir Ivanov wrote: > I'd like to revive review this review thread. > > Updated version: > http://cr.openjdk.java.net/~vlivanov/8038261/webrev.03/ > +1 Paul. signature.asc Description: Message signed with OpenPGP using GPGMail __

Re: [9] RFR (S): 8038261: JSR292: cache and reuse typed array accessors

2014-07-08 Thread Vladimir Ivanov
I'd like to revive review this review thread. Updated version: http://cr.openjdk.java.net/~vlivanov/8038261/webrev.03/ Thanks! Best regards, Vladimir Ivanov On 3/24/14 10:10 PM, Vladimir Ivanov wrote: http://cr.openjdk.java.net/~vlivanov/8038261/webrev.00 https://bugs.openjdk.java.net/browse/

[9] RFR (S): 8038261: JSR292: cache and reuse typed array accessors

2014-03-24 Thread Vladimir Ivanov
http://cr.openjdk.java.net/~vlivanov/8038261/webrev.00 https://bugs.openjdk.java.net/browse/JDK-8038261 56 lines changed: 26 ins; 14 del; 16 mod Cache typed array element getters/setters and reuse them. Initially, it was part of 8037209, but I decided to integrate it separately. Contributed-by