[llvm-commits] CVS: llvm/lib/Transforms/Scalar/LoopUnroll.cpp

Chris Lattner lattner at cs.uiuc.edu
Thu Jul 20 12:06:28 PDT 2006



Changes in directory llvm/lib/Transforms/Scalar:

LoopUnroll.cpp updated: 1.22 -> 1.23
---
Log message:

Minor comment tweaks


---
Diffs of the changes:  (+5 -6)

 LoopUnroll.cpp |   11 +++++------
 1 files changed, 5 insertions(+), 6 deletions(-)


Index: llvm/lib/Transforms/Scalar/LoopUnroll.cpp
diff -u llvm/lib/Transforms/Scalar/LoopUnroll.cpp:1.22 llvm/lib/Transforms/Scalar/LoopUnroll.cpp:1.23
--- llvm/lib/Transforms/Scalar/LoopUnroll.cpp:1.22	Wed Jul 19 00:48:45 2006
+++ llvm/lib/Transforms/Scalar/LoopUnroll.cpp	Thu Jul 20 14:06:16 2006
@@ -266,18 +266,17 @@
   else
     delete LI->removeLoop(std::find(LI->begin(), LI->end(), L));
 
-
-  // FIXME: Should update dominator analyses
-
-  // Remove LCSSA Phis from the exit block
+  // Remove single-entry Phis from the exit block.
   for (BasicBlock::iterator ExitInstr = LoopExit->begin();
        PHINode* PN = dyn_cast<PHINode>(ExitInstr); ++ExitInstr) {
     assert(PN->getNumIncomingValues() == 1
-           && "Block should only have one pred, so Phi's must be LCSSA");
+           && "Block should only have one pred, so Phi's must be single entry");
     PN->replaceAllUsesWith(PN->getOperand(0));
     PN->eraseFromParent();
   }
-
+  
+  // FIXME: Should update dominator analyses
+  
   // Now that everything is up-to-date that will be, we fold the loop block into
   // the preheader and exit block, updating our analyses as we go.
   LoopExit->getInstList().splice(LoopExit->begin(), BB->getInstList(),






More information about the llvm-commits mailing list