[llvm-commits] [llvm] r105541 - /llvm/trunk/lib/CodeGen/IfConversion.cpp

Jim Grosbach grosbach at apple.com
Mon Jun 7 12:12:21 PDT 2010


Author: grosbach
Date: Mon Jun  7 14:12:21 2010
New Revision: 105541

URL: http://llvm.org/viewvc/llvm-project?rev=105541&view=rev
Log:
Move exit check where it really belongs.

Modified:
    llvm/trunk/lib/CodeGen/IfConversion.cpp

Modified: llvm/trunk/lib/CodeGen/IfConversion.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/IfConversion.cpp?rev=105541&r1=105540&r2=105541&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/IfConversion.cpp (original)
+++ llvm/trunk/lib/CodeGen/IfConversion.cpp Mon Jun  7 14:12:21 2010
@@ -515,13 +515,13 @@
   MachineBasicBlock::iterator TI = TrueBBI.BB->begin();
   MachineBasicBlock::iterator FI = FalseBBI.BB->begin();
   while (1) {
+    if (TI == TrueBBI.BB->end() || FI == FalseBBI.BB->end())
+      break;
     // Skip dbg_value instructions
     if (TI->isDebugValue())
       ++TI;
     if (FI->isDebugValue())
       ++FI;
-    if (TI == TrueBBI.BB->end() || FI == FalseBBI.BB->end())
-      break;
 
     if (!TI->isIdenticalTo(FI))
       break;
@@ -533,13 +533,13 @@
   TI = firstNonBranchInst(TrueBBI.BB, TII);
   FI = firstNonBranchInst(FalseBBI.BB, TII);
   while (1) {
+    if (TI == TrueBBI.BB->begin() || FI == FalseBBI.BB->begin())
+      break;
     // Skip dbg_value instructions
     if (TI->isDebugValue())
       --TI;
     if (FI->isDebugValue())
       --FI;
-    if (TI == TrueBBI.BB->begin() || FI == FalseBBI.BB->begin())
-      break;
 
     if (!TI->isIdenticalTo(FI))
       break;





More information about the llvm-commits mailing list