RFR: 8335662: [AArch64] C1: guarantee(val < (1ULL << nbits)) failed: Field too big for insn [v2]

[Evgeny Astigeevich]

On Wed, 16 Oct 2024 12:09:22 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Chad Rakoczy has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Add blank line at end of test
>>  - Add jasm and update test description
>
> test/hotspot/jtreg/compiler/c1/ComplexLockingAndMultiThreading.jasm line 13:
> 
>> 11:          return;
>> 12:   }
>> 13:   public static Method main:"([Ljava/lang/String;)V"
> 
> There's no need to have the main method in jasm, right? I think it's only `synchronizedMethod` which triggers the issue. So ideally, the jasm file is as compact as possible to ease future maintenance of the test.

IMO we even don't need to run the code to trigger compilation. We can use WhiteBox to compile it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21473#discussion_r1803261299