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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to