[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