RFR: 8351949: RISC-V: Cleanup and enable store-load peephole for membars [v8]

Robbin Ehn rehn at openjdk.org
Thu Apr 3 11:39:55 UTC 2025


On Thu, 3 Apr 2025 11:02:23 GMT, Fei Yang <fyang at openjdk.org> wrote:

>> Seems we should only consider current code base, and not consider possible optimization in the future? : )
>> I'm not objecting current change, it's fine to me too.
>
> Another concern from myside is that the RISC-V ISA is still evolving. People are working on Load-Acquire & Store-Release (Check RISC-V Zalasr extension [[1]](https://github.com/riscv/riscv-zalasr)). From my knowledge on aarch64's support for ldar/stlr instructions, I am expecting quite some change in the WMO part when we have such a similar extension. But I don't think that the TSO part will be affected much. So better to decouple the two for now.

Ok, I'll ship as is.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/24035#discussion_r2026816043


More information about the hotspot-dev mailing list