[vectorIntrinsics] RFR: 8266775: Add VectorLoadConst node implementation for Arm SVE [v3]

[Wang Huang]

On Wed, 12 May 2021 05:59:40 GMT, Ningsheng Jian <njian at openjdk.org> wrote:

>> Wang Huang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   use sve_index
>
> src/hotspot/cpu/aarch64/aarch64_sve_ad.m4 line 2496:
> 
>> 2494: 
>> 2495: // ------------------------------ Vector Load Const -------------------------------
>> 2496: instruct loadconB(vReg dst, immI0 src)
> 
> Add a blank line between them?

Sure.

> src/hotspot/cpu/aarch64/aarch64_sve_ad.m4 line 2499:
> 
>> 2497: %{
>> 2498:   predicate(UseSVE > 0 &&
>> 2499:             n->bottom_type()->is_vect()->element_basic_type() == T_BYTE);
> 
> When the size is 16, the neon pattern will also be matched. Can you please add "UseSVE == 0" for NEON VectorLoadConst patterns?

Thank you for your review. I have changed that.

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

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