[PATCH] D90056: [AutoFDO] Remove a broken assert in merging inlinee samples
    Wenlei He via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Oct 23 10:14:32 PDT 2020
    
    
  
wenlei accepted this revision.
wenlei added a comment.
This revision is now accepted and ready to land.
Thanks for tracking down the assertion. The context profile adjustment and merging for AutoFDO today is a best effort approximation. Given the case we ran into, which is hard to handle perfectly and also very rare, removing the assertion makes sense to me.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90056/new/
https://reviews.llvm.org/D90056
    
    
More information about the llvm-commits
mailing list