RFR: 8272723: Don't use Access API to access primitive fields [v2]

Erik Österlund eosterlund at openjdk.java.net
Fri Aug 20 07:38:32 UTC 2021


On Fri, 20 Aug 2021 07:33:07 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

>> src/hotspot/share/prims/unsafe.cpp line 233:
>> 
>>> 231:   T get_volatile() {
>>> 232:     GuardUnsafeAccess guard(_thread);
>>> 233:     volatile T ret = Atomic::load_acquire(addr());
>> 
>> Is MO_SEQ_CST to load_acquire mapping correct?
>
> No it is not.

> Is MO_SEQ_CST to load_acquire mapping correct?

We should probably retain RawAccess<MO_SEQ_CST> here to get the right semantics.

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

PR: https://git.openjdk.java.net/jdk/pull/5187


More information about the hotspot-dev mailing list