[PATCH] D22491: Added ThinLTO inlining statistics

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 17:06:04 PDT 2016


Prazek added inline comments.

================
Comment at: lib/Transforms/Utils/InlinerStats.cpp:55
@@ +54,3 @@
+    /// Incremented every direct inline.
+    int16_t NumberOfInlines = 0;
+    /// Number of inlines into non imported function (possibly indirect via
----------------
eraman wrote:
> I don't expect the number to exceed max value of int16_t in real programs, but perhaps possible in generated test cases. Why not make it int32_t ?
sure


https://reviews.llvm.org/D22491





More information about the llvm-commits mailing list