Skip to content

Commit d25a4d2

Browse files
committed
[RISCV] Let LiveIntervals::shrinkToUses compute dead immediate. NFC
We can simplify removing dead AVL immediates > 31 by using the dead argument to shrinkToUses, since it will already compute dead values.
1 parent 600cae7 commit d25a4d2

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1648,17 +1648,9 @@ bool RISCVCoalesceVSETVLI::coalesceVSETVLIs(MachineBasicBlock &MBB) {
16481648
if (NextMI->getOperand(1).isReg())
16491649
NextMI->getOperand(1).setReg(RISCV::NoRegister);
16501650

1651-
if (OldVLReg && OldVLReg.isVirtual()) {
1652-
// NextMI no longer uses OldVLReg so shrink its LiveInterval.
1653-
LIS->shrinkToUses(&LIS->getInterval(OldVLReg));
1654-
1655-
MachineInstr *VLOpDef = MRI->getUniqueVRegDef(OldVLReg);
1656-
if (VLOpDef && TII->isAddImmediate(*VLOpDef, OldVLReg) &&
1657-
MRI->use_nodbg_empty(OldVLReg)) {
1658-
VLOpDef->eraseFromParent();
1659-
LIS->removeInterval(OldVLReg);
1660-
}
1661-
}
1651+
// NextMI no longer uses OldVLReg so shrink its LiveInterval.
1652+
if (OldVLReg && OldVLReg.isVirtual())
1653+
LIS->shrinkToUses(&LIS->getInterval(OldVLReg), &ToDelete);
16621654
MI.setDesc(NextMI->getDesc());
16631655
}
16641656
MI.getOperand(2).setImm(NextMI->getOperand(2).getImm());

0 commit comments

Comments
 (0)