[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