Author: Owen Pan Date: 2023-03-08T05:18:52-08:00 New Revision: 7b4e71bcf991a6b2e5cee26112a14bb62ceee0b8
URL: https://github.com/llvm/llvm-project/commit/7b4e71bcf991a6b2e5cee26112a14bb62ceee0b8 DIFF: https://github.com/llvm/llvm-project/commit/7b4e71bcf991a6b2e5cee26112a14bb62ceee0b8.diff LOG: [clang-format] Don't annotate left brace of class as FunctionLBrace The l_brace of class/struct/union was incorrectly annotated as TT_FunctionLBrace in the presence of attributes. This in turn would cause the RemoveSemicolon option to remove the semicolon at the end of the declaration, resulting in invalid code being generated. Fixes #61188. Differential Revision: https://reviews.llvm.org/D145344 (cherry picked from commit a02c3af9f19d805411364e7a0de785c1d1c4e348) Added: Modified: clang/lib/Format/UnwrappedLineParser.cpp clang/unittests/Format/FormatTest.cpp clang/unittests/Format/TokenAnnotatorTest.cpp Removed: ################################################################################ diff --git a/clang/lib/Format/UnwrappedLineParser.cpp b/clang/lib/Format/UnwrappedLineParser.cpp index 3e58b6f90559b..7e3957e62d3ae 100644 --- a/clang/lib/Format/UnwrappedLineParser.cpp +++ b/clang/lib/Format/UnwrappedLineParser.cpp @@ -3849,13 +3849,13 @@ void UnwrappedLineParser::parseJavaEnumBody() { void UnwrappedLineParser::parseRecord(bool ParseAsExpr) { const FormatToken &InitialToken = *FormatTok; nextToken(); + handleAttributes(); // The actual identifier can be a nested name specifier, and in macros // it is often token-pasted. - // An [[attribute]] can be before the identifier. while (FormatTok->isOneOf(tok::identifier, tok::coloncolon, tok::hashhash, tok::kw___attribute, tok::kw___declspec, - tok::kw_alignas, tok::l_square, tok::r_square) || + tok::kw_alignas) || ((Style.Language == FormatStyle::LK_Java || Style.isJavaScript()) && FormatTok->isOneOf(tok::period, tok::comma))) { if (Style.isJavaScript() && @@ -3873,15 +3873,10 @@ void UnwrappedLineParser::parseRecord(bool ParseAsExpr) { FormatTok->is(tok::identifier) && FormatTok->TokenText != FormatTok->TokenText.upper(); nextToken(); - // We can have macros or attributes in between 'class' and the class name. + // We can have macros in between 'class' and the class name. if (!IsNonMacroIdentifier) { if (FormatTok->is(tok::l_paren)) { parseParens(); - } else if (FormatTok->is(TT_AttributeSquare)) { - parseSquare(); - // Consume the closing TT_AttributeSquare. - if (FormatTok->Next && FormatTok->is(TT_AttributeSquare)) - nextToken(); } } } diff --git a/clang/unittests/Format/FormatTest.cpp b/clang/unittests/Format/FormatTest.cpp index 75da96a1b14cc..189c8ee9e9da8 100644 --- a/clang/unittests/Format/FormatTest.cpp +++ b/clang/unittests/Format/FormatTest.cpp @@ -25207,6 +25207,11 @@ TEST_F(FormatTest, RemoveSemicolon) { "}", Style); + verifyFormat("class [[deprecated(\"\")]] C {\n" + " int i;\n" + "};", + Style); + verifyIncompleteFormat("class C final [[deprecated(l]] {});", Style); // These tests are here to show a problem that may not be easily diff --git a/clang/unittests/Format/TokenAnnotatorTest.cpp b/clang/unittests/Format/TokenAnnotatorTest.cpp index 3ed6521d48c1f..a74a2992823e0 100644 --- a/clang/unittests/Format/TokenAnnotatorTest.cpp +++ b/clang/unittests/Format/TokenAnnotatorTest.cpp @@ -278,6 +278,10 @@ TEST_F(TokenAnnotatorTest, UnderstandsClasses) { Tokens = annotate("const class {} c;"); EXPECT_EQ(Tokens.size(), 7u) << Tokens; EXPECT_TOKEN(Tokens[2], tok::l_brace, TT_ClassLBrace); + + Tokens = annotate("class [[deprecated(\"\")]] C { int i; };"); + EXPECT_EQ(Tokens.size(), 17u) << Tokens; + EXPECT_TOKEN(Tokens[10], tok::l_brace, TT_ClassLBrace); } TEST_F(TokenAnnotatorTest, UnderstandsStructs) { _______________________________________________ llvm-branch-commits mailing list llvm-branch-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
