RFR: 8369190: JavaFrameAnchor on AArch64 has unnecessary barriers and wrong store order in MacroAssembler [v6]

Justin King jcking at openjdk.org
Wed Oct 8 12:31:39 UTC 2025


> Remove unnecessary release barriers in `JavaFrameAnchor::{copy,clear}` and fix `MacroAssembler::set_last_Java_frame` to set `sp` last as expected by the profiler.

Justin King has updated the pull request incrementally with eight additional commits since the last revision:

 - JDK-8369190: JavaFrameAnchor on AArch64 appears to be missing barriers
   
   Signed-off-by: Justin King <jcking at google.com>
 - Revert "JDK-8369190: JavaFrameAnchor on AArch64 appears to be missing barriers"
   
   This reverts commit 462c63e588ebf0dcd478b5347684ccb6c2438686.
 - Revert "Update comments"
   
   This reverts commit a0058eb4405a9afb0db80d9838e0657f725c9a15.
 - Revert "Rename ACGT -> AGCT"
   
   This reverts commit a141449425a72a4095b9740e28c758cb295c9352.
 - Revert "Remove barriers and just fix program order for sp store in MacroAssembler"
   
   This reverts commit 253417f98e7d5680a3ad6f8561b198fc0ae3b141.
 - Revert "Add comment to JavaFrameAnchor regarding no need for fencing"
   
   This reverts commit 016769dfe4272d7cb37d46524f32e9f7a4887b11.
 - Revert "Move compiler_barrier() to globalDefinitions.hpp and use it"
   
   This reverts commit 6dd80d947a343ad6c45d1738614643f7013afcbc.
 - Revert "Add back header inclusion"
   
   This reverts commit 7dc3313c9fd6a4f2739c39dc4d9d075103894aa9.

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/27645/files
  - new: https://git.openjdk.org/jdk/pull/27645/files/7dc3313c..b0f22bab

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=27645&range=05
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=27645&range=04-05

  Stats: 59 lines in 8 files changed: 17 ins; 34 del; 8 mod
  Patch: https://git.openjdk.org/jdk/pull/27645.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/27645/head:pull/27645

PR: https://git.openjdk.org/jdk/pull/27645


More information about the hotspot-dev mailing list