RFR: 8320682: [AArch64] C1 compilation fails with "Field too big for insn"
Daniel Lundén
duke at openjdk.org
Wed Dec 6 08:39:32 UTC 2023
On Mon, 4 Dec 2023 14:47:32 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
>> 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
>
> Looks good to me.
Thanks for the reviews @TobiHartmann and @theRealAph. Integrating now, but I need a sponsor.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16951#issuecomment-1842428572
More information about the hotspot-compiler-dev
mailing list