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