On Wed, 16 Sep 2026 05:58:02 GMT, Fei Yang <[email protected]> wrote:
> Besides stress tests like `jcstress`, I think it will be safer to have some > dedicated low-level tests to validate volatile load and store code emission > with this Zalasr extension. Like the `TestVolatiles*` family under > `test/hotspot/jtreg/compiler/c2/aarch64/` for the aarch64 counterpart [1]. We > already have a similar test subdirectory there > (`test/hotspot/jtreg/compiler/c2/riscv64/`) where we could put the new tests. > Would you consider that? > > [1] https://mail.openjdk.org/pipermail/aarch64-port-dev/2018-June/006119.html Yes, that makes sense. I've added dedicated C2 code-generation tests under compiler/c2/riscv64, modeled after the AArch64 TestVolatiles* tests, to verify the Zalasr acquire/release instructions and the expected membar elimination. BTW, All these tests have passed on QEMU system (both with Zalasr enabled and disabled). ------------- PR Comment: https://git.openjdk.org/jdk/pull/32309#issuecomment-5697049573
