================ @@ -150,8 +150,7 @@ generatedTypePrinter(Type def, AsmPrinter &printer); bool LLVMArrayType::isValidElementType(Type type) { return !llvm::isa<LLVMVoidType, LLVMLabelType, LLVMMetadataType, - LLVMFunctionType, LLVMTokenType, LLVMScalableVectorType>( - type); + LLVMFunctionType, LLVMTokenType>(type); ---------------- banach-space wrote:
That `LLVMScalableVectorType` was added long before SME: * https://reviews.llvm.org/D85663 But yes, "arrays of scalable vectors" are a thing and we rely on them. That said, I don't see any SME/SVE tests failing (I also check e2e locally), so this should be fine. https://github.com/llvm/llvm-project/pull/133286 _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits