[vectorIntrinsics] Integrated: 8274569: X86 backend related incorrectness issues in legacy store mask patterns

[Jatin Bhateja]

On Thu, 30 Sep 2021 19:34:43 GMT, Jatin Bhateja <jbhateja at openjdk.org> wrote:

> - Issue was seen while unit testing changes for masking related optimizations on vectorIntrinsics branch.
> 
> - Following patterns result into incorrect store mask computation, even though contextually store-mask is followed by StoreVector which does takes care of various vector sizes. But C2 does emit VectorStoreMask in IR to populate a byte vector. 
> 
>   - instruct storeMask2B
>   - instruct storeMask4B
>   - instruct storeMask8B
>   - instruct storeMask8B_avx
> 
> - Replicate with immI operand is correctly taking care of shorter vector lengths thus it can be fall back case for following pattern with immediate -1 argument. 
>   - instruct ReplI_M1
>   
> Patch also adds a test case which exhaustively coves load/store vector masks operations for different SPECIES.

This pull request has now been integrated.

Changeset: a7c26670
Author:    Jatin Bhateja <jbhateja at openjdk.org>
URL:       https://git.openjdk.java.net/panama-vector/commit/a7c26670cec4823ee793ce837da41689a5540254
Stats:     311 lines in 4 files changed: 273 ins; 1 del; 37 mod

8274569: X86 backend related incorrectness issues in legacy store mask patterns

Co-authored-by: Jatin Bhateja <jbhateja at openjdk.org>
Co-authored-by: Jie Fu <jiefu at openjdk.org>
Reviewed-by: sviswanathan

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

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