[vectorIntrinsics+mask] RFR: 8265109: SVE predicate register allocation support for vectorIntrinsics

Jatin Bhateja jbhateja at openjdk.java.net
Thu Apr 15 07:11:43 UTC 2021


On Thu, 15 Apr 2021 04:18:36 GMT, Ningsheng Jian <njian at openjdk.org> wrote:

>> It doesn't have much usage apart from semantic checking for bound registers. 
>> But for correct ness a bounded opmask reg class for x86 has two registers.
>> Could not locate bounded reg class definitions for SVE predicate registers in aarch64.ad
>
> Yes, aarch64 does not need this for now. I can either revert this change (leaving it to you for future update if required) or update to size 2 for x86. Which do you prefer?

I can see ADLC does parse the preprocessors in source sections, by adding a new #define SVE_ENABLE  1 in aarch64_sve.ad  and checking for the same similar to following handling may allow us to set size = 1 for SVE else 2.
https://github.com/openjdk/jdk/blob/master/src/hotspot/share/adlc/formssel.cpp#L2279

-------------

PR: https://git.openjdk.java.net/panama-vector/pull/65


More information about the panama-dev mailing list