From ecc52e0c5b5a21f606e43febf2f65ca7daab1c97 Mon Sep 17 00:00:00 2001 From: Matt Arsenault Date: Sat, 24 Aug 2013 19:55:38 +0000 Subject: [PATCH] Check if in set on insertion instead of separately git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@189179 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/Transforms/Vectorize/SLPVectorizer.cpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/lib/Transforms/Vectorize/SLPVectorizer.cpp b/lib/Transforms/Vectorize/SLPVectorizer.cpp index b1f097e2d8a..2610e2457e7 100644 --- a/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/lib/Transforms/Vectorize/SLPVectorizer.cpp @@ -1161,13 +1161,11 @@ Value *BoUpSLP::vectorizeTree(TreeEntry *E) { ValueList Operands; BasicBlock *IBB = PH->getIncomingBlock(i); - if (VisitedBBs.count(IBB)) { + if (!VisitedBBs.insert(IBB)) { NewPhi->addIncoming(NewPhi->getIncomingValueForBlock(IBB), IBB); continue; } - VisitedBBs.insert(IBB); - // Prepare the operand vector. for (unsigned j = 0; j < E->Scalars.size(); ++j) Operands.push_back(cast(E->Scalars[j])-> @@ -1851,9 +1849,8 @@ bool SLPVectorizer::vectorizeChainsInBlock(BasicBlock *BB, BoUpSLP &R) { break; // We may go through BB multiple times so skip the one we have checked. - if (VisitedInstrs.count(instr)) + if (!VisitedInstrs.insert(instr)) continue; - VisitedInstrs.insert(instr); // Stop constructing the list when you reach a different type. if (Incoming.size() && P->getType() != Incoming[0]->getType()) { @@ -1879,9 +1876,8 @@ bool SLPVectorizer::vectorizeChainsInBlock(BasicBlock *BB, BoUpSLP &R) { for (BasicBlock::iterator it = BB->begin(), e = BB->end(); it != e; it++) { // We may go through BB multiple times so skip the one we have checked. - if (VisitedInstrs.count(it)) + if (!VisitedInstrs.insert(it)) continue; - VisitedInstrs.insert(it); if (isa(it)) continue; -- 2.11.0