[PATCH] D127988: [Inliner] Introduce a backend option to suppress inlining of functions with large stack sizes

Wolfgang Pieb via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 15:01:01 PDT 2022


wolfgangp added a comment.

In D127988#3593143 <https://reviews.llvm.org/D127988#3593143>, @tschuett wrote:

> If I have lot of functions where the stack size is 1 byte below the threshold and do a lot of inlining, then the aggregated stack size will explode?

Right, but this is already the behavior today. One could think of a more sophisticated scheme, where the cost of inlining a function gets larger the more the caller's stack size grows, for example.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127988/new/

https://reviews.llvm.org/D127988



More information about the llvm-commits mailing list