[PATCH] D22491: Added ThinLTO inlining statistics

Mehdi AMINI via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 18 18:09:16 PDT 2016


mehdi_amini added inline comments.

================
Comment at: include/llvm/Transforms/IPO/InlinerStats.h:13
@@ +12,3 @@
+/// (2) Number of real inlined imported functions
+/// (3) Number of real not external inlined functions
+/// The difference between first and the second is that first stat counts
----------------
The definition for 3 isn't clear to me

================
Comment at: include/llvm/Transforms/IPO/InlinerStats.h:22
@@ +21,3 @@
+/// Then starting from non external functions that have some inlined calls
+/// inside, it walks to every inlined function and increment counter.
+///
----------------
It is not clear to me why you need a graph and can't just increment counters as the inlined moves on?


https://reviews.llvm.org/D22491





More information about the llvm-commits mailing list