RFR: 8343541: C1: Plain memory accesses are emitted with membars with +AlwaysAtomicAccesses [v2]
Xiaolong Peng
xpeng at openjdk.org
Mon Nov 18 09:39:59 UTC 2024
On Mon, 18 Nov 2024 09:36:51 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
> > It is a bit sad that we call `volatile_field_*` when we really mean atomic. But renaming that would lead to much deeper changes. So I am okay with doing just this, thanks.
>
> I'll fix it, it is not too deep, and they are only used by barrierSetC1.cpp
You are right, checked the related code, need to update the chain deep to LIR_Assembler, I'll leave them unchanged except if it is really necessary.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/22191#issuecomment-2482422778
More information about the hotspot-dev
mailing list