[llvm-commits] [llvm] r58651 - /llvm/trunk/lib/Transforms/Utils/BasicBlockUtils.cpp

Devang Patel dpatel at apple.com
Mon Nov 3 15:14:09 PST 2008


Author: dpatel
Date: Mon Nov  3 17:14:09 2008
New Revision: 58651

URL: http://llvm.org/viewvc/llvm-project?rev=58651&view=rev
Log:
Fix unused variable warnings.

Modified:
    llvm/trunk/lib/Transforms/Utils/BasicBlockUtils.cpp

Modified: llvm/trunk/lib/Transforms/Utils/BasicBlockUtils.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Utils/BasicBlockUtils.cpp?rev=58651&r1=58650&r2=58651&view=diff

==============================================================================
--- llvm/trunk/lib/Transforms/Utils/BasicBlockUtils.cpp (original)
+++ llvm/trunk/lib/Transforms/Utils/BasicBlockUtils.cpp Mon Nov  3 17:14:09 2008
@@ -205,7 +205,10 @@
 BasicBlock *llvm::SplitEdge(BasicBlock *BB, BasicBlock *Succ, Pass *P) {
   TerminatorInst *LatchTerm = BB->getTerminator();
   unsigned SuccNum = 0;
-  for (unsigned i = 0, e = LatchTerm->getNumSuccessors(); ; ++i) {
+#ifndef NDEBUG
+  unsigned e = LatchTerm->getNumSuccessors();
+#endif
+  for (unsigned i = 0; ; ++i) {
     assert(i != e && "Didn't find edge?");
     if (LatchTerm->getSuccessor(i) == Succ) {
       SuccNum = i;
@@ -224,6 +227,7 @@
     // If the successor only has a single pred, split the top of the successor
     // block.
     assert(SP == BB && "CFG broken");
+    SP = NULL;
     return SplitBlock(Succ, Succ->begin(), P);
   } else {
     // Otherwise, if BB has a single successor, split it at the bottom of the





More information about the llvm-commits mailing list