RFR: 8227083: ZGC: C2: Generates on_weak barrier for WeakCompareAndSwap
Nils Eliasson
nils.eliasson at oracle.com
Tue Jul 2 13:01:46 UTC 2019
Looks good!
// Nils
On 2019-07-02 14:52, Stefan Karlsson wrote:
> Hi all,
>
> Please review this patch to fix so that we generate an on_strong load
> barrier for WeakCompareAndSwap.
>
> http://cr.openjdk.java.net/~stefank/8227083/webrev.01/
> https://bugs.openjdk.java.net/browse/JDK-8227083
>
> ErikÖ spotted this problem when we were looking into the bug for
> JDK-8225770, where C2 also generated the wrong kind of barrier.
>
> In this bug there was a mix-up of terms, the weak nature of
> WeakCompareAndSwap is unrelated to the on_weak property of some load
> barriers. The barrier in this code path should be using the normal,
> on_strong, load barries.
>
> The verification code in JDK-8227085 did catch this bug, as expected,
> and with the fix for this bug we don't fail in the verification code
> anymore.
>
> Tested with tier 1-7.
>
> Thanks,
> StefanK
More information about the hotspot-compiler-dev
mailing list