[PATCH] D67805: [LV] Allow vectorization of hot short trip count loops with epilog

[Evgeniy via Phabricator via llvm-commits]

ebrevnov marked 2 inline comments as done.
ebrevnov added inline comments.


================
Comment at: llvm/include/llvm/Transforms/Utils/SizeOpts.h:25
 /// profile.
-bool shouldOptimizeForSize(Function *F, ProfileSummaryInfo *PSI,
-                           BlockFrequencyInfo *BFI);
+Optional<bool> shouldOptimizeForSize(Function *F, ProfileSummaryInfo *PSI,
+                                     BlockFrequencyInfo *BFI);
----------------
This change is motivated by use case in getScalarEpilogueLowering.


================
Comment at: llvm/lib/Transforms/Vectorize/LoopVectorize.cpp:942
 
-  // A special case of vectorisation with OptForSize: loops with a very small
+  // A special case of vectorization with OptForSize: loops with a very small
   // trip count are considered for vectorization under OptForSize, thereby
----------------
just to be consistent with the next line :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D67805