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