[PATCH] D32451: Improve profile-guided heuristics to use estimated trip count.

[Ayal Zaks via Phabricator via llvm-commits]

Ayal added inline comments.


================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:7728
+      (*EstimatedTC < TinyTripCountVectorThreshold) :
+      (LoopEntryFreq < ColdEntryFreq);
+    if (Hints.getForce() != LoopVectorizeHints::FK_Enabled && IsColdLoop)
----------------
When is the LoopEntryFreq < ColdEntryFreq criteria expected to kick in - when the loop latch has no associated frequencies (!EstimatedTC) but the function entry and loop preheader do? Looks like this criteria is effectively disabled, right?


https://reviews.llvm.org/D32451