RFR: 8341194: [REDO] Implement C2 VectorizedHashCode on AArch64
Tobias Hartmann
thartmann at openjdk.org
Mon Oct 7 05:53:34 UTC 2024
On Tue, 1 Oct 2024 12:48:21 GMT, Mikhail Ablakatov <duke at openjdk.org> wrote:
> 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.
Testing looks good. I leave it to the original reviewers to re-review this.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/21286#issuecomment-2395964292
More information about the hotspot-dev
mailing list