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

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 7 16:25:34 PDT 2022


wenlei added a comment.

I think it's also worth making `TotalAllocaSizeRecursiveCaller` tunable. We need to be more conservative for stack size for recursive inlining, so a general size limit may not be enough there. We ran into trouble with the default 1024 callee stack limit for a deep recursion. Will probably send up a patch to add a llvm flag.


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

https://reviews.llvm.org/D127988



More information about the llvm-commits mailing list