[PATCH] D91567: [llvm][inliner] Reuse the inliner pass to implement 'always inliner'
David Blaikie via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 16 16:20:34 PST 2020
dblaikie added a comment.
> Performing the mandatory inlinings first simplifies the problem the full inliner needs to solve
That confuses me a bit - is that suggesting that we don't run the AlwaysInliner when we are running the Inliner (ie: we only run the AlwaysInliner at -O0, and use the Inliner at higher optimization levels and let the Inliner do always inlining too)?
& sounds like this is suggesting that would change? That we would now perform always inlining separately from inlining? Maybe that's an orthogonal/separate change from one implementing the always inlining using the Inliner being run in a separate mode?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91567/new/
https://reviews.llvm.org/D91567
More information about the cfe-commits
mailing list