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