[llvm] [GlobalISel] Add SVE support for alloca (PR #178976)

Jameson Nash via llvm-commits llvm-commits at lists.llvm.org
Sat Jan 31 12:49:32 PST 2026


================
@@ -30624,18 +30624,12 @@ bool AArch64TargetLowering::fallBackToDAGISel(const Instruction &Inst) const {
   // Note that if EnableSVEGISel is true, we allow scalable vector types for
   // all instructions, regardless of whether they are actually supported.
   if (!EnableSVEGISel) {
-    if (Inst.getType()->isScalableTy()) {
+    if (Inst.getType()->isScalableTy())
       return true;
-    }
 
     for (unsigned i = 0; i < Inst.getNumOperands(); ++i)
       if (Inst.getOperand(i)->getType()->isScalableTy())
         return true;
-
-    if (const AllocaInst *AI = dyn_cast<AllocaInst>(&Inst)) {
----------------
vtjnash wrote:

Okay, but what technical reason is there for having this list of broken things contain something that isn't broken?

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


More information about the llvm-commits mailing list