[clang-format] Dont interpret variable named interface as keyword for C++

[sousajo]

Summary

Fixes: https://github.com/llvm/llvm-project/issues/53173

[sousajo]

As I dont have commit access if someone can land It for me:

Jorge Pinto Sousa
<[e-mail]>

[owenpan]

clang/unittests/Format/FormatTest.cpp

25432

25433–25439

The default style is LLVM.

[sousajo]

• removed FormatStyle Style = getLLVMStyle(); as llvm is the default style

[sousajo]

• inline string to be checked

[sousajo]

• rebased

[sousajo]

cannot reproduce the build failure locally. if this looks fine can someone land it for me? or should we wait until we have a green build after rebase?

clang/lib/Format/UnwrappedLineParser.cpp

case tok::identifier: {
         if (Keywords.isVerilogBegin(*FormatTok) ||
             Keywords.isVerilogHierarchy(*FormatTok)) {
           parseBlock();
           addUnwrappedLine();
           return;
         }
       }
 
-      if (FormatTok->is(Keywords.kw_interface)) {
+      if (!Style.isCpp() && FormatTok->is(Keywords.kw_interface)) {
         if (parseStructLike())
           return;
         break;
       }
 
       if (Style.isCpp() && FormatTok->is(TT_StatementMacro)) {
         parseStatementMacro();
         return;

clang/unittests/Format/FormatTest.cpp

TEST_F(FormatTest, InsertNewlineAtEOF) {
   verifyFormat("int i;\n", "int i;", Style);
 }
 
 TEST_F(FormatTest, SpaceAfterUDL) {
   verifyFormat("auto c = (4s).count();");
   verifyFormat("auto x = 5s .count() == 5;");
 }
 
+TEST_F(FormatTest, InterfaceAsClassMemberName) {
+  constexpr StringRef Code("class Foo {\n"
+                           "  int interface;\n"
+                           "  Foo::Foo(int iface) : interface{iface} {}\n"
+                           "}\n");
+  verifyFormat(Code);
+}
+
 } // namespace
 } // namespace test
 } // namespace format
 } // namespace clang