Remove some more set operations
authorChris Lattner <sabre@nondot.org>
Sun, 12 Dec 2004 22:22:18 +0000 (22:22 +0000)
committerChris Lattner <sabre@nondot.org>
Sun, 12 Dec 2004 22:22:18 +0000 (22:22 +0000)
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@18840 91177308-0d34-0410-b5e6-96231b3b80d8

lib/Transforms/Scalar/ADCE.cpp

index c7d349932c9e8c14d863d1ade3be66353e751917..fde1308aba260f4ae7748fca72b4e70ec13dfb06 100644 (file)
@@ -283,10 +283,8 @@ bool ADCE::doADCE() {
 
     BasicBlock *BB = I->getParent();
     if (!ReachableBBs.count(BB)) continue;
-    if (!AliveBlocks.count(BB)) {     // Basic block not alive yet...
-      AliveBlocks.insert(BB);         // Block is now ALIVE!
+    if (AliveBlocks.insert(BB).second)     // Basic block not alive yet.
       markBlockAlive(BB);             // Make it so now!
-    }
 
     // PHI nodes are a special case, because the incoming values are actually
     // defined in the predecessor nodes of this block, meaning that the PHI
@@ -294,10 +292,8 @@ bool ADCE::doADCE() {
     //
     if (PHINode *PN = dyn_cast<PHINode>(I))
       for (pred_iterator PI = pred_begin(BB), PE = pred_end(BB); PI != PE; ++PI)
-        if (!AliveBlocks.count(*PI)) {
-          AliveBlocks.insert(BB);         // Block is now ALIVE!
+        if (AliveBlocks.insert(*PI).second) // Block is now ALIVE!
           markBlockAlive(*PI);
-        }
 
     // Loop over all of the operands of the live instruction, making sure that
     // they are known to be alive as well...