[PATCH] D81761: [analyzer] Force dependency checkers to be hidden
Kristóf Umann via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jun 17 03:44:45 PDT 2020
Szelethus added a comment.
In D81761#2097561 <https://reviews.llvm.org/D81761#2097561>, @balazske wrote:
> My observation is, if there is an example checker, it should be really "example" and not "test". (The "custom" is probably good to rename to "test" but not the "example".) (The names of these files look not good too: A **CheckerOptionHandling.cpp** contains an example checker, should be probably **CheckerOptionHandlingExample.cpp**).
That is indeed a very fair point. My rationale was that these examples are in fact tested in the lit files, and my intention when writing them was to test checker registration from plugins.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D81761/new/
https://reviews.llvm.org/D81761
More information about the cfe-commits
mailing list