Integrated: 8320682: [AArch64] C1 compilation fails with "Field too big for insn"

[Daniel Lundén]

On Mon, 4 Dec 2023 14:19:10 GMT, Daniel Lundén <duke at openjdk.org> wrote:

> This changeset fixes an issue where addresses for float and double constants on aarch64 were sometimes out of range for PC-relative offsets using `adr`.
> 
> Changes:
> - Set an upper bound of `1M` for the flag `NMethodSizeLimit`, ensuring that float and double constants are in range for `adr`.
> - Revise tests in `TestC1Globals.java` to use the new upper bound of `1M` for `NMethodSizeLimit`. Also, remove no longer applicable tests in `TestC1Globals.java`.
> 
> ### Testing (in progress)
> Platforms: windows-x64, linux-x64, linux-aarch64, macosx-x64, macosx-aarch64
> - tier1, tier2, tier3, tier4, tier5
> - Targeted and repeated tests for `TestC1Globals.java` in all tiers

This pull request has now been integrated.

Changeset: 69014cd5
Author:    Daniel Lundén <daniel.lunden at oracle.com>
Committer: Roberto Castañeda Lozano <rcastanedalo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/69014cd55b59a0a63f4918fad575a6887640573e
Stats:     36 lines in 2 files changed: 3 ins; 23 del; 10 mod

8320682: [AArch64] C1 compilation fails with "Field too big for insn"

Reviewed-by: thartmann, aph, dlong

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

PR: https://git.openjdk.org/jdk/pull/16951