hintonda added a comment.

In D60151#1462850 <https://reviews.llvm.org/D60151#1462850>, @MyDeveloperDay 
wrote:

> are we supporting "-llvm-*" in existing .clang-tidy files?
>
> If people selectively turn checkers off, won't all of a sudden everyone start 
> getting llvm-project checks and fixes turned back on?
>
> https://github.com/search?q=-llvm-%2A&type=Code
>
> maybe we need to add something like:
>
>   llvm-header-guard (redirects to llvm-project-header-guard) 
> <llvm-header-guard>
>   llvm-header-include-order (redirects to llvm-project-include-order) 
> <llvm-include-order>
>   llvm-header-namespace-comment(redirects to llvm-project-namespace-comment) 
> <llvm-namespace-comment>
>   llvm-header-twine-local(redirects to llvm-project-twine-local) 
> <llvm-twine-local>
>  
>


I suppose we could keep the names and directory structure and just change the 
namespace.  That would just be a special case in the scripts.  Haven't looked 
into it yet, but will do so as soon as I can.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60151/new/

https://reviews.llvm.org/D60151



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to