[PATCH] D38171: [clang-tidy] Implement clang-tidy check aliases
Gábor Horváth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Nov 15 03:16:42 PST 2017
xazax.hun added a comment.
In https://reviews.llvm.org/D38171#909346, @leanil wrote:
> In https://reviews.llvm.org/D38171#901427, @xazax.hun wrote:
>
> > One problem to think about when we add all clang-diagnostic as "first or second" class citizen, `checkes=*` might now enable all the warnings which make no sense and might be surprising to the users. What do you think?
>
>
> This is a good point. Should I insert ",-clang-diagnostic*" after any (positive) * ?
@alexfh do you have some thoughts on this?
https://reviews.llvm.org/D38171
More information about the cfe-commits
mailing list