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

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Sun Sep 30 11:13:33 PDT 2018


labath added a comment.

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

> It seems that `CPlusPlusLanguage::MethodName` is backed by LLDB `CPlusPlusNameParser`, which can't parse demangled names...


What makes you say that? If you look at the MethodName unit tests (`unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp`), you will see that they all operate on demangled names -- there isn't a mangled name in the whole file.

Using RichManglingContext would work too, though I'm not sure if that will buy you anything, as that is just a wrapper around this parser.


Repository:
  rLLDB LLDB

https://reviews.llvm.org/D52461





More information about the lldb-commits mailing list