[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
Thu Aug 17 15:58:02 PDT 2017


eraman added a comment.

In https://reviews.llvm.org/D35850#843955, @haicheng wrote:

> Change the output of optimization remarks to something like below if anything like recursive call, dynamic alloca is found
>
> `foz is recursive and allocates too much stack space. Cost is not fully computed`
>  `foz not inlined into bar because it should never be inlined (cost=never)`
>
> Is this okay? Thank you.


Yes, that looks good. Even just a "Cost is not fully computed" is fine.


Repository:
  rL LLVM

https://reviews.llvm.org/D35850





More information about the llvm-commits mailing list