[Lldb-commits] [lldb] [lldb] Support MS style `struct`/`class` in C++ name parser (PR #196525)

[Michael Buch via lldb-commits]

================
@@ -34,6 +34,13 @@ TEST(CPlusPlusLanguage, MethodNameParsing) {
        "foo::bar"},
       {"int foo::bar::func01(int a, double b)", "int", "foo::bar", "func01",
        "(int a, double b)", "", "foo::bar::func01"},
+      {"foo(struct AStruct)", "", "", "foo", "(struct AStruct)", "", "foo"},
----------------
Michael137 wrote:

Since this isn't a valid C++ name, don't think we can have any concrete expectation as to what this should produce. Just erroring out is probably not an option if we can get here with valid C symbols.

Just adding the case as-is to ensure we don't do something crazy like crash, is still worth it

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