[PATCH] D81761: [analyzer] Force dependency checkers to be hidden
Kristóf Umann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 12 13:08:27 PDT 2020
Szelethus created this revision.
Szelethus added reviewers: NoQ, vsavchenko, dcoughlin, xazax.hun, balazske, martong, baloghadamsoftware.
Szelethus added a project: clang.
Herald added subscribers: cfe-commits, ASDenysPetrov, steakhal, Charusso, gamesh411, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, whisperity.
Szelethus edited the summary of this revision.
Since strong dependencies aren't user-facing (its hardly ever legal to disable them), lets enforce that they are hidden. Modeling checkers that aren't dependencies are of course not impacted, but there is only so much you can do against developers shooting themselves in the foot :^)
I also made some changes to the test files, reversing the "test" package for, well, testing.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D81761
Files:
clang/include/clang/StaticAnalyzer/Checkers/Checkers.td
clang/lib/Analysis/plugins/CheckerDependencyHandling/CheckerDependencyHandling.cpp
clang/lib/Analysis/plugins/CheckerOptionHandling/CheckerOptionHandling.cpp
clang/lib/Analysis/plugins/SampleAnalyzer/MainCallChecker.cpp
clang/lib/StaticAnalyzer/Frontend/CheckerRegistry.cpp
clang/test/Analysis/checker-plugins.c
clang/unittests/StaticAnalyzer/RegisterCustomCheckersTest.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81761.270491.patch
Type: text/x-patch
Size: 33357 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200612/38792371/attachment-0001.bin>
More information about the cfe-commits
mailing list