RFR: 8325467: Support methods with many arguments in C2 [v6]

[Dean Long]

On Thu, 26 Sep 2024 23:31:52 GMT, Dean Long <dlong 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 incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains nine additional commits since the last revision:
>> 
>>  - Remove leftover debug var
>>  - Update
>>  - Merge tag 'jdk-24+16' into HEAD
>>    
>>    Added tag jdk-24+16 for changeset c58fbef0
>>  - Formatting updates
>>  - Update
>>  - Update after Roberto's comments and suggestions
>>  - Add can_represent asserts
>>  - Remove leftover CHUNK_SIZE reference
>>  - Support methods with many arguments in C2
>
> src/hotspot/share/opto/optoreg.hpp line 208:
> 
>> 206:     assert(can_fit(n + 1), "overflow");
>> 207:     assert(can_fit(n), "overflow");
>> 208:     _second = n + 1;
> 
> I think we could just use checked_cast<>() here and then we don't need can_fit.  What do you think?

I see we still need can_fit() for the static asserts, so nevermind.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20404#discussion_r1777843867