[llvm-commits] [llvm] r138968 - /llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp

Bill Wendling isanbard at gmail.com
Thu Sep 1 14:29:49 PDT 2011


Author: void
Date: Thu Sep  1 16:29:49 2011
New Revision: 138968

URL: http://llvm.org/viewvc/llvm-project?rev=138968&view=rev
Log:
Reduce indentation. No functionality change.

Modified:
    llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp

Modified: llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp?rev=138968&r1=138967&r2=138968&view=diff
==============================================================================
--- llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp (original)
+++ llvm/trunk/lib/Transforms/InstCombine/InstructionCombining.cpp Thu Sep  1 16:29:49 2011
@@ -1571,25 +1571,26 @@
     // Do a quick scan over the function.  If we find any blocks that are
     // unreachable, remove any instructions inside of them.  This prevents
     // the instcombine code from having to deal with some bad special cases.
-    for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB)
-      if (!Visited.count(BB)) {
-        // Delete the instructions backwards, as it has a reduced likelihood of
-        // having to update as many def-use and use-def chains.
-        for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ) {
-          Instruction *Inst = &*I++;
-          if (isa<TerminatorInst>(Inst))
-            break;
-          if (!Inst->use_empty())
-            Inst->replaceAllUsesWith(UndefValue::get(Inst->getType()));
-          if (isa<LandingPadInst>(Inst))
-            continue;
-          if (!isa<DbgInfoIntrinsic>(Inst)) {
-            ++NumDeadInst;
-            MadeIRChange = true;
-          }
-          Inst->eraseFromParent();
+    for (Function::iterator BB = F.begin(), E = F.end(); BB != E; ++BB) {
+      if (Visited.count(BB)) continue;
+
+      // Delete the instructions backwards, as it has a reduced likelihood of
+      // having to update as many def-use and use-def chains.
+      for (BasicBlock::iterator I = BB->begin(), E = BB->end(); I != E; ) {
+        Instruction *Inst = &*I++;
+        if (isa<TerminatorInst>(Inst))
+          break;
+        if (!Inst->use_empty())
+          Inst->replaceAllUsesWith(UndefValue::get(Inst->getType()));
+        if (isa<LandingPadInst>(Inst))
+          continue;
+        if (!isa<DbgInfoIntrinsic>(Inst)) {
+          ++NumDeadInst;
+          MadeIRChange = true;
         }
+        Inst->eraseFromParent();
       }
+    }
   }
 
   while (!Worklist.isEmpty()) {





More information about the llvm-commits mailing list