[PATCH] D22491: Added ThinLTO inlining statistics

Teresa Johnson via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 27 18:42:17 PDT 2016


tejohnson added a comment.

Ok, lgtm with two minor nits.


================
Comment at: include/llvm/Transforms/Utils/ImportedFunctionsInliningStatistics.h:72
@@ +71,3 @@
+
+  void setModuleInfo(const Module &M);
+
----------------
doxygen comment

================
Comment at: include/llvm/Transforms/Utils/ImportedFunctionsInliningStatistics.h:100
@@ +99,3 @@
+  /// InlinedCallees vector. If it would store InlineGraphNode instead then the
+  /// address of the node would not be i  nvariant.
+  NodesMapTy NodesMap;
----------------
extra space inadvertently added "i nvariant"?


https://reviews.llvm.org/D22491





More information about the llvm-commits mailing list