[llvm] InlineSpiller: Delete assert that implicit_def has no implicit operands (PR #69087)

via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 18 10:53:02 PDT 2023


================
@@ -1071,8 +1071,7 @@ void InlineSpiller::insertReload(Register NewVReg,
 static bool isRealSpill(const MachineInstr &Def) {
   if (!Def.isImplicitDef())
     return true;
-  assert(Def.getNumOperands() == 1 &&
-         "Implicit def with more than one definition");
+
----------------
qcolombet wrote:

If we really want to keep an assert here, we could check that we have only one explicit operand.
Don't think that's worth it though.

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


More information about the llvm-commits mailing list