[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