RFR: 8351666: [PPC64] Make non-volatile VectorRegisters available for C2 register allocation [v17]
Martin Doerr
mdoerr at openjdk.org
Tue Apr 22 15:18:01 UTC 2025
On Tue, 22 Apr 2025 14:48:01 GMT, Richard Reingruber <rrich at openjdk.org> wrote:
>> Martin Doerr has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 26 additional commits since the last revision:
>>
>> - Merge remote-tracking branch 'origin' into 8351666_PPC64_nv_VRs
>> - C1: Avoid set_callee_saved for high halves like in sharedRuntime RegisterSaver.
>> - Fix regName for VSR VMRegs.
>> - Support displacement for new load/storeV16_Power9 nodes.
>> - Fix OopMap for Power10 Little Endian.
>> - Use consistent vector element ordering and simplify OopMap code.
>> - Handle alignment.
>> - Change VSR reg_defs to Op_RegF.
>> - Improve VMReg handling.
>> - Handle alignment.
>> - ... and 16 more: https://git.openjdk.org/jdk/compare/0bead8ff...5168c5b8
>
> src/hotspot/cpu/ppc/sharedRuntime_ppc.cpp line 256:
>
>> 254: RegisterSaver_LiveVSReg( VSR59 ),
>> 255: RegisterSaver_LiveVSReg( VSR60 ),
>> 256: RegisterSaver_LiveVSReg( VSR61 )
>
> What about VSR62 and VSR63?
Fixed. Thanks!
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23987#discussion_r2054338483
More information about the hotspot-dev
mailing list