[clang] [clang-format] Add BreakFunctionDeclarationParameters option. (PR #196567)
via cfe-commits
cfe-commits at lists.llvm.org
Sat May 9 01:23:30 PDT 2026
https://github.com/stativ updated https://github.com/llvm/llvm-project/pull/196567
>From 84484463808b79a974fa7835c958f226252d782e Mon Sep 17 00:00:00 2001
From: Lukas Jirkovsky <l.jirkovsky at gmail.com>
Date: Fri, 8 May 2026 13:18:50 +0200
Subject: [PATCH 1/4] [clang-format] Add BreakFunctionDeclarationParameters
option.
Adds an option the break function declaration parameters, always
putting them on the next line after the function opening parentheses.
This is an equivalent of `BreakFunctionDefinitionParameters`, but for
function declarations.
---
clang/docs/ClangFormatStyleOptions.rst | 14 ++++++++++
clang/include/clang/Format/Format.h | 16 ++++++++++++
clang/lib/Format/Format.cpp | 3 +++
clang/lib/Format/TokenAnnotator.cpp | 6 +++++
clang/unittests/Format/AlignBracketsTest.cpp | 9 +++++++
clang/unittests/Format/FormatTest.cpp | 27 ++++++++++++++++++++
6 files changed, 75 insertions(+)
diff --git a/clang/docs/ClangFormatStyleOptions.rst b/clang/docs/ClangFormatStyleOptions.rst
index d492f2364cf74..04d382df3f90f 100644
--- a/clang/docs/ClangFormatStyleOptions.rst
+++ b/clang/docs/ClangFormatStyleOptions.rst
@@ -3857,6 +3857,20 @@ the configuration (without a prefix: ``Auto``).
initializer2()
+.. _BreakFunctionDeclarationParameters:
+
+**BreakFunctionDeclarationParameters** (``Boolean``) :versionbadge:`clang-format 23` :ref:`¶ <BreakFunctionDeclarationParameters>`
+ If ``true``, clang-format will always break before function declaration
+ parameters.
+
+ .. code-block:: c++
+
+ true:
+ void functionDeclaration(
+ int A, int B);
+
+ false:
+ void functionDeclaration(int A, int B);
.. _BreakFunctionDefinitionParameters:
diff --git a/clang/include/clang/Format/Format.h b/clang/include/clang/Format/Format.h
index 98400a1609b6a..0e883837ac0e9 100644
--- a/clang/include/clang/Format/Format.h
+++ b/clang/include/clang/Format/Format.h
@@ -2644,6 +2644,20 @@ struct FormatStyle {
/// \version 5
BreakConstructorInitializersStyle BreakConstructorInitializers;
+ /// If ``true``, clang-format will always break before function declaration
+ /// parameters.
+ /// \code
+ /// true:
+ /// void functionDeclaration(
+ /// int A, int B);
+ ///
+ /// false:
+ /// void functionDeclaration(int A, int B);
+ ///
+ /// \endcode
+ /// \version 23
+ bool BreakFunctionDeclarationParameters;
+
/// If ``true``, clang-format will always break before function definition
/// parameters.
/// \code
@@ -6076,6 +6090,8 @@ struct FormatStyle {
BreakBeforeTernaryOperators == R.BreakBeforeTernaryOperators &&
BreakBinaryOperations == R.BreakBinaryOperations &&
BreakConstructorInitializers == R.BreakConstructorInitializers &&
+ BreakFunctionDeclarationParameters ==
+ R.BreakFunctionDeclarationParameters &&
BreakFunctionDefinitionParameters ==
R.BreakFunctionDefinitionParameters &&
BreakInheritanceList == R.BreakInheritanceList &&
diff --git a/clang/lib/Format/Format.cpp b/clang/lib/Format/Format.cpp
index 2147a812e27c1..74b31810843fc 100644
--- a/clang/lib/Format/Format.cpp
+++ b/clang/lib/Format/Format.cpp
@@ -1318,6 +1318,8 @@ template <> struct MappingTraits<FormatStyle> {
IO.mapOptional("BreakBinaryOperations", Style.BreakBinaryOperations);
IO.mapOptional("BreakConstructorInitializers",
Style.BreakConstructorInitializers);
+ IO.mapOptional("BreakFunctionDeclarationParameters",
+ Style.BreakFunctionDeclarationParameters);
IO.mapOptional("BreakFunctionDefinitionParameters",
Style.BreakFunctionDefinitionParameters);
IO.mapOptional("BreakInheritanceList", Style.BreakInheritanceList);
@@ -1885,6 +1887,7 @@ FormatStyle getLLVMStyle(FormatStyle::LanguageKind Language) {
LLVMStyle.BreakBeforeTernaryOperators = true;
LLVMStyle.BreakBinaryOperations = {FormatStyle::BBO_Never, {}};
LLVMStyle.BreakConstructorInitializers = FormatStyle::BCIS_BeforeColon;
+ LLVMStyle.BreakFunctionDeclarationParameters = false;
LLVMStyle.BreakFunctionDefinitionParameters = false;
LLVMStyle.BreakInheritanceList = FormatStyle::BILS_BeforeColon;
LLVMStyle.BreakStringLiterals = true;
diff --git a/clang/lib/Format/TokenAnnotator.cpp b/clang/lib/Format/TokenAnnotator.cpp
index 898759cb8ea1b..640f03a4ac130 100644
--- a/clang/lib/Format/TokenAnnotator.cpp
+++ b/clang/lib/Format/TokenAnnotator.cpp
@@ -5758,6 +5758,12 @@ bool TokenAnnotator::mustBreakBefore(AnnotatedLine &Line,
const FormatToken &Left = *Right.Previous;
+ if (Style.BreakFunctionDeclarationParameters && Line.MightBeFunctionDecl &&
+ !Line.mightBeFunctionDefinition() && Left.MightBeFunctionDeclParen &&
+ Left.ParameterCount > 0) {
+ return true;
+ }
+
if (Style.BreakFunctionDefinitionParameters && Line.MightBeFunctionDecl &&
Line.mightBeFunctionDefinition() && Left.MightBeFunctionDeclParen &&
Left.ParameterCount > 0) {
diff --git a/clang/unittests/Format/AlignBracketsTest.cpp b/clang/unittests/Format/AlignBracketsTest.cpp
index cd314305751e7..fcfcae20e3e11 100644
--- a/clang/unittests/Format/AlignBracketsTest.cpp
+++ b/clang/unittests/Format/AlignBracketsTest.cpp
@@ -731,6 +731,15 @@ TEST_F(AlignBracketsTest, FormatsDefinitionBreakAlways) {
"}",
BreakAlways);
+ // Ensure BreakFunctionDeclarationParameters interacts correctly when
+ // PackParameters.BinPack is set to BPPS_AlwaysOnePerLine.
+ BreakAlways.BreakFunctionDeclarationParameters = true;
+ verifyFormat("void f(\n"
+ " int a,\n"
+ " int b);",
+ BreakAlways);
+ BreakAlways.BreakFunctionDeclarationParameters = false;
+
// Ensure BreakFunctionDefinitionParameters interacts correctly when
// PackParameters.BinPack is set to BPPS_AlwaysOnePerLine.
BreakAlways.BreakFunctionDefinitionParameters = true;
diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp
index f5e496652e15e..4245bd1c58153 100644
--- a/clang/unittests/Format/FormatTest.cpp
+++ b/clang/unittests/Format/FormatTest.cpp
@@ -8119,6 +8119,33 @@ TEST_F(FormatTest, AllowAllArgumentsOnNextLine) {
Style);
}
+TEST_F(FormatTest, BreakFunctionDeclarationParameters) {
+ StringRef Input = "void functionDecl(int A, int B, int C);\n"
+ "void emptyFunctionDecl();\n"
+ "void functionDefinition(int A, int B, int C) {}";
+ verifyFormat(Input);
+
+ FormatStyle Style = getLLVMStyle();
+ EXPECT_FALSE(Style.BreakFunctionDeclarationParameters);
+ Style.BreakFunctionDeclarationParameters = true;
+ verifyFormat("void functionDecl(\n"
+ " int A, int B, int C);\n"
+ "void emptyFunctionDecl();\n"
+ "void functionDefinition(int A, int B, int C) {}",
+ Input, Style);
+
+ // Test the style where all parameters are on their own lines.
+ Style.AllowAllParametersOfDeclarationOnNextLine = false;
+ Style.PackParameters.BinPack = FormatStyle::BPPS_OnePerLine;
+ verifyFormat("void functionDecl(\n"
+ " int A,\n"
+ " int B,\n"
+ " int C);\n"
+ "void emptyFunctionDecl();\n"
+ "void functionDefinition(int A, int B, int C) {}",
+ Input, Style);
+}
+
TEST_F(FormatTest, BreakFunctionDefinitionParameters) {
StringRef Input = "void functionDecl(paramA, paramB, paramC);\n"
"void emptyFunctionDefinition() {}\n"
>From 3525e0bbc3596a00e624c7ef9f50909ced327ccc Mon Sep 17 00:00:00 2001
From: Lukas Jirkovsky <lukas.jirkovsky at aveco.com>
Date: Sat, 9 May 2026 08:46:25 +0200
Subject: [PATCH 2/4] Update ReleaseNotes
---
clang/docs/ReleaseNotes.rst | 2 ++
1 file changed, 2 insertions(+)
diff --git a/clang/docs/ReleaseNotes.rst b/clang/docs/ReleaseNotes.rst
index ac462e3bf4732..24d72e8ab48b4 100644
--- a/clang/docs/ReleaseNotes.rst
+++ b/clang/docs/ReleaseNotes.rst
@@ -738,6 +738,8 @@ clang-format
- Extend ``BreakBinaryOperations`` to accept a structured configuration with
per-operator break rules and minimum chain length gating via ``PerOperator``.
- Add ``AllowShortRecordOnASingleLine`` option and set it to ``EmptyAndAttached`` for LLVM style.
+- Add ``BreakFunctionDeclarationParameters`` option to always break before function
+ declaration parameters.
libclang
--------
>From 1058beab2000dc43aef9bf3ae1616816fd21b5c7 Mon Sep 17 00:00:00 2001
From: Lukas Jirkovsky <lukas.jirkovsky at aveco.com>
Date: Sat, 9 May 2026 08:52:39 +0200
Subject: [PATCH 3/4] Add test to ConfigParseTest
---
clang/unittests/Format/ConfigParseTest.cpp | 1 +
1 file changed, 1 insertion(+)
diff --git a/clang/unittests/Format/ConfigParseTest.cpp b/clang/unittests/Format/ConfigParseTest.cpp
index 64b0e8702872c..ccb9c837d8362 100644
--- a/clang/unittests/Format/ConfigParseTest.cpp
+++ b/clang/unittests/Format/ConfigParseTest.cpp
@@ -183,6 +183,7 @@ TEST(ConfigParseTest, ParsesConfigurationBools) {
CHECK_PARSE_BOOL(BreakBeforeCloseBracketSwitch);
CHECK_PARSE_BOOL(BreakBeforeTemplateCloser);
CHECK_PARSE_BOOL(BreakBeforeTernaryOperators);
+ CHECK_PARSE_BOOL(BreakFunctionDeclarationParameters);
CHECK_PARSE_BOOL(BreakStringLiterals);
CHECK_PARSE_BOOL(CompactNamespaces);
CHECK_PARSE_BOOL(DerivePointerAlignment);
>From ba249a49714f7b4ff4f1bebab28732a1002b9b23 Mon Sep 17 00:00:00 2001
From: Lukas Jirkovsky <lukas.jirkovsky at aveco.com>
Date: Sat, 9 May 2026 10:23:06 +0200
Subject: [PATCH 4/4] fix Clang.Format/docs_updated.test
---
clang/docs/ClangFormatStyleOptions.rst | 1 +
1 file changed, 1 insertion(+)
diff --git a/clang/docs/ClangFormatStyleOptions.rst b/clang/docs/ClangFormatStyleOptions.rst
index 04d382df3f90f..61f27bcf9dbbc 100644
--- a/clang/docs/ClangFormatStyleOptions.rst
+++ b/clang/docs/ClangFormatStyleOptions.rst
@@ -3857,6 +3857,7 @@ the configuration (without a prefix: ``Auto``).
initializer2()
+
.. _BreakFunctionDeclarationParameters:
**BreakFunctionDeclarationParameters** (``Boolean``) :versionbadge:`clang-format 23` :ref:`¶ <BreakFunctionDeclarationParameters>`
More information about the cfe-commits
mailing list