[PATCH] D130755: [LoopVectorize] Introduce trip count minimal value threshold to ignore tail-folding for scalable vectors

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 09:15:07 PDT 2022


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

A minor pre-push change but otherwise looks good.  Once landed can you please do the necessary to request an LLVM 15 backport and then we can turn our attention to potentially fixing up the cost model so this rather brute force fix can be removed.



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:10152
+      } else {
+        LLVM_DEBUG(dbgs() << "But the target considers the trip count too "
+                             "small to consider vectorizing.\n");
----------------
To make the output consistent please add a <space> before the 'But' because otherwise you'll end up printing `overheads are incurred.But the target considers...`


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

https://reviews.llvm.org/D130755



More information about the llvm-commits mailing list