[PATCH] D87439: [SVE] In LoopIdiomRecognize::isLegalStore bail out for scalable vectors

[Eli Friedman via Phabricator via llvm-commits]

efriedma added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:475
+  // non-constant stride too, since the code below currently only works for
+  // constant strides.
+  TypeSize SizeInBits = DL->getTypeSizeInBits(StoredVal->getType());
----------------
Do you have any idea why we're checking whether the store overflows an unsigned?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87439