RFR: 8325467: Support methods with many arguments in C2 [v16]
Emanuel Peter
epeter at openjdk.org
Wed Apr 23 08:49:14 UTC 2025
On Wed, 9 Apr 2025 11:32:16 GMT, Daniel Lundén <dlunden at openjdk.org> wrote:
>> Daniel Lundén has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Update comments
>> - Revise TestNestedSynchronize to make use of CompileFramework
>
> This is a rather intrusive changeset that may also affect ports not supported by Oracle. @offamitkumar @TheRealMDoerr @snazarkin @bulasevich @RealFYang: Can you please test these changes on your respective ports? In particular, please make sure to run the tests `compiler/arguments/TestMaxMethodArguments.java` and `compiler/locks/TestNestedSynchronize.java`.
@dlunde Again: thanks for working on this! It looks like a lot of work, and the existing code was not exactly in the best stlye already 😅 So don't get discouraged by my many comments, a lot of them are small things anyway, and many are just nits.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/20404#issuecomment-2823527474
More information about the hotspot-compiler-dev
mailing list