[PATCH] D140532: .clang-tidy: disable misc-use-anonymous-namespace check.

Haojian Wu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 2 04:08:54 PST 2023


hokein added a comment.

In D140532#4012879 <https://reviews.llvm.org/D140532#4012879>, @sammccall wrote:

> Yeah, good call. Do you mind if I first use this as a trigger to start a thread about this style rule on discourse, though? I've been meaning to, it came up in a review recently.

sure, please go ahead. (I found that these warnings are annoying when editing the llvm code file).

> I think there's a fair chance of changing the policy from "require static" to "allow either" or "require anon namespace", and having the issue "live" in clang-tidy might make for a more engaged discussion.

Yeah, the code style says `require static`, I think in practice we allow either.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140532



More information about the llvm-commits mailing list