[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