================
@@ -481,18 +481,22 @@ bool 
CPlusPlusLanguage::CxxMethodName::TrySimplifiedParse() {
       m_basename = full.substr(basename_begin, basename_end - basename_begin);
     }
 
-    if (IsTrivialBasename(m_basename)) {
+    // if the context has a white space it may have a return type.
----------------
dmpots wrote:

Could you add more details to the summary about what cases are fixed with this 
change and what happens without the fix (e.g. crash, incorrect output).

It looks like previously we were incorrectly handling cases that had both a 
context and a return type, but seems like it did handle the case where we had a 
return type without a context based on this test case

```
{"int main()", "int", "", "main", "()", "", "main"},
```


https://github.com/llvm/llvm-project/pull/169652
_______________________________________________
lldb-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to