diff options
| author | Richard Biener <rguenther@suse.de> | 2013-03-13 12:15:06 +0000 |
|---|---|---|
| committer | Richard Biener <rguenth@gcc.gnu.org> | 2013-03-13 12:15:06 +0000 |
| commit | c40eced02bcfaf28cad7ee97120d7767029f1f6d (patch) | |
| tree | 0a65db66bff99bc90eccf076a4a59308f90f93f8 /gcc/tree-vect-slp.c | |
| parent | 15ff4345c4be1bb0de1766f58607fbe74325ec87 (diff) | |
| download | gcc-c40eced02bcfaf28cad7ee97120d7767029f1f6d.zip gcc-c40eced02bcfaf28cad7ee97120d7767029f1f6d.tar.gz gcc-c40eced02bcfaf28cad7ee97120d7767029f1f6d.tar.bz2 | |
re PR tree-optimization/56608 (SLP seems to produce incorrect value with -ffast-math)
2013-03-13 Richard Biener <rguenther@suse.de>
PR tree-optimization/56608
* tree-vect-slp.c (vect_schedule_slp): Do not remove scalar
calls when vectorizing basic-blocks.
* gcc.dg/vect/fast-math-bb-slp-call-3.c: New testcase.
From-SVN: r196632
Diffstat (limited to 'gcc/tree-vect-slp.c')
| -rw-r--r-- | gcc/tree-vect-slp.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/gcc/tree-vect-slp.c b/gcc/tree-vect-slp.c index 3b2fc80..e184326 100644 --- a/gcc/tree-vect-slp.c +++ b/gcc/tree-vect-slp.c @@ -3181,7 +3181,15 @@ vect_schedule_slp (loop_vec_info loop_vinfo, bb_vec_info bb_vinfo) unsigned int j; gimple_stmt_iterator gsi; - vect_remove_slp_scalar_calls (root); + /* Remove scalar call stmts. Do not do this for basic-block + vectorization as not all uses may be vectorized. + ??? Why should this be necessary? DCE should be able to + remove the stmts itself. + ??? For BB vectorization we can as well remove scalar + stmts starting from the SLP tree root if they have no + uses. */ + if (loop_vinfo) + vect_remove_slp_scalar_calls (root); for (j = 0; SLP_TREE_SCALAR_STMTS (root).iterate (j, &store) && j < SLP_INSTANCE_GROUP_SIZE (instance); j++) |