On Mon, 1 Jun 2026 07:16:42 GMT, Joel Sikström <[email protected]> wrote:
> Hello, > > The StoreStore membar in the AArch64 implementation of TemplateTable::aaload > is redundant after > [JDK-8378862](https://bugs.openjdk.org/browse/JDK-8378862), which calls > `OrderAccess::storestore()` in the VM instead. > > I suggest we remove it so we don't have a double storestore-membar for this > path. I don't think we need a comment here explaining that the VM handles > memory ordering for us. > > Testing: > * Oracle's tier1-4 on linux-aarch64 release/fastdebug. > > > > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). lgtm. Thanks for fixing this. It is much easier to have model where the runtime calls encapsulate the JMM rather than having to mitigate for this in the JITed code. ------------- Marked as reviewed by aboldtch (Committer). PR Review: https://git.openjdk.org/valhalla/pull/2495#pullrequestreview-4399675875
