[PATCH] D81953: [clang-tidy] warnings-as-error no longer exits with ErrorCount

Jon Roelofs via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 16 14:02:20 PDT 2020


I didn’t have a specific use case in mind for it, so no preference either
way.

LGTM as well

On Tue, Jun 16, 2020 at 1:05 PM Aaron Ballman via Phabricator <
reviews at reviews.llvm.org> wrote:

> aaron.ballman accepted this revision.
> aaron.ballman added a comment.
> This revision is now accepted and ready to land.
>
> LGTM unless @jroelofs has a reason why the code was originally written
> that way, but can you add test coverage for it?
>
>
> Repository:
>   rG LLVM Github Monorepo
>
> CHANGES SINCE LAST ACTION
>   https://reviews.llvm.org/D81953/new/
>
> https://reviews.llvm.org/D81953
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200616/2f5fe72d/attachment.html>


More information about the cfe-commits mailing list