[Lldb-commits] [PATCH] D36804: Add initial support to PowerPC64 little endian (POWER8)

2017-08-18 Thread Gustavo Serra Scalet via Phabricator via lldb-commits
gut added a comment. Can I please get some review on this? ps: check comments on phabricator as it was not being published on lldb-commits mailing list. https://reviews.llvm.org/D36804 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http

[Lldb-commits] [PATCH] D36885: [cmake] Explicitly link dependency libraries in the Host library

2017-08-18 Thread Michał Górny via Phabricator via lldb-commits
mgorny created this revision. mgorny added a project: LLDB. Add explicit linkage to the necessary system libraries in the Host library. Otherwise, the library fails to build with -Wl,--as-needed. The system libraries ended up being listed on the linker command-line before the static libraries need

[Lldb-commits] [PATCH] D36886: [unittests] Build LLVMTestingSupport for out-of-source builds

2017-08-18 Thread Michał Górny via Phabricator via lldb-commits
mgorny created this revision. mgorny added a project: LLDB. The Process/gdb-remote test now requires the LLVMTestingSupport library that is not installed by LLVM. As a result, when doing an out-of-source build it fails being unable to find the library. To solve that, build a local copy of the libr

[Lldb-commits] [lldb] r311207 - Commiting Christopher Brook's patch for

2017-08-18 Thread Jason Molenda via lldb-commits
Author: jmolenda Date: Fri Aug 18 15:57:59 2017 New Revision: 311207 URL: http://llvm.org/viewvc/llvm-project?rev=311207&view=rev Log: Commiting Christopher Brook's patch for "Prevent negative chars from being sign-extended into isprint and isspace which take and int and crash if the int is nega

[Lldb-commits] [PATCH] D36620: Fix crash on parsing gdb remote packets

2017-08-18 Thread Jason Molenda via Phabricator via lldb-commits
jasonmolenda closed this revision. jasonmolenda added a comment. Sendingsource/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp Transmitting file data .done Committing transaction... Committed revision 311207. https://reviews.llvm.org/D36620 __

[Lldb-commits] [PATCH] D36886: [unittests] Build LLVMTestingSupport for out-of-source builds

2017-08-18 Thread Zachary Turner via Phabricator via lldb-commits
zturner accepted this revision. zturner added a comment. This revision is now accepted and ready to land. Seems fine to me, beanz? Repository: rL LLVM https://reviews.llvm.org/D36886 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http:

[Lldb-commits] [PATCH] D36885: [cmake] Explicitly link dependency libraries in the Host library

2017-08-18 Thread Zachary Turner via Phabricator via lldb-commits
zturner added inline comments. Comment at: source/Host/CMakeLists.txt:166-168 + if (LIBXML2_FOUND) +list(APPEND EXTRA_LIBS ${LIBXML2_LIBRARIES}) + endif() Even if libxml is found, that doesn't mean the build is configured to use it, does it? Correct me if