[jdk21u-dev] RFR: 8320379: C2: Sort spilling/unspilling sequence for better ld/st merging into ldp/stp on AArch64 [v2]

Xiaolong Peng duke at openjdk.org
Fri Jun 7 13:59:40 UTC 2024


> Backport JDK-8320379: C2: Sort spilling/unspilling sequence for better ld/st merging into ldp/stp on AArch64
> 
> ### Validation
> 
> Tested on AWS Graviton instance by running ```JTREG_KEYWORDS="!headful & !external-dep & !printer" make test TEST=all```, same the test for PR on jdk17, only saw few test failures in CAInterop.java and compiler/intrinsics. 
> 
> - CAInterop tests are known intermittently failing tests, it should be fine to ignore. 
> - The test failures from compiler/intrinsics has error message "Option 'UseSHA3Intrinsics' is expected to have 'true' value Option 'UseSHA3Intrinsics' should be enabled by default", it seems a bug but not related to the backport, will follow up we need to create a bug.

Xiaolong Peng has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:

 - Merge branch 'openjdk:master' into backport
 - Backport 3ccd02f14211a3384f27fba1633f9d8421378c9a

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

Changes:
  - all: https://git.openjdk.org/jdk21u-dev/pull/665/files
  - new: https://git.openjdk.org/jdk21u-dev/pull/665/files/bf8a0da8..ef11ef0a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=665&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk21u-dev&pr=665&range=00-01

  Stats: 1328 lines in 26 files changed: 1213 ins; 54 del; 61 mod
  Patch: https://git.openjdk.org/jdk21u-dev/pull/665.diff
  Fetch: git fetch https://git.openjdk.org/jdk21u-dev.git pull/665/head:pull/665

PR: https://git.openjdk.org/jdk21u-dev/pull/665


More information about the jdk-updates-dev mailing list