RFR: 8325467: Support methods with many arguments in C2 [v24]
Daniel Lundén
dlunden at openjdk.org
Tue Sep 16 09:09:30 UTC 2025
On Tue, 2 Sep 2025 14:05:25 GMT, Emanuel Peter <epeter at openjdk.org> wrote:
>> Sure, we can rename them. I think `RM_SIZE_IN_INTS` and `RM_SIZE_IN_WORDS` would be most suitable. I avoided such a change in this changeset to not make it bigger than it already is. Isn't it easier to do the renaming in a follow-up RFE though, instead of before this PR? I'm fine with both though, not that much extra work to do it before.
>
> I think it would be easier to review if you do it first.
> That PR won't be super controversial, and just makes the code nicer.
> And then when we come back here, we may even be able to drop some comments, or be able to catch bugs just because the reviewers understand better what's going on ;)
Closing this thread as https://github.com/openjdk/jdk/pull/27215 is now integrated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20404#discussion_r2351579319
More information about the hotspot-compiler-dev
mailing list