[PATCH] D81953: [clang-tidy] warnings-as-error no longer exits with ErrorCount
Nathan James via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 16 13:45:00 PDT 2020
njames93 added a comment.
In D81953#2096388 <https://reviews.llvm.org/D81953#2096388>, @aaron.ballman wrote:
> LGTM unless @jroelofs has a reason why the code was originally written that way, but can you add test coverage for it?
How would you suggest I add test coverage for this, afaik llvm-lit doesn't seem to handle checking specific exit codes, only whether is was 0 or not.
There are already test cases for a non-zero exit code with `-warnings-as-errors`.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81953/new/
https://reviews.llvm.org/D81953
More information about the cfe-commits
mailing list