RFR: 8342156: C2: Compilation failure with fewer arguments after JDK-8329032 [v2]

Daniel Lundén dlunden at openjdk.org
Tue Oct 22 09:54:43 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:

  Apply suggestions from code review
  
  Co-authored-by: Roberto Castañeda Lozano <robcasloz at users.noreply.github.com>

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21612/files
  - new: https://git.openjdk.org/jdk/pull/21612/files/b6ceb7a8..6f3ccc40

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

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 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