labath added a comment.

In https://reviews.llvm.org/D52461#1266302, @aleksandr.urakov wrote:

>   `operator<<A>'::`2'::B::operator>
>


The reason we had to use clang lexer for parsing itanium names is because 
parsing itanium demangled names is tricky precisely for cases like these. If 
the MSVC demangler makes these cases trivial by enclosing them in quotes, maybe 
a separate (simpler) parser is not such a bad idea.

However, I still think this should be done within the scope of 
`CPlusPlusLanguage::MethodName` otherwise, you'll have to special case MSVC for 
all existing uses of this class.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D52461



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to