[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