summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
diff options
context:
space:
mode:
authorMichael Kuperstein <mkuper@google.com>2017-01-11 19:23:57 +0000
committerMichael Kuperstein <mkuper@google.com>2017-01-11 19:23:57 +0000
commitf69e64662bda7498fd29418a54b4c5b8fb1e3476 (patch)
tree1b88a16a8d71efd5fa8a3ae32c1f6758c706714c /llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
parent3b5edf4fa7d971117c6623348227758767b6a325 (diff)
downloadbcm5719-llvm-f69e64662bda7498fd29418a54b4c5b8fb1e3476.tar.gz
bcm5719-llvm-f69e64662bda7498fd29418a54b4c5b8fb1e3476.zip
[SLP] Remove bogus assert.
The removed assert seems bogus - it's perfectly legal for the roots of the vectorized subtrees to be equal even if the original scalar values aren't, if the original scalars happen to be equivalent. This fixes PR31599. Differential Revision: https://reviews.llvm.org/D28539 llvm-svn: 291692
Diffstat (limited to 'llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp')
-rw-r--r--llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp4
1 files changed, 0 insertions, 4 deletions
diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
index bcaa8439cff..1c7cbc7edf9 100644
--- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
+++ b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
@@ -2493,10 +2493,6 @@ Value *BoUpSLP::vectorizeTree(TreeEntry *E) {
Value *LHS = vectorizeTree(LHSVL);
Value *RHS = vectorizeTree(RHSVL);
- if (LHS == RHS && isa<Instruction>(LHS)) {
- assert((VL0->getOperand(0) == VL0->getOperand(1)) && "Invalid order");
- }
-
if (Value *V = alreadyVectorized(E->Scalars))
return V;
OpenPOWER on IntegriCloud