RFR: 8351666: [PPC64] Make non-volatile VectorRegisters available for C2 register allocation [v15]
Martin Doerr
mdoerr at openjdk.org
Wed Apr 16 20:27:48 UTC 2025
On Wed, 16 Apr 2025 19:55:36 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
>> Martin Doerr has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Support displacement for new load/storeV16_Power9 nodes.
>
> src/hotspot/cpu/ppc/macroAssembler_ppc.cpp line 807:
>
>> 805: addi(spill_addr, dst, offset);
>> 806: stxvd2x(as_VectorRegister(i)->to_vsr(), spill_addr);
>> 807: }
>
> This distinction isn't strictly necessary, is it?
It isn't necessary for correctness. But, it may be confusing to have the elements on stack in a different order than everywhere else.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23987#discussion_r2047693009
More information about the hotspot-dev
mailing list