[PATCH] D23144: Add instruction_count MD to imported functions
Piotr Padlewski via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 3 17:05:15 PDT 2016
Prazek added inline comments.
================
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
----------------
This one is small change, that I don't know if you like or not, so I didn't want to push it without review.
================
Comment at: include/llvm/Transforms/Utils/ImportedFunctionsInliningStatistics.h:80
@@ -79,2 +79,3 @@
bool Visited = false;
+ unsigned NumberOfInstructions = 0;
};
----------------
This one is should not be part of patch. I will remove it.
https://reviews.llvm.org/D23144
More information about the llvm-commits
mailing list