From 41a628237ce7892e081f126826db6b2391b100aa Mon Sep 17 00:00:00 2001 From: Jim Grosbach Date: Mon, 7 Jun 2010 19:12:21 +0000 Subject: [PATCH] Move exit check where it really belongs. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@105541 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/IfConversion.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/lib/CodeGen/IfConversion.cpp b/lib/CodeGen/IfConversion.cpp index 2abcb51b4e2..2e630f71d60 100644 --- a/lib/CodeGen/IfConversion.cpp +++ b/lib/CodeGen/IfConversion.cpp @@ -515,13 +515,13 @@ bool IfConverter::ValidDiamond(BBInfo &TrueBBI, BBInfo &FalseBBI, 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 @@ bool IfConverter::ValidDiamond(BBInfo &TrueBBI, BBInfo &FalseBBI, 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;