Reference.reachabilityFence

[Paul Sandoz]

Hi,

I have updated the patches:

  http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8133348-reachability-fence-jdk/webrev/
  http://cr.openjdk.java.net/~psandoz/jdk9/JDK-8133348-reachability-fence-hotspot/webrev/

There is now more documentation on Reference (copied and suitable rearranged from 166 Fences.java). The method name remains the same.

I have included a ReachabilityFenceTest derived from the tests by Aleksey and Peter. The test uses iterations rather than time, to be more certain that test methods will get OSR’ed and then compiled. It runs quite quickly. (I spruced up the inlining test in the hotspot patch but i will probably remove it).


Ongoing some future aspects to consider (from conversation with John):

1) Using java.lang.invoke.DontInline would be nice. There is a certain pressure building to move this and other related annotations into an internal non-exported package. I will follow up with that.

2) It would be useful to have a guard method call in reachabilityFence that throws an exception (perhaps as an assert) if the method was inlined into another method. At the moment i don’t quite know how to do this.

Paul.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 841 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/attachments/20151126/ee1a057f/signature.asc>