[PATCH] D115497: [Inline] Disable deferred inlining

Tue Feb 8 17:34:34 PST 2022

aeubanks added a comment.

In D115497#3306538 <https://reviews.llvm.org/D115497#3306538>, @xbolva00 wrote:

> In D115497#3306525 <https://reviews.llvm.org/D115497#3306525>, @jonpa wrote:
>
>> In D115497#3306497 <https://reviews.llvm.org/D115497#3306497>, @xbolva00 wrote:
>>
>>> If it was enabled in old pm, but not in new pm, then just oversight? Patch welcome.

partial inlining looks like it's still in the new PM, search `RunPartialInlining` in the LLVM sources

>> I was assuming the inliner (and outliner?) heuristics may have undergone a reworking with the new pass-manager, but I don't really know...
>
> cc @aeubanks

I don't think the general inlining heuristics have been changed, although the new PM has access to some analyses that the old PM doesn't when inlining that may cause inlining decisions to be different, namely `BlockFrequencyInfo`. The inlining order may also be different.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115497