[PATCH] D149479: [LV] Enable scalable outer loop vectorization

Kolya Panchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 8 12:52:26 PDT 2023

nikolaypanchenko marked an inline comment as done.
nikolaypanchenko added a comment.

In D149479#4568588 <https://reviews.llvm.org/D149479#4568588>, @iamlouk wrote:

> @nikolaypanchenko, thank you for this patch! Sorry to bother you, but will you update the revision (to support scalable vectorization factors)? I have written a patch myself with the same intention (although it works slightly differently, I also modified `determineVPlanVF`), not knowing that this revision already exists, so if you do not have the resources for this at the moment, I could present an alternative one, but only if you are ok with that of course.

@iamlouk this patch addressed 3 different problems: outerloop vectorization for scalable vectors; vec-report for outerloop and stability fix for outerloop; As @fhahn  suggested, I split last 2 into their own review: D150696 <https://reviews.llvm.org/D150696> and D150700 <https://reviews.llvm.org/D150700>.
I assume your approach to enable outerloop vectorization for scalable vectors is more generic than mine adhoc approach here, so can you please post your patch instead ?

  rG LLVM Github Monorepo



More information about the llvm-commits mailing list