diff options
-rw-r--r-- | llvm/lib/CodeGen/LiveIntervalAnalysis.cpp | 4 | ||||
-rw-r--r-- | llvm/unittests/MI/LiveIntervalTest.cpp | 24 |
2 files changed, 27 insertions, 1 deletions
diff --git a/llvm/lib/CodeGen/LiveIntervalAnalysis.cpp b/llvm/lib/CodeGen/LiveIntervalAnalysis.cpp index 622b9c25d0e..1a8360b226b 100644 --- a/llvm/lib/CodeGen/LiveIntervalAnalysis.cpp +++ b/llvm/lib/CodeGen/LiveIntervalAnalysis.cpp @@ -1314,6 +1314,8 @@ private: if (TargetRegisterInfo::isVirtualRegister(Reg)) { SlotIndex LastUse = Before; for (MachineOperand &MO : MRI.use_nodbg_operands(Reg)) { + if (MO.isUndef()) + continue; unsigned SubReg = MO.getSubReg(); if (SubReg != 0 && LaneMask != 0 && (TRI.getSubRegIndexLaneMask(SubReg) & LaneMask) == 0) @@ -1353,7 +1355,7 @@ private: // Check if MII uses Reg. for (MIBundleOperands MO(*MII); MO.isValid(); ++MO) - if (MO->isReg() && + if (MO->isReg() && !MO->isUndef() && TargetRegisterInfo::isPhysicalRegister(MO->getReg()) && TRI.hasRegUnit(MO->getReg(), Reg)) return Idx.getRegSlot(); diff --git a/llvm/unittests/MI/LiveIntervalTest.cpp b/llvm/unittests/MI/LiveIntervalTest.cpp index e4567363d39..e0b3d5529af 100644 --- a/llvm/unittests/MI/LiveIntervalTest.cpp +++ b/llvm/unittests/MI/LiveIntervalTest.cpp @@ -329,6 +329,30 @@ TEST(LiveIntervalTest, MoveUpValNos) { }); } +TEST(LiveIntervalTest, MoveOverUndefUse0) { + // findLastUseBefore() used by handleMoveUp() must ignore undef operands. + liveIntervalTest( +" %0 = IMPLICIT_DEF\n" +" NOOP\n" +" NOOP implicit undef %0\n" +" %0 = IMPLICIT_DEF implicit %0(tied-def 0)\n", + [](MachineFunction &MF, LiveIntervals &LIS) { + testHandleMove(MF, LIS, 3, 1); + }); +} + +TEST(LiveIntervalTest, MoveOverUndefUse1) { + // findLastUseBefore() used by handleMoveUp() must ignore undef operands. + liveIntervalTest( +" %rax = IMPLICIT_DEF\n" +" NOOP\n" +" NOOP implicit undef %rax\n" +" %rax = IMPLICIT_DEF implicit %rax(tied-def 0)\n", + [](MachineFunction &MF, LiveIntervals &LIS) { + testHandleMove(MF, LIS, 3, 1); + }); +} + int main(int argc, char **argv) { ::testing::InitGoogleTest(&argc, argv); initLLVM(); |