Prabhuk wrote:
New version of this PR is uploaded by Paul Kirth here:
https://github.com/llvm/llvm-project/pull/98424
Closing this pull request.
https://github.com/llvm/llvm-project/pull/98088
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
h
https://github.com/Prabhuk closed
https://github.com/llvm/llvm-project/pull/98088
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Prabhuk updated
https://github.com/llvm/llvm-project/pull/98088
>From 94ee0f39f77264a4a5ccdd7d626bfa00841b7b22 Mon Sep 17 00:00:00 2001
From: prabhukr
Date: Mon, 8 Jul 2024 15:18:10 -0700
Subject: [PATCH] [libc] Update libc namespace clang-tidy checks
Namespace macro that sh
https://github.com/Prabhuk updated
https://github.com/llvm/llvm-project/pull/98088
>From 63bcfd56f8b0cbdc767798608f144983e1f9a512 Mon Sep 17 00:00:00 2001
From: prabhukr
Date: Mon, 8 Jul 2024 15:18:10 -0700
Subject: [PATCH] [libc] Update libc namespace clang-tidy checks
Namespace macro that sh
@@ -11,6 +11,8 @@
namespace clang::tidy::llvm_libc {
const static llvm::StringRef RequiredNamespaceStart = "__llvm_libc";
+const static llvm::StringRef RequiredNamespaceHiddenMacroName =
frobtech wrote:
I think it would be better to call this one "...Namespac
https://github.com/Prabhuk created
https://github.com/llvm/llvm-project/pull/98088
Namespace macro that should be used to declare a new namespace is
updated from LIBC_NAMESPACE to LIBC_NAMESPACE_DECL which by default has
hidden visibility (#97109). This commit updates the clang-tidy checks to
ma
