[analyzer] Prefer returns values to out-params in CheckerRegistry.cpp
authorKristof Umann <dkszelethus@gmail.com>
Sat, 15 Dec 2018 15:44:05 +0000 (15:44 +0000)
committerKristof Umann <dkszelethus@gmail.com>
Sat, 15 Dec 2018 15:44:05 +0000 (15:44 +0000)
commitf282d272154edd6a4d86d9d770ceea0822800062
treec8ffa94252ebfe004af40c0179cf59a4321181ae
parent8c8724dd0d12b06178797b825177583c33002d96
[analyzer] Prefer returns values to out-params in CheckerRegistry.cpp

Renaming collectCheckers to getEnabledCheckers
Changing the functionality to acquire all enabled checkers, rather then collect
checkers for a specific CheckerOptInfo (for example, collecting all checkers for
{ "core", true }, which meant enabling all checkers from the core package, which
was an unnecessary complication).
Removing CheckerOptInfo, instead of storing whether the option was claimed via a
field, we handle errors immediately, as getEnabledCheckers can now access a
DiagnosticsEngine. Realize that the remaining information it stored is directly
accessible through AnalyzerOptions.CheckerControlList.
Fix a test with -analyzer-disable-checker -verify accidentally left in.

llvm-svn: 349274
clang/include/clang/StaticAnalyzer/Core/CheckerRegistry.h
clang/include/clang/StaticAnalyzer/Frontend/FrontendActions.h
clang/lib/FrontendTool/ExecuteCompilerInvocation.cpp
clang/lib/StaticAnalyzer/Core/CheckerRegistry.cpp
clang/lib/StaticAnalyzer/Frontend/CheckerRegistration.cpp
clang/test/Analysis/disable-all-checks.c