Skip to content

Commit ada6779

Browse files
committed
Check AVL has one def
1 parent fd1dbad commit ada6779

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

llvm/lib/Target/RISCV/RISCVInsertVSETVLI.cpp

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1553,15 +1553,12 @@ static bool canMutatePriorConfig(const MachineInstr &PrevMI,
15531553

15541554
auto &AVL = MI.getOperand(1);
15551555
auto &PrevAVL = PrevMI.getOperand(1);
1556-
assert(!AVL.isReg() || !AVL.getReg().isVirtual() ||
1557-
MRI.hasOneDef(AVL.getReg()));
1558-
assert(!PrevAVL.isReg() || !PrevAVL.getReg().isVirtual() ||
1559-
MRI.hasOneDef(PrevAVL.getReg()));
15601556

15611557
// If the AVL is a register, we need to make sure MI's AVL dominates PrevMI.
15621558
// For now just check that PrevMI uses the same virtual register.
15631559
if (AVL.isReg() && AVL.getReg() != RISCV::X0 &&
1564-
(!PrevAVL.isReg() || PrevAVL.getReg() != AVL.getReg()))
1560+
(!MRI.hasOneDef(AVL.getReg()) || !PrevAVL.isReg() ||
1561+
PrevAVL.getReg() != AVL.getReg()))
15651562
return false;
15661563
}
15671564

0 commit comments

Comments
 (0)