[PATCH] D15401: Refactor threshold computation for inline cost analysis

David Li via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 12 11:43:58 PST 2016


davidxl accepted this revision.
davidxl added a reviewer: davidxl.
davidxl added a comment.
This revision is now accepted and ready to land.

Sorry about the delay!

This patch is NFC cleanup with general direction agreed upon. The version LGTM (with the documentation nit). Please follow up post-commit if there are more issues to be handled.


================
Comment at: lib/Transforms/IPO/InlineSimple.cpp:41
@@ -40,2 +40,3 @@
 class SimpleInliner : public Inliner {
+  int DefaultThreshold;
 
----------------
Document this field. The DefaultThreshold can either be the default threshold specified by -inline-threshold, or the opt level dependent default threshold or a user specified value.


http://reviews.llvm.org/D15401





More information about the llvm-commits mailing list