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

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Nov 10 07:11:07 PST 2018


xazax.hun updated this revision to Diff 173513.
xazax.hun marked 11 inline comments as done.
xazax.hun added a comment.

- Move the checklist up before additional info in the HTML file.
- Fix minor nits.
- Add missing bullet points (thanks @Szelethus for noticing)

I did not add any coding convention related item yet. I wonder if it is a good idea to have our own coding guidelines even if it is derived from the LLVM one.


https://reviews.llvm.org/D52984

Files:
  www/analyzer/checker_dev_manual.html

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52984.173513.patch
Type: text/x-patch
Size: 8086 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181110/74fb8922/attachment-0001.bin>


More information about the cfe-commits mailing list