[vectorIntrinsics+mask] Integrated: 8273367: [Vector API] Fix a regression of rearrange intrinsic after JDK-8272479

[Jie Fu]

On Mon, 6 Sep 2021 02:43:48 GMT, Jie Fu <jiefu at openjdk.org> wrote:

> Hi all, 
> 
> After JDK-8272479, the following crash was observed on x86 with `-ea -esa -Xcomp -XX:CompileThreshold=100`
> 
> 
> test Double256VectorTests.RearrangeDouble256VectorTests(double[-i * 5], shuffle[random]): success
> test Double256VectorTests.RearrangeDouble256VectorTests(double[i * 5], shuffle[random]): success
> test Double256VectorTests.RearrangeDouble256VectorTests(double[i + 1], shuffle[random]): success
> test Double256VectorTests.RearrangeDouble256VectorTests(double[cornerCaseValue(i)], shuffle[random]): success
> o114  ReplicateD  === _ o73  [[o340 ]]  #vectory[4]:{double}
> 
> --N: o114  ReplicateD  === _ o73  [[o340 ]]  #vectory[4]:{double}
> 
>    --N: o73  ConI  === o0  [[o166 o173 o114 ]]  #int:0
>    IMMI  10  IMMI
>    IMMI_0  0  IMMI_0
>    IMMI2  0  IMMI2
>    IMMU7  5  IMMU7
>    IMMI8  5  IMMI8
>    IMMU8  5  IMMU8
>    IMMI16  10  IMMI16
>    IMMU31  0  IMMU31
>    RREGI  50  loadConI0
>    RAX_REGI  50  loadConI0
>    RBX_REGI  50  loadConI0
>    RCX_REGI  50  loadConI0
>    RDX_REGI  50  loadConI0
>    RDI_REGI  50  loadConI0
>    NO_RAX_RDX_REGI  50  loadConI0
>    STACKSLOTI  150  storeSSI
> 
> 
> This is because the type of broadcasted ZERO is incorrect.
> 
> Other affected tests include
> 
> jdk/incubator/vector/DoubleMaxVectorTests.java
> jdk/incubator/vector/Float128VectorTests.java
> jdk/incubator/vector/Float256VectorTests.java
> jdk/incubator/vector/FloatMaxVectorTests.java
> jdk/incubator/vector/Long256VectorTests.java
> jdk/incubator/vector/LongMaxVectorTests.java
> 
> 
> Let's fix it.
> 
> Thanks.
> Best regards,
> Jie

This pull request has now been integrated.

Changeset: d65687c0
Author:    Jie Fu <jiefu at openjdk.org>
Committer: Ningsheng Jian <njian at openjdk.org>
URL:       https://git.openjdk.java.net/panama-vector/commit/d65687c0fb62f4b64897b928fdf22dc97e985bc9
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8273367: [Vector API] Fix a regression of rearrange intrinsic after JDK-8272479

Reviewed-by: njian

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

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