[PATCH] D60151: [clang-tidy] Rename llvm checkers to llvm-project

Don Hinton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 11 10:15:57 PDT 2019


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





More information about the cfe-commits mailing list