[PATCH] D42946: Verify profile data confirms large loop trip counts.

David Li via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 5 22:12:25 PST 2018


davidxl added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:8353
 
-  if (!HasExpectedTC && LoopVectorizeWithBlockFrequency) {
+  // ExpectedTC may be large because it's bound by a variable. Check
+  // profiling information to validate we should vectorize.
----------------
What is the ExpectedTC returned in this case?  Why does it not return CouldNotCompute  (or 0)?


Repository:
  rL LLVM

https://reviews.llvm.org/D42946





More information about the llvm-commits mailing list