[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