On Thu, 25 Jun 2026 12:13:14 GMT, Johan Sjölen <[email protected]> wrote:

>> This isn't sufficient, it needs a Java volatile load as it needs to be 
>> ordered with a different variable. This just happens to work with our 
>> current choice of loadAcquire instruction on ARM64 (as @merykitty has 
>> pointed out).
>
> Maybe a full fence is the cleaner expression here? It directly says that 
> everything before the fence, including the successful exchange, stays before 
> everything after it, including ordinary loads.
> 
> @DougLea is the expert, though :P

I've already changed this to getVolatile locally, this is why I moved this PR 
to draft - just completing testing/perf before committing

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/31465#discussion_r3477514739

Reply via email to