[Lldb-commits] [PATCH] D52461: [PDB] Introduce `PDBNameParser`

[Aleksandr Urakov via Phabricator via lldb-commits]

aleksandr.urakov added a comment.

In https://reviews.llvm.org/D52461#1265633, @zturner wrote:

> Just handle the `anonymous namespace' thing specially before passing to `CPlusPlusNameParser`.


Yes, it's an interesting idea to somehow preprocess an MSVC demangled name and make a GCC demangled name from it (and make an MSVC-like name back after parsing). But then we need to handle not only anonymous namespaces, also things like this:

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

Such a preprocessing will be comparable to the current implementation of `PDBNameParser` by complexity (or even more complex). I'll try to somehow estimate the complexity of this approach, thanks.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D52461