[llvm] [Inline] Accumulate the cost of the inlined function to the new call site (PR #111104)
Arthur Eubanks via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 10 10:55:18 PDT 2024
aeubanks wrote:
I believe part of the design of the new inliner was that it should be idempotent. As in rerunning the inliner should not result in any changes. If you accumulate inline costs between function calls, that breaks idempotency. I'd want more justification besides a vague "the inliner inlined too much stuff in one case" to break that.
https://github.com/llvm/llvm-project/pull/111104
More information about the llvm-commits
mailing list