[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