From c34befc78fc9f9a93ca6043814d9f8b53e1fccc3 Mon Sep 17 00:00:00 2001 From: Jim Grosbach Date: Fri, 25 Jun 2010 23:05:46 +0000 Subject: [PATCH] make sure to handle dbg_value instructions in the middle of the block, not just at the head, when doing diamond if-conversion. rdar://7797940 llvm-svn: 106907 --- llvm/lib/CodeGen/IfConversion.cpp | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/llvm/lib/CodeGen/IfConversion.cpp b/llvm/lib/CodeGen/IfConversion.cpp index ea4e68dbdaf6..51fea7f94e5e 100644 --- a/llvm/lib/CodeGen/IfConversion.cpp +++ b/llvm/lib/CodeGen/IfConversion.cpp @@ -1228,10 +1228,18 @@ bool IfConverter::IfConvertDiamond(BBInfo &BBI, IfcvtKind Kind, ++DI2; BBI1->NonPredSize -= NumDups1; BBI2->NonPredSize -= NumDups1; - while (NumDups1 != 0) { + + // Skip past the dups on each side separately since there may be + // differing dbg_value entries. + for (unsigned i = 0; i < NumDups1; ++i) { ++DI1; + if (!DI1->isDebugValue()) + ++i; + } + while (NumDups1 != 0) { ++DI2; - --NumDups1; + if (!DI2->isDebugValue()) + --NumDups1; } UpdatePredRedefs(BBI1->BB->begin(), DI1, Redefs, TRI);