RFR: JDK8 C2 fixes

Roman Kennke rkennke at redhat.com
Thu Dec 15 16:09:42 UTC 2016


Am Donnerstag, den 15.12.2016, 16:57 +0100 schrieb Roland Westrelin:
> > - in inline_unsafe_access(), read-barriers should be moved up,
> > otherwise we'd have one store in the else branch that does not have
> > a
> > read-barrier on its value.
> 
> Is this one required? The else branch stores outside the heap as I
> understand.

You are right, it's not needed. The problem goes away just with the
arraycopy fix:

http://cr.openjdk.java.net/~rkennke/jdk8-c2-fix/webrev.01/

Will push that then...

Roman



More information about the shenandoah-dev mailing list