diff options
author | Richard Sandiford <richard.sandiford@arm.com> | 2019-06-18 09:18:17 +0000 |
---|---|---|
committer | Richard Sandiford <rsandifo@gcc.gnu.org> | 2019-06-18 09:18:17 +0000 |
commit | fcae0292de06aeb54c44d26cfb80d798df60e339 (patch) | |
tree | f98a62e46bca2b433c2c6b950f710a6af3f1183c /gcc/ChangeLog | |
parent | a9e47ccf267fb088b004461c29e2daf9167bd102 (diff) | |
download | gcc-fcae0292de06aeb54c44d26cfb80d798df60e339.zip gcc-fcae0292de06aeb54c44d26cfb80d798df60e339.tar.gz gcc-fcae0292de06aeb54c44d26cfb80d798df60e339.tar.bz2 |
Restore correct iv step for fully-masked loops
r272233 introduced a large number of execution failures on SVE.
The patch hard-coded an IV step of VF, but for SLP groups it needs
to be VF * group size.
Also, iv_precision had type widest_int but only needs to be unsigned int.
2019-06-18 Richard Sandiford <richard.sandiford@arm.com>
gcc/
* tree-vect-loop-manip.c (vect_set_loop_masks_directly): Remove
vf parameter. Restore the previous iv step of nscalars_step,
but give it iv_type rather than compare_type. Tweak code order
to match the comments.
(vect_set_loop_condition_masked): Update accordingly.
* tree-vect-loop.c (vect_verify_full_masking): Use "unsigned int"
for iv_precision. Tweak comment formatting.
From-SVN: r272411
Diffstat (limited to 'gcc/ChangeLog')
-rw-r--r-- | gcc/ChangeLog | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 06cdba5..15bc6d7 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,13 @@ +2019-06-18 Richard Sandiford <richard.sandiford@arm.com> + + * tree-vect-loop-manip.c (vect_set_loop_masks_directly): Remove + vf parameter. Restore the previous iv step of nscalars_step, + but give it iv_type rather than compare_type. Tweak code order + to match the comments. + (vect_set_loop_condition_masked): Update accordingly. + * tree-vect-loop.c (vect_verify_full_masking): Use "unsigned int" + for iv_precision. Tweak comment formatting. + 2019-06-18 Iain Sandoe <iain@sandoe.co.uk> * config/darwin.c: Strip trailing whitespace. |