diff options
| author | Ted Kremenek <kremenek@apple.com> | 2012-07-25 07:12:13 +0000 |
|---|---|---|
| committer | Ted Kremenek <kremenek@apple.com> | 2012-07-25 07:12:13 +0000 |
| commit | 2b619662eefba57c433b30964cb4c6a2e13eb34f (patch) | |
| tree | b82a4ab838ac05f1f3380345447fdf6cae850333 | |
| parent | 2589b9808e2a08ee51b2b0e03164303d2b74c6ed (diff) | |
| download | bcm5719-llvm-2b619662eefba57c433b30964cb4c6a2e13eb34f.tar.gz bcm5719-llvm-2b619662eefba57c433b30964cb4c6a2e13eb34f.zip | |
Promote warn_unknown_analyzer_checker to an error. Addresses <rdar://problem/10987863>.
llvm-svn: 160706
| -rw-r--r-- | clang/include/clang/Basic/DiagnosticFrontendKinds.td | 2 | ||||
| -rw-r--r-- | clang/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp | 2 | ||||
| -rw-r--r-- | clang/test/Misc/warning-flags.c | 6 |
3 files changed, 5 insertions, 5 deletions
diff --git a/clang/include/clang/Basic/DiagnosticFrontendKinds.td b/clang/include/clang/Basic/DiagnosticFrontendKinds.td index 5a67617ab1c..417a22c96df 100644 --- a/clang/include/clang/Basic/DiagnosticFrontendKinds.td +++ b/clang/include/clang/Basic/DiagnosticFrontendKinds.td @@ -107,7 +107,7 @@ def warn_unknown_warning_specifier : Warning< "unknown %0 warning specifier: '%1'">, InGroup<DiagGroup<"unknown-warning-option"> >; -def warn_unknown_analyzer_checker : Warning< +def err_unknown_analyzer_checker : Error< "no analyzer checkers are associated with '%0'">; def warn_incompatible_analyzer_plugin_api : Warning< "checker plugin '%0' is not compatible with this version of the analyzer">, diff --git a/clang/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp b/clang/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp index c06da0d9e4b..0229aed6bda 100644 --- a/clang/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp +++ b/clang/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp @@ -118,7 +118,7 @@ CheckerManager *ento::createCheckerManager(const AnalyzerOptions &opts, for (unsigned i = 0, e = checkerOpts.size(); i != e; ++i) { if (checkerOpts[i].isUnclaimed()) - diags.Report(diag::warn_unknown_analyzer_checker) + diags.Report(diag::err_unknown_analyzer_checker) << checkerOpts[i].getName(); } diff --git a/clang/test/Misc/warning-flags.c b/clang/test/Misc/warning-flags.c index f7c23df0936..a63937bb24d 100644 --- a/clang/test/Misc/warning-flags.c +++ b/clang/test/Misc/warning-flags.c @@ -1,4 +1,5 @@ -RUN: diagtool list-warnings | FileCheck %s +RUN: diagtool list-warnings > %t 2>&1 +RUN: FileCheck --input-file=%t %s This test serves two purposes: @@ -17,7 +18,7 @@ This test serves two purposes: The list of warnings below should NEVER grow. It should gradually shrink to 0. -CHECK: Warnings without flags (131): +CHECK: Warnings without flags (130): CHECK-NEXT: pp_include_next_absolute_path CHECK-NEXT: pp_include_next_in_primary CHECK-NEXT: pp_invalid_string_literal @@ -144,7 +145,6 @@ CHECK-NEXT: warn_undef_interface CHECK-NEXT: warn_undef_interface_suggest CHECK-NEXT: warn_undef_protocolref CHECK-NEXT: warn_undefined_internal -CHECK-NEXT: warn_unknown_analyzer_checker CHECK-NEXT: warn_unknown_method_family CHECK-NEXT: warn_use_out_of_scope_declaration CHECK-NEXT: warn_weak_identifier_undeclared |
