From 3ab4c4ba3045821b8aa5615a7672ca30f017dd10 Mon Sep 17 00:00:00 2001 From: Cameron Zwarich Date: Thu, 21 Feb 2013 08:51:58 +0000 Subject: [PATCH] Don't rely on the isDead() MachineOperand flag when updating LiveIntervals. llvm-svn: 175732 --- llvm/lib/CodeGen/PHIElimination.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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. -- 2.7.4