[PATCH] D23377: Add a new method to create SimpleInliner instance and make pre-inliner use this

[David Li via llvm-commits]

davidxl added inline comments.

================
Comment at: lib/Transforms/IPO/PassManagerBuilder.cpp:264
@@ +263,3 @@
+    IP.DefaultThreshold = PreInlineThreshold;
+    IP.HotCallSiteThreshold = PreInlineThreshold;
+    // FIXME: The hint threshold has the same value used by the regular inliner.
----------------
I think it is better to make these two as optional field in InlineParams.


https://reviews.llvm.org/D23377