[PATCH] D52984: [analyzer] Checker reviewer's checklist

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 1 19:27:03 PDT 2018


NoQ added a comment.

Ok, so where do we put it? :)

I think we should put the everyday checklist into the checker developer manual under a title like "Making Your Checker Better" and put the out-of-alpha checklist somewhere into docs/analyzer as plain text (because that's not a sort of info you expect to find in the manual), though we could duplicate some points from there on the website (eg., "write visitors!").


Repository:
  rC Clang

https://reviews.llvm.org/D52984





More information about the cfe-commits mailing list