[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