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

[Easwaran Raman via llvm-commits]

eraman added a comment.



> I'm just suggesting changing the *APIs* to deal in symbolic settings. Clearly the flag will be numeric. I'm suggested sinking the inline threshold flag into the inline cost analysis completely.


I attempted to do that and hit an issue. InlineSimple.cpp provides a createFunctionInliningPass((int Threshold) API. To sink thresholds to InlineCost, this needs to be removed, but this is called by LLVMPassManagerBuilderUseInlinerWithThreshold which is exposed by the llvm-c API


Repository:
  rL LLVM

http://reviews.llvm.org/D15401