RFR(M): 8209835: Aarch64: elide barriers on all volatile operations

Roland Westrelin rwestrel at redhat.com
Fri Oct 5 09:10:52 UTC 2018


>> http://cr.openjdk.java.net/~roland/8209835/webrev.00/
>> 
>> This adds barrier elision support for all volatile operations that were
>> not supported so far and extends the tests to cover all of them. I ran
>> full jcstress successfully with this.
> Yes, that looks good.

Thanks for the review.

> Did you also eyeball the generated code? I know jcstress ought to find
> any bugs but it would be good to be doubly sure :-)

I didn't eyeball the code (there's a lot to check!). I extended tests to
cover all combination of atomic operations/argument types. So at least,
tests should guarantee that the right variant in the ad file is
picked. Then, of course, all variants must be correctly implemented. For
that, I checked a couple times that the right boolean flag was passed.

Roland.


More information about the hotspot-compiler-dev mailing list