[clang-tools-extra] [clang-tidy] Update llvmlibc-implementation-in-namespace to new rules (PR #66504)

Piotr Zegar via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 18 08:27:16 PDT 2023


================
@@ -30,15 +32,18 @@ void ImplementationInNamespaceCheck::check(
     return;
 
   if (const auto *NS = dyn_cast<NamespaceDecl>(MatchedDecl)) {
-    if (NS->getName() != RequiredNamespace) {
-      diag(NS->getLocation(), "'%0' needs to be the outermost namespace")
-          << RequiredNamespace;
-    }
+    if (!Result.SourceManager->isMacroBodyExpansion(NS->getLocation()))
+      diag(NS->getLocation(),
+           "the outermost namespace should be the '%0' macro")
+          << RequiredNamespaceMacroName;
+    else if (!NS->getName().starts_with(RequiredNamespaceStart))
----------------
PiotrZSL wrote:

getName will assert if Namespace got not identifier. What about anonymous namespaces ? Or declarations without name ?

https://github.com/llvm/llvm-project/pull/66504


More information about the cfe-commits mailing list