[vectorIntrinsics+mask] RFR: 8265109: SVE predicate register allocation support for vectorIntrinsics
Sandhya Viswanathan
sviswanathan at openjdk.java.net
Wed Apr 14 17:43:49 UTC 2021
On Wed, 14 Apr 2021 06:21:35 GMT, Ningsheng Jian <njian at openjdk.org> wrote:
>> src/hotspot/share/opto/chaitin.cpp line 1392:
>>
>>> 1390: }
>>> 1391: return OptoReg::Bad; // will cause chunk change, and retry next chunk
>>> 1392: } else if (lrg._is_predicate) {
>>
>> lrg._is_predicate is set to 1 for all architectures, so using scalable_reg_slots on this path doesn't look correct.
>
> This is inside lrg.is_scalable() condition of L1362, which handles scalable predicate register specifically. I think for non-scalable arch, the normal path of regmask handling should be fine.
I missed that. Thanks, this looks good.
-------------
PR: https://git.openjdk.java.net/panama-vector/pull/65
More information about the panama-dev
mailing list