[PATCH] D33157: [Inliner] Do not mix callsite and callee hotness based updates.

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 10:49:50 PDT 2017


eraman added a comment.

Turns out that this change is incorrect.  When sample profile is used, we use callsite hotness even without BFI. In a separate patch, I plan to add a method in ProfileSummaryInfo to determine if the module has sample profile. Then I can fix this patch by using callsite hotness either when sample profile is used or when BFI is available.


https://reviews.llvm.org/D33157





More information about the llvm-commits mailing list