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

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 21 09:38:23 PDT 2017


lenary added a comment.

In https://reviews.llvm.org/D36054#847561, @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.
>
> ...
>
> Yep, just be prepared to see bot failures if an intermittent version is picked up by a bot.


The New PM pass remarks output fix landed on Saturday, and I updated these tests to use the NewPM too (see the diffs to `test/Trasforms/Inline/optimization*`).

Yeah, intermittent but expected failures I can cope with, unexpected ones are less fun.


Repository:
  rL LLVM

https://reviews.llvm.org/D36054





More information about the llvm-commits mailing list