[PATCH] D91567: [llvm][inliner] Reuse the inliner pass to implement 'always inliner'

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 18 09:50:16 PST 2020


mtrofin added a comment.

In D91567#2403216 <https://reviews.llvm.org/D91567#2403216>, @aeubanks wrote:

>>> I'll run this through llvm-compile-time-tracker to see what the compile time implications are.
>>
>> You mean for the variant where we ran some of the function passes, or you'd try running all of them? Probably the latter would be quite interesting as a 'worst case'.
>
> I was trying the previous patch, but will also try running all function passes, definitely would be interesting.

Awesome! Thanks!

If the rest of the (now NFC) patch seems reasonable, I'll go through those pesky pass manager tests to finish it up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91567



More information about the llvm-commits mailing list