Re: [Vect] Fix mask check on Scatter loads/stores

2021-03-10 Thread Richard Sandiford via Gcc-patches
Joel Hutton writes: >>> gcc/testsuite/ChangeLog: >>> >>> PR target/99102 >>> * gcc.target/aarch64/sve/pr99102.c: New test. >> >>(The filename is out of date, but the git hook would catch that.) > Fixed and committed. > >> >>> >>> diff --git a/gcc/testsuite/gcc.dg/vect/pr99102.c >>> b/gcc/test

RE: [Vect] Fix mask check on Scatter loads/stores

2021-03-10 Thread Joel Hutton via Gcc-patches
>> gcc/testsuite/ChangeLog: >> >> PR target/99102 >> * gcc.target/aarch64/sve/pr99102.c: New test. > >(The filename is out of date, but the git hook would catch that.) Fixed and committed. > >> >> diff --git a/gcc/testsuite/gcc.dg/vect/pr99102.c >> b/gcc/testsuite/gcc.dg/vect/pr99102.c >> new

Re: [Vect] Fix mask check on Scatter loads/stores

2021-03-10 Thread Richard Sandiford via Gcc-patches
ch64. Regression tested on > aarch64_sve > under qemu. > > [Vect] Fix mask check on Scatter loads/stores > > Previously, IFN_MASK_SCATTER_STORE was used if 'loop_masks' was > non-null, but the mask used is 'final_mask'. This caused a bug where > a 'MASK_ST

Re: [Vect] Fix mask check on Scatter loads/stores

2021-03-10 Thread Richard Biener
n tested on aarch64. Regression tested on > aarch64_sve > under qemu. OK. Thanks, Richard. > > [Vect] Fix mask check on Scatter loads/stores > > Previously, IFN_MASK_SCATTER_STORE was used if 'loop_masks' was > non-null, but the mask used is 'final_mask&#x

[Vect] Fix mask check on Scatter loads/stores

2021-03-10 Thread Joel Hutton via Gcc-patches
Hi all, This patch fixes PR99102. For masked gather loads/scatter stores the 'loop_masks' variable was checked to be non-null, but the 'final_mask' was the actual mask used. bootstrapped and regression tested on aarch64. Regression tested on aarch64_sve under qemu. [Vec