On Fri, 15 May 2026 13:04:02 GMT, Manuel Hässig <[email protected]> wrote:
> This PR updates the Test `compiler/stable/TestStableArrayMembars.java` for > Valhalla. This involves creating a code path for migrated value classes and a > slight change to C2 to be able to clean up more barriers than previously. > > Unfortunately, with `--enable-preview` C2 is not able to fully clean up all > membars as it is for mainline. This is due to the later elision of the loads. > After parsing, membars do not carry information that could relate them to any > particular memory access, so we cannot remove them without potentially > violating the JMM. Further, the implementation of > `Unsafe.getFlatValueAcquire` emits a barrier in Java that cannot be > eliminated by the JIT anyways. > > Testing: > - [x] tier1,tier2 and additional stress testing. > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: f87d6d1c Author: Manuel Hässig <[email protected]> Committer: Tobias Hartmann <[email protected]> URL: https://git.openjdk.org/valhalla/commit/f87d6d1cea3b578b458075a411f7aaa5dddb2cd9 Stats: 56 lines in 3 files changed: 42 ins; 2 del; 12 mod 8380921: [lworld] compiler/stable/TestStableArrayMembars.java IR mismatch with preview enabled Reviewed-by: thartmann ------------- PR: https://git.openjdk.org/valhalla/pull/2441
