[llvm] [MachineInstr] add insert method for variadic instructions (PR #67699)

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 27 15:48:52 PDT 2023


================
@@ -2475,3 +2475,24 @@ MachineInstr::getFirst5RegLLTs() const {
       Reg2, getRegInfo()->getType(Reg2), Reg3, getRegInfo()->getType(Reg3),
       Reg4, getRegInfo()->getType(Reg4));
 }
+
+void MachineInstr::insert(mop_iterator It, ArrayRef<MachineOperand> Ops) {
+  assert(isVariadic() && "can only modify variadic instructions");
+  assert(It->getParent() == this && "iterator points to operand of other inst");
----------------
MatzeB wrote:

Also scratch this whole comment. It seems addOperand() and this insert() here actually copies the operands by value so no need for such a check at all.

https://github.com/llvm/llvm-project/pull/67699


More information about the llvm-commits mailing list