Re: RFR 8151706: Update VarHandle implementation to use @Stable arrays

2016-04-11 Thread Claes Redestad
Looks good /Claes On 2016-04-11 12:55, Paul Sandoz wrote: On 8 Apr 2016, at 16:11, Michael Haupt wrote: Hi Paul, note this is a lower-case review. Having looked at 8151705, thumbs up for this one as well - they go hand in hand and looking at one of them only doesn't feel right. :-) Thank

Re: RFR 8151706: Update VarHandle implementation to use @Stable arrays

2016-04-11 Thread Paul Sandoz
> On 8 Apr 2016, at 16:11, Michael Haupt wrote: > > Hi Paul, > > note this is a lower-case review. Having looked at 8151705, thumbs up for > this one as well - they go hand in hand and looking at one of them only > doesn't feel right. :-) > Thanks. I got ‘em swapped the wrong way around. I

Re: RFR 8151706: Update VarHandle implementation to use @Stable arrays

2016-04-08 Thread Aleksey Shipilev
On 04/08/2016 12:56 PM, Paul Sandoz wrote: > http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8151706-VH-form-table-stable/webrev/ > > > Now that @Stable arrays are supported by C1 (thanks Vladimir!) we > can switch fr

Re: RFR 8151706: Update VarHandle implementation to use @Stable arrays

2016-04-08 Thread Michael Haupt
Hi Paul, note this is a lower-case review. Having looked at 8151705, thumbs up for this one as well - they go hand in hand and looking at one of them only doesn't feel right. :-) Best, Michael > Am 08.04.2016 um 11:56 schrieb Paul Sandoz : > > Hi, > > Please review: > > > http://cr.openj

RFR 8151706: Update VarHandle implementation to use @Stable arrays

2016-04-08 Thread Paul Sandoz
Hi, Please review: http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8151706-VH-form-table-stable/webrev/ Now that @Stable arrays are supported by C1 (thanks Vladimir!) we can switch from the explicit use of Mem