[PATCH] D76017: [SLPVectorizer][SVE] Bail out early for scalable vector.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 11 13:02:11 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:403
+  // Make sure Instructions are not already vectorized.
+  if (llvm::any_of(VL, [](Value *V) { return V->getType()->isVectorTy(); }))
+    return InstructionsState(VL[BaseIndex], nullptr, nullptr);
----------------
Should we be using `isValidElementType(V->getType())` here, instead of explicitly checking for vector types?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76017/new/

https://reviews.llvm.org/D76017





More information about the llvm-commits mailing list