[PATCH] D108265: .clang-tidy: Push variable related readability-identifier-naming options down to projects

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 18 09:48:21 PDT 2021


MaskRay added a comment.

The number of top-level projects using `VariableName` is smaller than the number of projects not using the style.
The top-level variable style just provoked projects to either override the options (flang/, lld/, mlir/) or disable the check.
`VariableName` is not even a suitable suggestion for new projects.

So the  `VariableName` setting does not belong to the top-level. llvm/ and clang-tools-extra/ should set it by themselves.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108265



More information about the cfe-commits mailing list