[llvm] [AArch64][GlobalISel] Avoid generating inserts for undefs when selecting G_BUILD_VECTOR (PR #84452)

David Green via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 11 01:32:43 PDT 2024


================
@@ -5963,12 +5966,21 @@ bool AArch64InstructionSelector::selectBuildVector(MachineInstr &I,
     MachineOperand &RegOp = I.getOperand(1);
     RegOp.setReg(Reg);
     RBI.constrainGenericRegister(DstReg, *RC, MRI);
-  } else {
-    // We don't need a subregister copy. Save a copy by re-using the
-    // destination register on the final insert.
-    assert(PrevMI && "PrevMI was null?");
+  } else if (PrevMI) {
+    // PrevMI is not nullptr, so we have generated inserts and don't need a
+    // subregister copy. Save a copy by re-using the destination register on the
+    // final insert.
     PrevMI->getOperand(0).setReg(I.getOperand(0).getReg());
     constrainSelectedInstRegOperands(*PrevMI, TII, TRI, RBI);
+  } else {
----------------
davemgreen wrote:

I see. Which one? Setting PrevMI = ScalarToVec? It sound like the output of the variable could not be constrained as INSERT_SUBREG is a generic operation, would it work better if it was constrained to fpr128?

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


More information about the llvm-commits mailing list