Withdrawn: 8310939: [c1] The visibility of write-volatile requires membar_volatile instead of membar
SUN Guoyun
duke at openjdk.org
Thu Jul 20 11:04:54 UTC 2023
On Tue, 27 Jun 2023 12:46:14 GMT, SUN Guoyun <duke at openjdk.org> wrote:
> For c1 now, a volatile write case:
>
> membar_release // LoadStore | StoreStore
> write volatile
> membar
>
> Just like c2, here `membar` should be defined `membar_volatile` clearly, then for risc-v, ppc and loongarch can use StoreLoad for `membar_volatile` for better performance.
>
> Testing:
> GHA testing
> jtreg tier1-3 for loongarch64
This pull request has been closed without being integrated.
-------------
PR: https://git.openjdk.org/jdk/pull/14677
More information about the hotspot-gc-dev
mailing list