[PATCH] D13791: In machine block placement, use 60% instead of 80% as the hot branch probability threshold when profile data is available.

Benjamin Kramer via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 16 02:10:32 PDT 2015


bkramer added a subscriber: bkramer.
bkramer added a comment.

We use 80% as the 'hot' probability in many places. Would it make sense to add a getHotProbability(bool HasProfileData) to BranchProb so we can remove the magic numbers here? No need to do that in this patch but maybe in a follow-up.

Then we can check if the other users of 80% can also make smarter decisions when they know the probabilities come from prof data.


http://reviews.llvm.org/D13791





More information about the llvm-commits mailing list