[Bug tree-optimization/62075] [4.8/4.9/4.10 Regression] Vectorizer ICE on dolphin

2014-08-11 Thread rguenth at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62075

--- Comment #4 from Richard Biener  ---
Author: rguenth
Date: Mon Aug 11 14:48:24 2014
New Revision: 213815

URL: https://gcc.gnu.org/viewcvs?rev=213815&root=gcc&view=rev
Log:
2014-08-11  Richard Biener  

PR tree-optimization/62075
* tree-vect-slp.c (vect_detect_hybrid_slp_stmts): Properly
handle uses in patterns.

* gcc.dg/vect/pr62075.c: New testcase.

Added:
trunk/gcc/testsuite/gcc.dg/vect/pr62075.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-vect-slp.c

[Bug tree-optimization/62075] [4.8/4.9/4.10 Regression] Vectorizer ICE on dolphin

2014-08-11 Thread rguenth at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62075

Richard Biener  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |rguenth at gcc dot 
gnu.org

--- Comment #3 from Richard Biener  ---
The issue here is that we have a load group with one stmt feeding a condition. 
For some reason we think that we can support vectorizing

_7 = _4 != 0

with _4 being defined by a pure-SLP def.  Which looks odd - the this is clearly
not a pure SLP node!

Fix:

Index: tree-vect-slp.c
===
--- tree-vect-slp.c (revision 213809)
+++ tree-vect-slp.c (working copy)
@@ -1793,7 +1793,8 @@ vect_detect_hybrid_slp_stmts (slp_tree n
&& (stmt_vinfo = vinfo_for_stmt (use_stmt))
&& !STMT_SLP_TYPE (stmt_vinfo)
 && (STMT_VINFO_RELEVANT (stmt_vinfo)
-|| VECTORIZABLE_CYCLE_DEF (STMT_VINFO_DEF_TYPE (stmt_vinfo)))
+|| VECTORIZABLE_CYCLE_DEF (STMT_VINFO_DEF_TYPE (stmt_vinfo))
+   || STMT_VINFO_IN_PATTERN_P (stmt_vinfo))
&& !(gimple_code (use_stmt) == GIMPLE_PHI
  && STMT_VINFO_DEF_TYPE (stmt_vinfo)
   == vect_reduction_def))

[Bug tree-optimization/62075] [4.8/4.9/4.10 Regression] Vectorizer ICE on dolphin

2014-08-11 Thread rguenth at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62075

Richard Biener  changed:

   What|Removed |Added

 Status|RESOLVED|NEW
   Last reconfirmed||2014-08-11
 Resolution|DUPLICATE   |---
   Target Milestone|--- |4.8.4
 Ever confirmed|0   |1

--- Comment #2 from Richard Biener  ---
I don't think this is a dup.  Confirmed.

[Bug tree-optimization/62075] [4.8/4.9/4.10 Regression] Vectorizer ICE on dolphin

2014-08-09 Thread trippels at gcc dot gnu.org 
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62075

Markus Trippelsdorf  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||trippels at gcc dot gnu.org
 Resolution|--- |DUPLICATE

--- Comment #1 from Markus Trippelsdorf  ---
dup.

*** This bug has been marked as a duplicate of bug 61634 ***