[PATCH] D78895: [InstCombine][SVE] Fix visitInsertElementInst for scalable type.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 7 11:20:45 PDT 2020


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM with one minor nit



================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineSimplifyDemanded.cpp:1193
                                                 bool AllowMultipleUsers) {
-  unsigned VWidth = cast<VectorType>(V->getType())->getNumElements();
+  // Can not analyze scalable type. The number of vector elements is not a
+  // compile-time constant.
----------------
Cannot


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78895





More information about the llvm-commits mailing list