[PATCH] D38171: Implement clang-tidy check aliases.

AndrĂ¡s Leitereg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 27 08:02:49 PDT 2017


leanil added a comment.

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) * ?


https://reviews.llvm.org/D38171





More information about the cfe-commits mailing list