[PATCH] D32768: [PM] Add ProfileSummaryAnalysis as a required pass in the new pipeline.

[Davide Italiano via llvm-commits]

Makes sense. Is there a way to exercise this behaviour?

Thanks,

--
Davide

On Wed, May 3, 2017 at 11:52 AM, Easwaran Raman <eraman at google.com> wrote:
> Without this, the call to MAM.getCachedResult<ProfileSummaryAnalysis>(M) in
> the run method will return nullptr. The inliner doesn't fail because of
> this, but it effectively becomes a non-PGO inliner.
>
> On Wed, May 3, 2017 at 11:49 AM, Davide Italiano via Phabricator
> <reviews at reviews.llvm.org> wrote:
>>
>> davide added a comment.
>>
>> This is reasonable. Did you get failures because the profile info weren't
>> available in the inliner?
>>
>>
>> https://reviews.llvm.org/D32768
>>
>>
>>
>