[PATCH] D150989: [Inliner] Mark inlinings stopped with inlining history as noinline

Jeremy Morse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 03:40:44 PDT 2023


jmorse accepted this revision.
jmorse added a comment.
This revision is now accepted and ready to land.

This speeds up compile time for the reproducer we have by one or two orders of magnitude, definitely feels like the right direction to take, LGTM.

How do you feel about this being in a LLVM-16 point release, exponential behaviour is a reasonably serious defect to be avoided IMO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D150989



More information about the llvm-commits mailing list