[PATCH] D137285: [LV] Account for minimum vscale when rejecting scalable vectorization of short loops

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 08:58:11 PST 2022


david-arm accepted this revision.
david-arm added a comment.
This revision is now accepted and ready to land.

LGTM!



================
Comment at: llvm/test/Transforms/LoopVectorize/RISCV/short-trip-count.ll:58
+; Note: This test uses a vscale_range starting at 1, which is technically incompatibile
+; with +v.  If we expose a target hook for minimum vlen, this example will need reworked
+define void @small_trip_count_min_vlen_32(i32* nocapture %a) nounwind vscale_range(1,1024) {
----------------
Ah ok I hadn't realised this is incompatible. I mainly suggested min=1 as a way to still show the tail-folding behaviour for low trip counts, but I guess I could also have suggested raising the known trip count to a higher value. Anyway, thank you!


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

https://reviews.llvm.org/D137285



More information about the llvm-commits mailing list