[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