[PATCH] D120584: [NewPM][Inliner] Mark inlined calls to functions in same SCC as callee as noinline

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 6 10:29:02 PST 2022


xbolva00 added a comment.

In D120584#3362467 <https://reviews.llvm.org/D120584#3362467>, @davidxl wrote:

> Adding an option is a way to to forward, but I think it should be off by default.  If it is on by default, it is very likely to cause some performance regressions.

Or.. many bug reports about infinite compile times. Bad trade off.

The number one goal of every compiler should be correct and working compilation process.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120584/new/

https://reviews.llvm.org/D120584



More information about the llvm-commits mailing list