RFR: 8327652: S390x: Implements SLP support [v2]
Sidraya Jayagond
sjayagond at openjdk.org
Mon Mar 18 06:33:27 UTC 2024
On Fri, 15 Mar 2024 13:43:53 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:
>> Sidraya Jayagond has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Address review comments
>
> src/hotspot/cpu/s390/s390.ad line 754:
>
>> 752: Z_VR31, Z_VR31_H, Z_VR31_J, Z_VR31_K,
>> 753: );
>> 754:
>
> Why do you use only VR16 to VR31? If there is a reason, it should be explained in a comment.
> PPC64 has these explanations:
> // 1st 32 VSRs are aliases for the FPRs which are already defined above.
> // 2nd 32 VSRs are aliases for the VRs which are only defined here.
> // VSR52-VSR63 // nv!
> (s390 has no nv VRs which makes them easier to use.)
In s390, 1st 16 VRs are aliases for the FPRs same as PPC. I will add proper explanation for this.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18162#discussion_r1527915833
More information about the hotspot-dev
mailing list