[PATCH] D22491: Added ThinLTO inlining statistics

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 22 13:54:02 PDT 2016


Prazek added inline comments.

================
Comment at: lib/Transforms/Utils/ThinLTOInlinerStats.cpp:203
@@ +202,3 @@
+  static ThinLTOInlinerStatistics Stats;
+  return Stats;
+}
----------------
mehdi_amini wrote:
> Global mutable state is seldom the right choice, I assume this is what you're gonna move to be state in the inliner?
Exactly


https://reviews.llvm.org/D22491





More information about the llvm-commits mailing list