[PATCH] D35850: [InlineCost] Add cl::opt to allow full inline cost to be computed for debugging purposes.

Easwaran Raman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 9 11:31:01 PDT 2017


eraman added a comment.

analyzeBlock bails out when inlining is not feasible due to things like recursive call, dynamic alloca etc. Arguably it doesn't make sense to compute the full inline cost even with this new option (and that's the behavior with this patch). In that case, especially with the ORE hookup, there should be some indication that the cost is not fully computed because the callee can not get inlined even if cost < threshold.


Repository:
  rL LLVM

https://reviews.llvm.org/D35850





More information about the llvm-commits mailing list