On 2 May 2016, at 23:37, Paul Sandoz <paul.san...@oracle.com> wrote:
> Hi, > > Please review: > > > http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8155794-checkIndex-bifunc-internal-jdk/webrev/ > > http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8155794-checkIndex-bifunc-internal-hotspot/webrev/ The changes look fine to me Paul. -Chris. > This patch is based on that for 8155258 (VarHandle impl improvements) [1] > sent previously. > > The hotspot changes are really small. Likewise for the 8155258 changes is > there is precedent in such cases to push through jdk9-dev rather than hs? > > CCC reviewers strongly indicated for the advanced methods that can customise > the exceptions: "You aren't gonna need it”. > > For expediency I propose to move such methods to an internal class > jdk.internal.util.Preconditions. I would still like to sweep through > java.base and leverage these methods while preserving exception reporting > where possible. > > The hotspot changes are just for renaming of the intrinsic method signatures. > Since the intrinsic method is now internal i have added an @ForceInline on > the corresponding public method, given the potential for this to be used in > performance sensitive code. > > > JPRT core and hotspot tests pass. > > Paul. > > [1] http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-May/040740.html > <http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-May/040740.html>
