[clang] [clang-format] Fix a crash in AlignArrayOfStructures (PR #167099)
via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 7 22:19:07 PST 2025
https://github.com/owenca created https://github.com/llvm/llvm-project/pull/167099
Fixes #157405
>From 981d3ac3bf6d8bdd212b5a93944e185c468e0956 Mon Sep 17 00:00:00 2001
From: Owen Pan <owenpiano at gmail.com>
Date: Mon, 8 Sep 2025 21:27:07 -0700
Subject: [PATCH] [clang-format] Fix a crash in AlignArrayOfStructures
Fixes #157405
---
clang/lib/Format/WhitespaceManager.cpp | 5 ++++-
clang/unittests/Format/FormatTest.cpp | 13 +++++++++++++
2 files changed, 17 insertions(+), 1 deletion(-)
diff --git a/clang/lib/Format/WhitespaceManager.cpp b/clang/lib/Format/WhitespaceManager.cpp
index f24b8ab14bdce..d37834e2df902 100644
--- a/clang/lib/Format/WhitespaceManager.cpp
+++ b/clang/lib/Format/WhitespaceManager.cpp
@@ -1215,7 +1215,10 @@ void WhitespaceManager::alignArrayInitializers() {
bool FoundComplete = false;
for (unsigned InsideIndex = ChangeIndex + 1; InsideIndex < ChangeEnd;
++InsideIndex) {
- if (Changes[InsideIndex].Tok == C.Tok->MatchingParen) {
+ const auto *Tok = Changes[InsideIndex].Tok;
+ if (Tok->is(tok::pp_define))
+ break;
+ if (Tok == C.Tok->MatchingParen) {
alignArrayInitializers(ChangeIndex, InsideIndex + 1);
ChangeIndex = InsideIndex + 1;
FoundComplete = true;
diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp
index ca9e7925e5e95..36ce5ef35acb3 100644
--- a/clang/unittests/Format/FormatTest.cpp
+++ b/clang/unittests/Format/FormatTest.cpp
@@ -22265,6 +22265,19 @@ TEST_F(FormatTest, CatchAlignArrayOfStructuresLeftAlignment) {
"});",
Style);
+ verifyNoCrash(
+ "PANEL_Ic PANEL_ic[PANEL_IC_NUMBER] =\n"
+ " {\n"
+ " {PIC(0), PIC(0), PIC(99), PIC(81), 0}, // Backbox\n"
+ " {PIC(1), PIC(83), PIC(191), PIC(137), 0}, // AK47\n"
+ "\n"
+ "#define PICALL1(a, b, c, d) \\\n"
+ " { PIC(a), PIC(b), PIC(c), PIC(d), 1 }\n"
+ "\n"
+ " PICALL1(1, 1, 75, 50),\n"
+ "};",
+ Style);
+
Style.AlignEscapedNewlines = FormatStyle::ENAS_DontAlign;
verifyFormat("#define FOO \\\n"
" int foo[][2] = { \\\n"
More information about the cfe-commits
mailing list