[PATCH] D76655: [InstSimplify][SVE] Minor fix SimplifyGEPInst for scalable vector.

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/Analysis/InstructionSimplify.cpp:4150
+    if (!OffsetTypeSize.isScalable() &&
+        OffsetTypeSize.getFixedSize() == IdxWidth) {
       APInt BasePtrOffset(IdxWidth, 0);
----------------
This is supposed to be comparing the size of the scalar types, I think?  Scalar types can't be scalable.

That said, it looks like the code isn't actually doing that at the moment.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76655