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

Daniel Lundén dlunden at openjdk.org
Tue Sep 23 14:51:14 UTC 2025


On Fri, 28 Mar 2025 13:24:09 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:

>> Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Extend example with offset register mask
>
> As we discussed offline, the test coverage of register mask operations with extended dynamic parts, non-zero offsets, etc. is fairly low (basically limited to the new JTReg tests included in this changeset). To increase coverage, I have extended `test_regmask.cpp` with tests that perform random operations on a register mask and on a reference bit set and check that the result is equivalent on both data structures. Here is the extension: https://github.com/openjdk/jdk/commit/4ee703f1ab73f8f43d4603d7fa88dcc8f4950ec0. I ran the random tests a few times on different platforms and could not find any failure, which gives a good confidence of the correctness of the register mask operation changes. I also tested the effectiveness of the tests themselves by injecting a few failures in the register mask implementation and confirming their detection. Feel free to include the test extensions in this changeset (you might want to go through the code and clean it up a bit before, though, things l
 ike e.g. naming consistency).

Thanks @robcasloz, updated! I'm running some additional sanity testing for the changeset currently. I plan to integrate tomorrow.

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

PR Comment: https://git.openjdk.org/jdk/pull/20404#issuecomment-3324353001


More information about the hotspot-compiler-dev mailing list