[PATCH] D22491: Added ThinLTO inlining statistics

[Teresa Johnson via llvm-commits]

tejohnson added inline comments.

================
Comment at: lib/Transforms/IPO/InlinerStats.cpp:51
@@ +50,3 @@
+      dbgs() << "Inlined "
+             << (Node.second.Imported ? "imported " : "not external ")
+             << "function [" << Node.first->getName() << "]"
----------------
s/not external/not imported/

Also, it would be useful to know here whether the caller that we are inlining into is imported or not.

================
Comment at: lib/Transforms/IPO/InlinerStats.cpp:61
@@ +60,3 @@
+         << NumberOfRealUniqueInlinedImportedFunctions
+         << "\nNumber of real not external inlined functions: "
+         << NumberOFRealUniqueInlinedNotExternalFunctions << "\n";
----------------
Should be trivial I think to compute the reverse as well: how many imported functions were not inlined into a non-imported function (possibly via other inlines).


https://reviews.llvm.org/D22491