[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