diff options
| author | Michael Kuperstein <mkuper@google.com> | 2017-01-11 19:23:57 +0000 |
|---|---|---|
| committer | Michael Kuperstein <mkuper@google.com> | 2017-01-11 19:23:57 +0000 |
| commit | f69e64662bda7498fd29418a54b4c5b8fb1e3476 (patch) | |
| tree | 1b88a16a8d71efd5fa8a3ae32c1f6758c706714c /llvm/lib/Transforms/Vectorize | |
| parent | 3b5edf4fa7d971117c6623348227758767b6a325 (diff) | |
| download | bcm5719-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')
| -rw-r--r-- | llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | 4 |
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; |
