[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