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

2018-10-08 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added inline comments. Comment at: www/analyzer/checker_dev_manual.html:708 +Checker Reviewer Checklist + I think we actually need two separate checklists: * for common bugs (careful use of non-fatal error nodes, etc. - stuff we check for on every review)

[PATCH] D52993: [analyzer][www] Add more useful links

2018-10-08 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Thx! Comment at: www/analyzer/checker_dev_manual.html:703-704 Useful Links +http://lcs.ios.ac.cn/~xuzb/canalyze/memmodel.pdf";>Xu, Zhongxing & Maybe merge wit

[PATCH] D52969: [analyzer][www] Update alpha_checks.html

2018-10-08 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Thanks! Comment at: www/analyzer/alpha_checks.html:450 - -Variable Argument Alpha Checkers - I think these are missing on the available checkers list, so we should

[PATCH] D52983: [analyzer] Support Reinitializes attribute in MisusedMovedObject check

2018-10-08 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Yay, these look useful. Is there also an attribute for methods that should never be called on a 'moved-from' object? Repository: rC Clang https://reviews.llvm.org/D52983 _

[clang-tools-extra] r344015 - [clang-tidy] The patch extends the existing command line option -check-suffix

2018-10-08 Thread Zinovy Nis via cfe-commits
Author: zinovy.nis Date: Mon Oct 8 22:40:03 2018 New Revision: 344015 URL: http://llvm.org/viewvc/llvm-project?rev=344015&view=rev Log: [clang-tidy] The patch extends the existing command line option -check-suffix (with alias -check-suffixes) to accept multiple comma-separated FileCheck prefixe

[PATCH] D52971: [clang-tidy] Customize FileCheck prefix in check_clang-tidy.py to support multiple prefixes

2018-10-08 Thread Zinovy Nis via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL344015: [clang-tidy] The patch extends the existing command line option -check-suffix (authored by zinovy.nis, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://

[clang-tools-extra] r344016 - [clang-tidy][docs] Update docs for `--check-suffixes`

2018-10-08 Thread Zinovy Nis via cfe-commits
Author: zinovy.nis Date: Mon Oct 8 22:48:57 2018 New Revision: 344016 URL: http://llvm.org/viewvc/llvm-project?rev=344016&view=rev Log: [clang-tidy][docs] Update docs for `--check-suffixes` Differential Revision: https://reviews.llvm.org/D52971 Modified: clang-tools-extra/trunk/docs/clang-t

<    1   2