[PATCH] D70655: [AutoFDO] Top-down Inlining for specialization with context-sensitive profile
Wei Mi via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 27 09:22:46 PST 2019
wmi accepted this revision.
wmi added a comment.
This revision is now accepted and ready to land.
I did performance test for this change and the result is neutral.
================
Comment at: llvm/lib/Transforms/IPO/SampleProfile.cpp:1786
+ for (auto F : buildFunctionOrder(M, CG)) {
+ assert(!F->isDeclaration());
+ clearFunctionData();
----------------
Add some message for the assertion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70655/new/
https://reviews.llvm.org/D70655
More information about the llvm-commits
mailing list