[clang] [clang][FMV] Allow declaration of function versions in namespaces. (PR #93044)
via cfe-commits
cfe-commits at lists.llvm.org
Wed May 22 08:13:34 PDT 2024
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff e5936b245e9af0cea69a7e4eae22a05b7ffcf5a3 e03d8cfdc697bf714d1f13247233cf514b77f0de -- clang/test/CodeGenCXX/fmv-namespace.cpp clang/test/Sema/fmv-namespace.cpp clang/lib/Sema/SemaDecl.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp
index 297a2feb66..99057c50cf 100644
--- a/clang/lib/Sema/SemaDecl.cpp
+++ b/clang/lib/Sema/SemaDecl.cpp
@@ -11869,9 +11869,9 @@ static bool CheckMultiVersionFunction(Sema &S, FunctionDecl *NewFD,
if (!OldDecl || !OldDecl->getAsFunction() ||
(OldDecl->getDeclContext()->getRedeclContext() !=
- NewFD->getDeclContext()->getRedeclContext() &&
+ NewFD->getDeclContext()->getRedeclContext() &&
OldDecl->getDeclContext()->getEnclosingNamespaceContext() !=
- NewFD->getDeclContext()->getEnclosingNamespaceContext())) {
+ NewFD->getDeclContext()->getEnclosingNamespaceContext())) {
// If there's no previous declaration, AND this isn't attempting to cause
// multiversioning, this isn't an error condition.
if (MVKind == MultiVersionKind::None)
``````````
</details>
https://github.com/llvm/llvm-project/pull/93044
More information about the cfe-commits
mailing list