From: Cameron Zwarich Date: Thu, 21 Feb 2013 08:51:58 +0000 (+0000) Subject: Don't rely on the isDead() MachineOperand flag when updating LiveIntervals. X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3ab4c4ba3045821b8aa5615a7672ca30f017dd10;p=platform%2Fupstream%2Fllvm.git Don't rely on the isDead() MachineOperand flag when updating LiveIntervals. llvm-svn: 175732 --- diff --git a/llvm/lib/CodeGen/PHIElimination.cpp b/llvm/lib/CodeGen/PHIElimination.cpp index b84d9d33..5584708 100644 --- a/llvm/lib/CodeGen/PHIElimination.cpp +++ b/llvm/lib/CodeGen/PHIElimination.cpp @@ -320,7 +320,9 @@ void PHIElimination::LowerPHINode(MachineBasicBlock &MBB, } LiveInterval &DestLI = LIS->getInterval(DestReg); - if (NewInstr->getOperand(0).isDead()) { + assert(DestLI.begin() != DestLI.end() && + "PHIs should have nonempty LiveIntervals."); + if (DestLI.endIndex().isDead()) { // A dead PHI's live range begins and ends at the start of the MBB, but // the lowered copy, which will still be dead, needs to begin and end at // the copy instruction.