[PATCH] D77635: [LV] Vectorize with FoldTail when Primary Induction is absent

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

Thanks for the patch, Ayal!

I've put up D77577 <https://reviews.llvm.org/D77577> yesterday to allow mapping the primary IV used to a different IR value during codegen as alternative, but I think adding the recipe is more straight forward in the end, as there is a single place we need to use the primary IV.



================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:333
+  /// Hold the primary induction phi of the vector loop.
+  Value *PrimaryInduction = nullptr;
+
----------------
Maybe include Vector in the name, e.g. VectorInduction, to avoid confusion with Legal's PrimaryInduction


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.h:1166
+
+  /// Generate the wide load/store.
+  void execute(VPTransformState &State) override;
----------------
Comment needs updating.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77635