[clang-format] Mark pragma region lines as StringLiterals

[thieta]

Summary

In our code-base we auto-generate pragma regions the regions
look like method signatures like:

#pragma region MYREGION(Foo: bar)

The problem here was that the rest of the line after region
was not marked as stringliteral as in the case of pragma mark
so clang-format tried to change the formatting based on the
method signature.

Added test and mark it similar as pragma mark.

[owenpan]

Can you add a test to TokenAnnotatorTest.cpp?

clang/unittests/Format/FormatTest.cpp

19973–19974

Redundant?

[thieta]

Added TokenAnnotatorTest and removed reduntant FormatTests

[thieta]

In D136336#3877238, @owenpan wrote:

Can you add a test to TokenAnnotatorTest.cpp?

Thanks for the review. Fixed it.

[owenpan]

Can you run git-clang-format?

clang/lib/Format/TokenAnnotator.cpp

1340

Over 80 columns.

clang/unittests/Format/TokenAnnotatorTest.cpp

9 ↗

(On Diff #470092)

Extraneous.

[thieta]

git clang-format
removed accidental include

[owenpan]

LGTM

[HazardyKnusperkeks]

clang/lib/Format/TokenAnnotator.cpp

1344

This shouldn't be here. There is no closing brace for this, or am I missing something?

clang/unittests/Format/TokenAnnotatorTest.cpp

653 ↗

(On Diff #470395)

Maybe you can add that in a additional commit.

[thieta]

clang/lib/Format/TokenAnnotator.cpp

1344

No you are right it was a miss in the rebase. But it's fixed on main now.

[owenpan]

clang/lib/Format/TokenAnnotator.cpp

1344

Probably caused by https://github.com/llvm/llvm-project/issues/58161, which was fixed in 37e754e5801c.

clang/lib/Format/TokenAnnotator.cpp

void parseWarningOrError() {
     }
   }
 
   void parsePragma() {
     next(); // Consume "pragma".
     if (CurrentToken &&
         CurrentToken->isOneOf(Keywords.kw_mark, Keywords.kw_option,
                               Keywords.kw_region)) {
-      bool IsMark = CurrentToken->is(Keywords.kw_mark);
+      bool IsMarkOrRegion = CurrentToken->isOneOf(Keywords.kw_mark, Keywords.kw_region);
       next();
       next(); // Consume first token (so we fix leading whitespace).
       while (CurrentToken) {
-        if (IsMark || CurrentToken->Previous->is(TT_BinaryOperator))
+        if (IsMarkOrRegion || CurrentToken->Previous->is(TT_BinaryOperator))
           CurrentToken->setType(TT_ImplicitStringLiteral);
         next();
       }
     }
   }
 
   void parseHasInclude() {
     if (!CurrentToken || !CurrentToken->is(tok::l_paren))

clang/unittests/Format/FormatTest.cpp

 }
 
 TEST_F(FormatTest, UnderstandPragmaRegion) {
   auto Style = getLLVMStyleWithColumns(0);
   verifyFormat("#pragma region TEST(FOO : BAR)", Style);
 
   EXPECT_EQ("#pragma region TEST(FOO : BAR)",
             format("#pragma region TEST(FOO : BAR)", Style));
+
+  verifyFormat("#pragma region TEST(FOO: NOSPACE)", Style);
+  EXPECT_EQ("#pragma region TEST(FOO: NOSPACE)",
+            format("#pragma region TEST(FOO: NOSPACE)", Style));
+
 }
 
 TEST_F(FormatTest, OptimizeBreakPenaltyVsExcess) {
   FormatStyle Style = getLLVMStyleWithColumns(20);
 
   // See PR41213
   EXPECT_EQ("/*\n"
             " *\t9012345\n"