RFR: 8343541: C1: Plain memory accesses are emitted with membars with +AlwaysAtomicAccesses [v2]
Aleksey Shipilev
shade at openjdk.org
Mon Nov 18 09:39:58 UTC 2024
On Mon, 18 Nov 2024 09:30:25 GMT, Xiaolong Peng <xpeng 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
No need. If we go down that road, we also want to rename `lir_move_volatile` and all that jazz, which would probably entail a few rounds of careful looking. Submit a follow-up issue for it instead.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/22191#issuecomment-2482422228
More information about the hotspot-dev
mailing list