[clang] [polly] [clang-format] Correctly annotate braces in macros (PR #87953)

Owen Pan via cfe-commits cfe-commits at lists.llvm.org
Sun Apr 7 17:54:14 PDT 2024


================
@@ -538,16 +538,6 @@ void UnwrappedLineParser::calculateBraceTypes(bool ExpectClassBody) {
         if (Style.Language == FormatStyle::LK_Proto) {
           ProbablyBracedList = NextTok->isOneOf(tok::comma, tok::r_square);
         } else {
-          // Skip NextTok over preprocessor lines, otherwise we may not
-          // properly diagnose the block as a braced intializer
-          // if the comma separator appears after the pp directive.
-          while (NextTok->is(tok::hash)) {
-            ScopedMacroState MacroState(*Line, Tokens, NextTok);
-            do {
-              NextTok = Tokens->getNextToken();
-            } while (NextTok->isNot(tok::eof));
-          }
----------------
owenca wrote:

This loop is superfluous due to line 498 above.

https://github.com/llvm/llvm-project/pull/87953


More information about the cfe-commits mailing list