[PATCH] D52984: [analyzer] Checker reviewer's checklist
Umann Kristóf via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Oct 8 07:32:33 PDT 2018
Szelethus added a comment.
I love the idea. I got a couple more:
Is `-analyzer-checker=core` included in all `RUN:` lines?
Is the description in `Checkers.td` clear to a regular user?
Is `Checkers.td` and the CMakeLists file alphabetically sorted?
Repository:
rC Clang
https://reviews.llvm.org/D52984
More information about the cfe-commits
mailing list