[clang-tools-extra] [libc] Update libc namespace clang-tidy checks (PR #98424)
Paul Kirth via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 11 08:18:26 PDT 2024
================
@@ -30,20 +30,35 @@ void ImplementationInNamespaceCheck::check(
const auto *MatchedDecl =
Result.Nodes.getNodeAs<Decl>("child_of_translation_unit");
const auto *NS = dyn_cast<NamespaceDecl>(MatchedDecl);
+
+ // LLVM libc declarations should be inside of a non-anonymous namespace.
if (NS == nullptr || NS->isAnonymousNamespace()) {
----------------
ilovepi wrote:
@PiotrZSL shouldn't his handle the anonymous namespace concern below?
https://github.com/llvm/llvm-project/pull/98424
More information about the cfe-commits
mailing list