[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