[PATCH] D101076: [SVE][LoopVectorize] Add support for scalable vectorization of first-order recurrences

[David Sherwood via Phabricator via llvm-commits]

david-arm added inline comments.


================
Comment at: llvm/lib/IR/IRBuilder.cpp:1035
+
+  if (isa<ScalableVectorType>(Ty)) {
+    Module *M = BB->getParent()->getParent();
----------------
I guess if you remove `Ty` as @sdesmalen suggests it then makes sense to write here:

  if (auto VTy = dyn_cast<ScalableVectorType>(V1->getType()))

and replace the `Ty` below with `VTy`?


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

https://reviews.llvm.org/D101076