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

Adam Nemet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 21 08:47:02 PDT 2017


anemet added a comment.

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.

In https://reviews.llvm.org/D36054#847133, @lenary wrote:

> Ok, I have patches for the two other repositories which had failing tests.
>
> - https://reviews.llvm.org/D36949
> - https://reviews.llvm.org/D36950
>
>   I have no idea how to make these all land at the same time, but roughly the same time is ok too, I think.


Yep, just be prepared to see bot failures if an intermittent version is picked up by a bot.


Repository:
  rL LLVM

https://reviews.llvm.org/D36054





More information about the llvm-commits mailing list