No problem. You can't test on all platforms, so changes like this are bound to sometimes introduce regressions. If you can remember to watch the bots for systems you don't run or test on that makes the turnaround smoother. Otherwise, if it's a MacOS problem the QE folks here are sure to tell me about it...
Jim > On Apr 4, 2017, at 10:03 PM, Eugene Zemtsov <ezemt...@google.com> wrote: > > Sorry about that and thanks for rolling it back. I'll take a look tomorrow. > > On Tue, Apr 4, 2017 at 5:22 PM, Jim Ingham <jing...@apple.com> wrote: > This patch is causing a testsuite failure in ObjC breakpoint setting: > > functionalities/breakpoint/objc/TestObjCBreakpoints.py > > 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. > > Jim > > > > On Apr 3, 2017, at 12:12 PM, Eugene Zemtsov via Phabricator > > <revi...@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> > > > > > -- > Thanks, > Eugene Zemtsov. _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits