From 8405888af1fb2cb082d1189b3b0b5bb0562c9190 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 llvm-svn: 189179 --- llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp b/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp index b1f097e..2610e24 100644 --- a/llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp +++ b/llvm/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.7.4