[PATCH] D28729: [clang-tidy] Add -enable-alpha-checks command

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 14 06:06:13 PST 2017


aaron.ballman accepted this revision.
aaron.ballman added a reviewer: aaron.ballman.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM with a few minor nits, but in the future, please provide some summary of what your patch is going to do rather than leave it entirely blank. It makes it easier on the reviewers if they have an idea of what to expect and why it's needed. ;-)

Also, this should probably be added to the release notes.



================
Comment at: clang-tidy/ClangTidyOptions.h:78
 
+  /// \brief Turns on experimental alpha checkers from static-analyzer.
+  llvm::Optional<bool> EnableAlphaChecks;
----------------
from static-analyzer -> from the static analyzer


https://reviews.llvm.org/D28729





More information about the cfe-commits mailing list