[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