[PATCH] D39105: Patch bug 27628 (clang-tidy should exit with a failure code when there are errors)

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 20 16:07:10 PDT 2017


alexfh added inline comments.


================
Comment at: clang-tidy/tool/ClangTidyMain.cpp:452
 
+  if (FoundErrors) {
+      return 1;
----------------
Failed tests reminded me of the -fix-errors flag.

I suppose, clang-tidy should not return an error code, when all errors it noticed, contained fixes that were successfully applied by -fix-errors.


https://reviews.llvm.org/D39105





More information about the cfe-commits mailing list