diff options
| author | Richard Biener <rguenther@suse.de> | 2021-08-31 11:04:51 +0200 |
|---|---|---|
| committer | Richard Biener <rguenther@suse.de> | 2021-08-31 12:27:04 +0200 |
| commit | 67927342290c61d7e054430f1d7a7281f1f97fae (patch) | |
| tree | e5ed7ed50d168232fef26a3cb999b77eabdf4420 /gcc | |
| parent | de7a795c321e76826d123c92b99e73e144666b60 (diff) | |
| download | gcc-67927342290c61d7e054430f1d7a7281f1f97fae.zip gcc-67927342290c61d7e054430f1d7a7281f1f97fae.tar.gz gcc-67927342290c61d7e054430f1d7a7281f1f97fae.tar.bz2 | |
tree-optimization/102142 - fix typo in loop BB reduc cost adjustment
This fixes a typo in the condition guarding the cleanup of the
visited flag of costed scalar stmts.
2021-08-31 Richard Biener <rguenther@suse.de>
PR tree-optimization/102142
* tree-vect-slp.c (vect_bb_vectorization_profitable_p): Fix
condition under which to unset the visited flag.
* g++.dg/torture/pr102142.C: New testcase.
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/testsuite/g++.dg/torture/pr102142.C | 9 | ||||
| -rw-r--r-- | gcc/tree-vect-slp.c | 2 |
2 files changed, 10 insertions, 1 deletions
diff --git a/gcc/testsuite/g++.dg/torture/pr102142.C b/gcc/testsuite/g++.dg/torture/pr102142.C new file mode 100644 index 0000000..8e3ea5d --- /dev/null +++ b/gcc/testsuite/g++.dg/torture/pr102142.C @@ -0,0 +1,9 @@ +/* { dg-do compile } */ + +extern short arr_597[]; +extern bool arr_601[]; +int test_var_13; +void test(short arr_391[][9][2][2]) { + for (int i_60 = 0; i_60 < 11; i_60 += test_var_13) + arr_597[22] = arr_601[i_60] = arr_391[0][0][1][4]; +} diff --git a/gcc/tree-vect-slp.c b/gcc/tree-vect-slp.c index 4ca2440..fa3566f 100644 --- a/gcc/tree-vect-slp.c +++ b/gcc/tree-vect-slp.c @@ -5396,7 +5396,7 @@ vect_bb_vectorization_profitable_p (bb_vec_info bb_vinfo, /* Unset visited flag. This is delayed when the subgraph is profitable and we process the loop for remaining unvectorized if-converted code. */ - if (orig_loop && !profitable) + if (!orig_loop || !profitable) FOR_EACH_VEC_ELT (scalar_costs, i, cost) gimple_set_visited (cost->stmt_info->stmt, false); |