[PATCH] D23144: Add instruction_count MD to imported functions

Easwaran Raman via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 3 17:24:07 PDT 2016


eraman added a comment.

I can imaging you wanting other summary information (callsite count, for example) too. Why not create a broader summary metadata and put everything there?


================
Comment at: include/llvm/Transforms/IPO/FunctionImport.h:32
@@ -31,2 +31,3 @@
 public:
+  using ImportThresholdTy = unsigned;
   /// Set of functions to import from a source module. Each entry is a map
----------------
It seems unnecessary to me, but if you do this change you should use ImportThresholdTy everywhere Threshold is used.


https://reviews.llvm.org/D23144





More information about the llvm-commits mailing list