[clang] [clang-format] Fix a bug in `DerivePointerAlignment: true` (PR #150387)
Owen Pan via cfe-commits
cfe-commits at lists.llvm.org
Thu Jul 24 01:24:42 PDT 2025
https://github.com/owenca created https://github.com/llvm/llvm-project/pull/150387
Fixes #150327
>From 12777c2044cc9daac3fffcb96c0c1f4c772e9147 Mon Sep 17 00:00:00 2001
From: Owen Pan <owenpiano at gmail.com>
Date: Thu, 24 Jul 2025 01:22:36 -0700
Subject: [PATCH] [clang-format] Fix a bug in `DerivePointerAlignment: true`
Fixes #150327
---
clang/lib/Format/Format.cpp | 5 +++--
clang/unittests/Format/FormatTest.cpp | 2 ++
2 files changed, 5 insertions(+), 2 deletions(-)
diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp
index 62feb3db0ed5e..3eab44f0d1642 100644
--- a/clang/lib/Format/Format.cpp
+++ b/clang/lib/Format/Format.cpp
@@ -2643,13 +2643,14 @@ class Formatter : public TokenAnalyzer {
for (FormatToken *Tok = Line->First; Tok && Tok->Next; Tok = Tok->Next) {
if (Tok->isNot(TT_PointerOrReference))
continue;
- // Don't treat space in `void foo() &&` as evidence.
+ // Don't treat space in `void foo() &&` or `void() &&` as evidence.
if (const auto *Prev = Tok->getPreviousNonComment()) {
if (Prev->is(tok::r_paren) && Prev->MatchingParen) {
if (const auto *Func =
Prev->MatchingParen->getPreviousNonComment()) {
if (Func->isOneOf(TT_FunctionDeclarationName, TT_StartOfName,
- TT_OverloadedOperator)) {
+ TT_OverloadedOperator) ||
+ Func->isTypeName(LangOpts)) {
continue;
}
}
diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp
index dbf6950446ef0..e21d7a88e81bd 100644
--- a/clang/unittests/Format/FormatTest.cpp
+++ b/clang/unittests/Format/FormatTest.cpp
@@ -12103,6 +12103,8 @@ TEST_F(FormatTest, UnderstandsFunctionRefQualification) {
Prefix = "void a() const &;\n"
"void b() const &;\n";
verifyFormat(Prefix + "int *x;", Prefix + "int* x;", DerivePointerAlignment);
+
+ verifyGoogleFormat("MACRO(int*, std::function<void() &&>);");
}
TEST_F(FormatTest, PointerAlignmentFallback) {
More information about the cfe-commits
mailing list