[clang-tools-extra] r220865 - Shrink the set of checks enabled by default.

Alexander Kornienko alexfh at google.com
Wed Oct 29 11:25:09 PDT 2014


Author: alexfh
Date: Wed Oct 29 13:25:09 2014
New Revision: 220865

URL: http://llvm.org/viewvc/llvm-project?rev=220865&view=rev
Log:
Shrink the set of checks enabled by default.

Summary:
Enable only compiler diagnostics and safe static analyzer checks by
default. Let the defaults be conservative and safe for an average project.

Reviewers: djasper

Reviewed By: djasper

Subscribers: cfe-commits

Differential Revision: http://reviews.llvm.org/D6027

Modified:
    clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp

Modified: clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp?rev=220865&r1=220864&r2=220865&view=diff
==============================================================================
--- clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp (original)
+++ clang-tools-extra/trunk/clang-tidy/tool/ClangTidyMain.cpp Wed Oct 29 13:25:09 2014
@@ -35,11 +35,10 @@ static cl::extrahelp ClangTidyHelp(
     "  option, command-line option takes precedence. The effective\n"
     "  configuration can be inspected using -dump-config.\n\n");
 
-const char DefaultChecks[] =
-    "*,"                       // Enable all checks, except these:
-    "-clang-analyzer-alpha*,"  // Too many false positives.
-    "-llvm-include-order,"     // Not implemented yet.
-    "-google-*,";              // Doesn't apply to LLVM.
+const char DefaultChecks[] =  // Enable these checks:
+    "clang-diagnostic-*,"     //   * compiler diagnostics
+    "clang-analyzer-*,"       //   * Static Analyzer checks
+    "-clang-analyzer-alpha*"; //   * but not alpha checks: many false positives
 
 static cl::opt<std::string>
 Checks("checks", cl::desc("Comma-separated list of globs with optional '-'\n"





More information about the cfe-commits mailing list