Skip to content

Commit f581d8a

Browse files
authored
AMDGPU: Fix a comment (#169403)
This verifier check will complain if there aren't enough implicit operands -- so it doesn't *allow* those operands, it *requires* them.
1 parent 51d93e7 commit f581d8a

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

llvm/lib/Target/AMDGPU/SIInstrInfo.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5518,9 +5518,10 @@ bool SIInstrInfo::verifyInstruction(const MachineInstr &MI,
55185518
Desc.getNumOperands() + Desc.implicit_uses().size();
55195519
const unsigned NumImplicitOps = IsDst ? 2 : 1;
55205520

5521-
// Allow additional implicit operands. This allows a fixup done by the post
5522-
// RA scheduler where the main implicit operand is killed and implicit-defs
5523-
// are added for sub-registers that remain live after this instruction.
5521+
// Require additional implicit operands. This allows a fixup done by the
5522+
// post RA scheduler where the main implicit operand is killed and
5523+
// implicit-defs are added for sub-registers that remain live after this
5524+
// instruction.
55245525
if (MI.getNumOperands() < StaticNumOps + NumImplicitOps) {
55255526
ErrInfo = "missing implicit register operands";
55265527
return false;

0 commit comments

Comments
 (0)