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

Roberto Castañeda Lozano rcastanedalo at openjdk.org
Tue Aug 20 16:07:53 UTC 2024


On Tue, 20 Aug 2024 14:03:37 GMT, Daniel Lundén <dlunden at openjdk.org> wrote:

>> src/hotspot/share/opto/regmask.hpp line 144:
>> 
>>> 142: 
>>> 143:   // Return a suitable arena for (extended) register mask allocation.
>>> 144:   static Arena* _get_arena();
>> 
>> Maybe inline this function into its only user?
>
> I would like to, but that results in a cyclic header inclusion problem between `regmask.hpp` and `compile.hpp`. It could maybe be solved, but it doesn't look trivial. Putting the definition in `regmask.cpp` seems like the simplest solution, but I'm open to suggestions.

I see, fair enough, thanks.

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

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


More information about the hotspot-compiler-dev mailing list