[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