RFR: 8341194: [REDO] Implement C2 VectorizedHashCode on AArch64 [v2]

Mikhail Ablakatov duke at openjdk.org
Tue Oct 8 09:47:47 UTC 2024


> Fix and redo [JDK-8322770](https://bugs.openjdk.org/browse/JDK-8322770) which got backed out in [JDK-8341197](https://bugs.openjdk.org/browse/JDK-8341197) due to test failures / jlink failure. Initial PR: https://github.com/openjdk/jdk/pull/18487.
> 
> Compared to the initial PR, this one eliminates a mismatch between the `arrays_hashcode()` template and stub. The template defined v0-v7 & v12-v17 as TEMP  registers but the stub used v8-v9 which were not on the list. Changing v8-v9 in the stub to v12-v13 eliminates this mismatch. Other changes in aarch64.ad  is just a cleanup as some registers on the list were not used by the stub at all anymore.
> 
> Following the discussion on [JDK-8341197](https://bugs.openjdk.org/browse/JDK-8341197), we've added a `make bootcycle-images` step to our local CI pipeline to verify that the issue is fixed.

Mikhail Ablakatov has updated the pull request incrementally with one additional commit since the last revision:

  Ensure that the inline code and the stub use the same registers

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21286/files
  - new: https://git.openjdk.org/jdk/pull/21286/files/90e68228..9c6ad7a5

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21286&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21286&range=00-01

  Stats: 37 lines in 5 files changed: 32 ins; 2 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/21286.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21286/head:pull/21286

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


More information about the hotspot-dev mailing list