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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
