[llvm-branch-commits] [cfe-branch] r347829 - Merging r343105:

Tom Stellard via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed Nov 28 18:54:32 PST 2018

Author: tstellar
Date: Wed Nov 28 18:54:32 2018
New Revision: 347829

URL: http://llvm.org/viewvc/llvm-project?rev=347829&view=rev
Merging r343105:

r343105 | lebedevri | 2018-09-26 06:08:44 -0700 (Wed, 26 Sep 2018) | 17 lines

[analyzer] scan-build: if --status-bugs is passed, don't forget about the exit status of the actual build

This has been bothering me for a while, but only now i have actually looked into this.
I'm using one CI job for static analysis - clang static analyzers as compilers + clang-tidy via cmake.
And i'd like for the build to fail if at least one of those finds issues.
If clang-tidy finds issues, it will fail the build since the warnings-as-errors is set.
If static analyzer finds anything, since --status-bugs is set, it will fail the build.
But if clang-tidy find anything, but static analyzer does not, the build succeeds :/

Reviewers: sylvestre.ledru, alexfh, jroelofs, ygribov, george.karpenkov, krememek

Reviewed By: jroelofs

Subscribers: xazax.hun, szepet, a.sidorin, mikhail.ramalho, Szelethus, cfe-commits

Differential Revision: https://reviews.llvm.org/D52530


Modified: cfe/branches/release_70/tools/scan-build/bin/scan-build
URL: http://llvm.org/viewvc/llvm-project/cfe/branches/release_70/tools/scan-build/bin/scan-build?rev=347829&r1=347828&r2=347829&view=diff
--- cfe/branches/release_70/tools/scan-build/bin/scan-build (original)
+++ cfe/branches/release_70/tools/scan-build/bin/scan-build Wed Nov 28 18:54:32 2018
@@ -1192,7 +1192,7 @@ OPTIONS:
    By default, the exit status of scan-build is the same as the executed build
    command. Specifying this option causes the exit status of scan-build to be 1
-   if it found potential bugs and 0 otherwise.
+   if it found potential bugs and the exit status of the build itself otherwise.
  --use-cc [compiler path]
  --use-cc=[compiler path]
@@ -1878,7 +1878,7 @@ if (defined $Options{OutputFormat}) {
     if ($Options{ExitStatusFoundBugs}) {
       exit 1 if ($NumBugs > 0);
-      exit 0;
+      exit $ExitStatus;

More information about the llvm-branch-commits mailing list