[PATCH] D36054: Emit only A Single Opt Remark When Inlining

[Sam Elliott via Phabricator via llvm-commits]

lenary added a comment.

In https://reviews.llvm.org/D36054#846121, @anemet wrote:

> LGTM.  Are you going to commit this as is or fix the new PM -pass-remarks-output first and update this?  I am fine either way.


I think I'm going to pause on this patch for a moment until the other is accepted and committed.

I think I have that fix (as far as code is concerned), but I think that patch should include updating all tests that currently only use the old pass manager to emit remark yaml files to also make them do exactly the same with the new pass manager, and that will take a little longer. Expect the complete patch of both code and tests some time over the weekend, I think.


https://reviews.llvm.org/D36054