[Lldb-commits] [lldb] r314488 - Fix Android remote debugging tests running on Windows

2017-09-28 Thread Eugene Zemtsov via lldb-commits
Author: eugene Date: Thu Sep 28 20:25:25 2017 New Revision: 314488 URL: http://llvm.org/viewvc/llvm-project?rev=314488&view=rev Log: Fix Android remote debugging tests running on Windows Use make based OS check, instad of relying on shell. Modified: lldb/trunk/packages/Python/lldbsuite/test

[Lldb-commits] [lldb] r314487 - Fix compilation error

2017-09-28 Thread Eugene Zemtsov via lldb-commits
Author: eugene Date: Thu Sep 28 20:15:08 2017 New Revision: 314487 URL: http://llvm.org/viewvc/llvm-project?rev=314487&view=rev Log: Fix compilation error Modified: lldb/trunk/source/Plugins/ExpressionParser/Clang/ClangASTSource.cpp lldb/trunk/source/Symbol/ClangASTContext.cpp Modified:

[Lldb-commits] [lldb] r314458 - [Expression parser] Setting to enable use of ExternalASTMerger

2017-09-28 Thread Sean Callanan via lldb-commits
Author: spyffe Date: Thu Sep 28 13:20:25 2017 New Revision: 314458 URL: http://llvm.org/viewvc/llvm-project?rev=314458&view=rev Log: [Expression parser] Setting to enable use of ExternalASTMerger This setting can be enabled like this at the target level: (lldb) settings set target.experimental.u

[Lldb-commits] [lldb] r314455 - Add a few missing newlines in lldb-server messages

2017-09-28 Thread Stephane Sezer via lldb-commits
Author: sas Date: Thu Sep 28 12:49:00 2017 New Revision: 314455 URL: http://llvm.org/viewvc/llvm-project?rev=314455&view=rev Log: Add a few missing newlines in lldb-server messages Reviewers: fjricci, clayborg Subscribers: lldb-commits Differential Revision: https://reviews.llvm.org/D38373 Mod

[Lldb-commits] [PATCH] D38373: Add a few missing newlines in lldb-server messages

2017-09-28 Thread Stephane Sezer via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL314455: Add a few missing newlines in lldb-server messages (authored by sas). Repository: rL LLVM https://reviews.llvm.org/D38373 Files: lldb/trunk/tools/lldb-server/lldb-gdbserver.cpp Index: lldb

[Lldb-commits] [PATCH] D38373: Add a few missing newlines in lldb-server messages

2017-09-28 Thread Stephane Sezer via Phabricator via lldb-commits
sas created this revision. https://reviews.llvm.org/D38373 Files: tools/lldb-server/lldb-gdbserver.cpp Index: tools/lldb-server/lldb-gdbserver.cpp === --- tools/lldb-server/lldb-gdbserver.cpp +++ tools/lldb-server/lldb-gdbserver.

[Lldb-commits] [PATCH] D38323: Enable breakpoints and read/write GPRs for ppc64le

2017-09-28 Thread Greg Clayton via Phabricator via lldb-commits
clayborg added a comment. Looks fine. One main questions for new linux archs in particular: is linux using the lldb-server to debug these days even when debugging locally? If so, then this patch only needs to implement both a native register content and not the lldb_private::RegisterContext sub

[Lldb-commits] [PATCH] D38323: Enable breakpoints and read/write GPRs for ppc64le

2017-09-28 Thread Greg Clayton via Phabricator via lldb-commits
clayborg added inline comments. Comment at: source/Plugins/Process/Linux/NativeRegisterContextLinux_ppc64le.h:71 + RegInfo m_reg_info; + Reg m_gpr_ppc64le[48]; // 64-bit general purpose registers. + Could we share the structure that backs all registers from the

[Lldb-commits] [PATCH] D38328: Assume git-generated patch files in build scripts

2017-09-28 Thread Francis Ricci via Phabricator via lldb-commits
fjricci added a comment. Oh I see from the discussion on https://github.com/apple/swift-lldb/pull/252 that it was accidental - don't worry about it Repository: rL LLVM https://reviews.llvm.org/D38328 ___ lldb-commits mailing list lldb-commits@li

[Lldb-commits] [PATCH] D38328: Assume git-generated patch files in build scripts

2017-09-28 Thread Francis Ricci via Phabricator via lldb-commits
fjricci added a comment. @jasonmolenda - just an FYI that I'm an llvm committer (I contribute frequently to compiler-rt, and used to contribute quite a bit to lldb as well). No worries about this patch though, especially given how small it is. Repository: rL LLVM https://reviews.llvm.org/D3