RFR: 8342156: C2: Compilation failure with fewer arguments after JDK-8329032 [v3]
Daniel Lundén
dlunden at openjdk.org
Tue Oct 29 10:42:26 UTC 2024
> Adding C2 register allocation support for APX EGPRs ([JDK-8329032](https://bugs.openjdk.org/browse/JDK-8329032)) reduced, due to unfortunate rounding in the register mask size computation, the available space for incoming/outgoing method arguments in register masks.
>
> ### Changeset
>
> - Bump the number of 32-bit words dedicated to incoming/outgoing arguments in register masks from 3 to 4.
> - Add a regression test.
>
> ### Testing
>
> - [GitHub Actions](https://github.com/dlunde/jdk/actions/runs/11436050131)
> - `tier1` to `tier4` (and additional Oracle-internal testing) on Windows x64, Linux x64, Linux aarch64, macOS x64, and macOS aarch64.
> - C2 compilation time benchmarking for DaCapo on Windows x64, Linux x64, Linux aarch64, macOS x64, and macOS aarch64. There is no observable difference in C2 compilation time.
Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
Update mask size comment after suggestions
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/21612/files
- new: https://git.openjdk.org/jdk/pull/21612/files/6f3ccc40..873a8ffe
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=21612&range=02
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=21612&range=01-02
Stats: 7 lines in 1 file changed: 3 ins; 0 del; 4 mod
Patch: https://git.openjdk.org/jdk/pull/21612.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/21612/head:pull/21612
PR: https://git.openjdk.org/jdk/pull/21612
More information about the hotspot-compiler-dev
mailing list