[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