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).

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

Commit messages:
 - 8385673: [lworld] Redundant StoreStore membar in AArch64 
TemplateTable::aaload

Changes: https://git.openjdk.org/valhalla/pull/2495/files
  Webrev: https://webrevs.openjdk.org/?repo=valhalla&pr=2495&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8385673
  Stats: 3 lines in 1 file changed: 0 ins; 3 del; 0 mod
  Patch: https://git.openjdk.org/valhalla/pull/2495.diff
  Fetch: git fetch https://git.openjdk.org/valhalla.git pull/2495/head:pull/2495

PR: https://git.openjdk.org/valhalla/pull/2495

Reply via email to