[Lldb-commits] [PATCH] D31451: New C++ function name parsing logic

Jim Ingham via lldb-commits lldb-commits at lists.llvm.org
Tue Apr 4 17:22:07 PDT 2017

This patch is causing a testsuite failure in ObjC breakpoint setting:


You must be grabbing ObjC names and somehow treating them as C++ names.

I reverted the patch till the test gets fixed.  If you can't get your hands on 
an ObjC binary to test I can send you one.  You don't need to run code, you just
need to load a binary, which you should be able to do on any lldb host.


> On Apr 3, 2017, at 12:12 PM, Eugene Zemtsov via Phabricator <reviews at reviews.llvm.org> wrote:
> This revision was automatically updated to reflect the committed changes.
> Closed by commit rL299374: New C++ function name parsing logic (authored by eugene).
> Changed prior to commit:
>  https://reviews.llvm.org/D31451?vs=93694&id=93910#toc
> Repository:
>  rL LLVM
> https://reviews.llvm.org/D31451
> Files:
>  lldb/trunk/source/Plugins/Language/CPlusPlus/CMakeLists.txt
>  lldb/trunk/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.cpp
>  lldb/trunk/source/Plugins/Language/CPlusPlus/CPlusPlusLanguage.h
>  lldb/trunk/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.cpp
>  lldb/trunk/source/Plugins/Language/CPlusPlus/CPlusPlusNameParser.h
>  lldb/trunk/unittests/Language/CPlusPlus/CPlusPlusLanguageTest.cpp
> <D31451.93910.patch>

More information about the lldb-commits mailing list