RFR: 8325467: Support methods with many arguments in C2 [v27]
Daniel Lundén
dlunden at openjdk.org
Tue Sep 16 11:31:04 UTC 2025
On Tue, 16 Sep 2025 09:45:07 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> Daniel Lundén has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 39 commits:
>>
>> - Clarify comments in regmask.hpp
>> - Merge remote-tracking branch 'upstream/master' into many-arguments-8325467+pr-updates
>> - Address review comments (renaming on the way in a separate PR)
>> - Update src/hotspot/share/opto/regmask.hpp
>>
>> Co-authored-by: Emanuel Peter <emanuel.peter at oracle.com>
>> - Restore modified java/lang/invoke tests
>> - Sort includes (new requirement)
>> - Merge remote-tracking branch 'upstream/master' into many-arguments-8325467+pr-updates
>> - Add clarifying comments at definitions of register mask sizes
>> - Fix implicit zero and nullptr checks
>> - Add deep copy comment
>> - ... and 29 more: https://git.openjdk.org/jdk/compare/60930a3e...c1f41288
>
> src/hotspot/share/adlc/formsopt.cpp line 174:
>
>> 172: // The array of Register Mask bits should be large enough to cover all the
>> 173: // machine registers and usually all parameters that need to be passed on the
>> 174: // stack (stack registers) up to some interesting limit. On Intel, the limit
>
> What do you mean by `usually`? It could be misunderstood that sometimes it may not be large enough to even cover those "upt to some interesting limit". Consider rephrasing for clarity ;)
Sure, I'll rephrase it
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20404#discussion_r2352096087
More information about the hotspot-compiler-dev
mailing list