[PATCH] D67948: [LV] Interleaving should not exceed estimated loop trip count.

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 4 14:21:35 PDT 2019


hsaito added a comment.

Vectorizer code change looks fine with me. I'd like to see the comments updated, though. Any more changes needed for the LIT tests?



================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5212
 
   // Do not interleave loops with a relatively small trip count.
+  auto BestKnownTC = getSmallBestKnownTC(*PSE.getSE(), TheLoop);
----------------
small estimated or constant trip count


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:5265-5266
 
   // If the trip count is constant, limit the interleave count to be less than
   // the trip count divided by VF.
+  if (BestKnownTC) {
----------------
If trip count is expected to be small, limit the interleave count to be less than the trip count divided by VF


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

https://reviews.llvm.org/D67948





More information about the llvm-commits mailing list