[PATCH] D22491: Added ThinLTO inlining statistics

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 27 15:24:31 PDT 2016


Prazek marked an inline comment as done.

================
Comment at: lib/Transforms/Utils/ImportedFunctionsInliningStatistics.cpp:31
@@ +30,3 @@
+    ValueLookup->Imported = F.getMetadata("thinlto_src_module") != nullptr;
+    ValueLookup->FunctionName = F.getName();
+  }
----------------
you were right. It's funny that I didn't thought about the solution that is right now. It is even better becase the functionname and Imported is set only once


https://reviews.llvm.org/D22491





More information about the llvm-commits mailing list