[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