diff options
| author | Nadav Rotem <nrotem@apple.com> | 2013-07-15 22:52:48 +0000 |
|---|---|---|
| committer | Nadav Rotem <nrotem@apple.com> | 2013-07-15 22:52:48 +0000 |
| commit | 135e81efe3c1848a308c96dfd65e4d88b0d8667b (patch) | |
| tree | 1e3b24eb8025e5b15e050d40cf6b066df9373183 /lib/Transforms | |
| parent | a44c37f880c8ca84b7388dd52fb2708495697a18 (diff) | |
| download | external_llvm-135e81efe3c1848a308c96dfd65e4d88b0d8667b.zip external_llvm-135e81efe3c1848a308c96dfd65e4d88b0d8667b.tar.gz external_llvm-135e81efe3c1848a308c96dfd65e4d88b0d8667b.tar.bz2 | |
PR16628: Fix a bug in the code that merges compares.
Compares return i1 but they compare different types.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@186359 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Transforms')
| -rw-r--r-- | lib/Transforms/Vectorize/SLPVectorizer.cpp | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/Transforms/Vectorize/SLPVectorizer.cpp b/lib/Transforms/Vectorize/SLPVectorizer.cpp index 5449f39..3090aa8 100644 --- a/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -703,9 +703,11 @@ void BoUpSLP::buildTree_rec(ArrayRef<Value *> VL, unsigned Depth) { case Instruction::FCmp: { // Check that all of the compares have the same predicate. CmpInst::Predicate P0 = dyn_cast<CmpInst>(VL0)->getPredicate(); + Type *ComparedTy = cast<Instruction>(VL[0])->getOperand(0)->getType(); for (unsigned i = 1, e = VL.size(); i < e; ++i) { CmpInst *Cmp = cast<CmpInst>(VL[i]); - if (Cmp->getPredicate() != P0) { + if (Cmp->getPredicate() != P0 || + Cmp->getOperand(0)->getType() != ComparedTy) { newTreeEntry(VL, false); DEBUG(dbgs() << "SLP: Gathering cmp with different predicate.\n"); return; |