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

[Andrew Haley]

On Fri, 7 Jun 2024 13:59:40 GMT, Xiaolong Peng <duke at openjdk.org> wrote:

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

Marked as reviewed by aph (Reviewer).

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

PR Review: https://git.openjdk.org/jdk21u-dev/pull/665#pullrequestreview-2112421308