[PATCH] D143624: Inlining: Run the legacy AlwaysInliner before the regular inliner.
Arthur Eubanks via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 9 13:14:14 PST 2023
aeubanks added a comment.
In D143624#4116318 <https://reviews.llvm.org/D143624#4116318>, @aemerson wrote:
> Sure, the exponential compile time case is actually just a side benefit here. The motivating reason for this change is actually to improve code size when building codebases that make heavy use of always_inline.
Ah I didn't see that part of the commit message. Mentioning code size more explicitly in the message would be good.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143624/new/
https://reviews.llvm.org/D143624
More information about the cfe-commits
mailing list