[PATCH] D98439: [NFC][Inliner] Debugging support to print funtion size after each inlining.

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 11 11:43:14 PST 2021


wenlei added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Inliner.cpp:842
 
+      LLVM_DEBUG(dbgs() << "    Function size: " << F.getInstructionCount()
+                        << "\n");
----------------
We have debug prints for negative decision, but not positive decision, so we may end up with a bunch of "function size: ..". Would it be more helpful to say "Function size: X after inlining Y" or something like that?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98439/new/

https://reviews.llvm.org/D98439



More information about the llvm-commits mailing list