RFR: 8320682: [AArch64] C1 compilation fails with "Field too big for insn" [v2]
Daniel Lundén
duke at openjdk.org
Thu Dec 7 15:03:54 UTC 2023
> This changeset fixes an issue on aarch64 where addresses for float and double constants were sometimes out of range for PC-relative offsets using `adr`.
>
> Changes:
> - Fix the issue by replacing `adr` with `lea`.
> - Add a regression test.
>
> Thanks to @fisk and @xmas92 for the assistance.
>
> ### Testing
> Tests: tier1, tier2, tier3, tier4, tier5
> Platforms: windows-x64, linux-x64, linux-aarch64, macosx-x64, macosx-aarch64
Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
Revert fix and restrict NMethodSizeLimit instead
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/16951/files
- new: https://git.openjdk.org/jdk/pull/16951/files/ed27abec..a35bad0e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=16951&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=16951&range=00-01
Stats: 44 lines in 3 files changed: 0 ins; 35 del; 9 mod
Patch: https://git.openjdk.org/jdk/pull/16951.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/16951/head:pull/16951
PR: https://git.openjdk.org/jdk/pull/16951
More information about the hotspot-compiler-dev
mailing list