RFR: 8351666: [PPC64] Make non-volatile VectorRegisters available for C2 register allocation [v4]
Martin Doerr
mdoerr at openjdk.org
Thu Apr 10 15:12:46 UTC 2025
On Wed, 9 Apr 2025 17:24:19 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
>> src/hotspot/cpu/ppc/stubGenerator_ppc.cpp line 220:
>>
>>> 218: BLOCK_COMMENT("Call frame manager or native entry.");
>>> 219: // Call frame manager or native entry.
>>> 220: assert_different_registers(r_arg_entry, r_top_of_arguments_addr, r_arg_method, r_arg_thread);
>>
>> Since you're at it: please adjust the 2 comment lines above: there's no frame manager. We're about to call the interpreter or native entry.
>> Also L222 (just remove "on entry ..."), L245. L265
>
> Alternatively create a cleanup RFE for the stale "frame manager" comments.
I think a separate RFE will be better because "frame manager" is still referenced at more places.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23987#discussion_r2037651201
More information about the hotspot-dev
mailing list