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