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

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 11 14:43:55 PDT 2019


MyDeveloperDay added a comment.



>> 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.
> 
> Isn't that matching done on strings? I.e. is there difference between `-llvm-*` and `-ll*` ?

it would be if they only do it to -llm-* (which could possibly catch most cases)

but what if someone is doing

-llvm-header-guard

We wouldn't catch those..

This is a snippet of a .clang-tidy file from a  project I work on, we turn off specific checks by fully qualifying the checker

  ....
  llvm-*,
  -llvm-header-guard,
  -llvm-include-order,
  misc-*,
  modernize-*,
  ...


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