[PATCH] D76655: [InstSimplify] Fix SimplifyGEPInst when GEP index type is vector type.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 16 13:56:52 PDT 2020


efriedma added inline comments.


================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:4143
+  // The following folds are only valid when GEP index types are scalar types.
+  // Bailout early if GEP index type is vector type.
+  if (!IsScalableVec && !isa<VectorType>(Ops.back()->getType()) &&
----------------
I think the transform is still valid, it just isn't implemented.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76655





More information about the llvm-commits mailing list