[Lldb-commits] [PATCH] D80130: [mlir][SystemZ] Fix incompatible datalayout in SystemZ

2020-05-19 Thread Mehdi AMINI via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9f2ce5b915a5: [mlir][SystemZ] Fix incompatible datalayout in SystemZ (authored by imaihal, committed by mehdi_amini). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[Lldb-commits] [PATCH] D80150: [lldb/DataFormatter] Check for overflow when finding NSDate epoch

2020-05-19 Thread Muhammad Omair Javaid via Phabricator via lldb-commits
omjavaid reopened this revision. omjavaid added a comment. This revision is now accepted and ready to land. This patch breaks lldb unit tests on lldb-arm-ubuntu buildbot. http://lab.llvm.org:8014/builders/lldb-arm-ubuntu/builds/1697 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[Lldb-commits] [PATCH] D80130: [mlir][SystemZ] Fix incompatible datalayout in SystemZ

2020-05-19 Thread Haruki Imai via Phabricator via lldb-commits
imaihal added a comment. @ftynse and @mehdi_amini, Could you help to commit this if this patch ready for commit? I don't have write access to the repository. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80130/new/

[Lldb-commits] [PATCH] D80130: [mlir][SystemZ] Fix incompatible datalayout in SystemZ

2020-05-19 Thread Haruki Imai via Phabricator via lldb-commits
imaihal marked 2 inline comments as done. imaihal added inline comments. Comment at: mlir/lib/ExecutionEngine/ExecutionEngine.cpp:123 } - std::unique_ptr machine( - target->createTargetMachine(targetTriple, "generic", "", {}, {})); + std::string cpu =

[Lldb-commits] [PATCH] D80130: [mlir][SystemZ] Fix incompatible datalayout in SystemZ

2020-05-19 Thread Haruki Imai via Phabricator via lldb-commits
imaihal updated this revision to Diff 265107. imaihal added a comment. Changed getHostCPUName() part to reflect the comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80130/new/ https://reviews.llvm.org/D80130 Files:

[Lldb-commits] [PATCH] D79726: Add terminateCommands to lldb-vscode protocol

2020-05-19 Thread António Afonso via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rGa3609b0ec685: Add terminateCommands to lldb-vscode protocol (authored by aadsm, committed by António Afonso aa...@fb.com). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[Lldb-commits] [PATCH] D80254: Prevent GetNumChildren from transitively walking pointer chains

2020-05-19 Thread Shafik Yaghmour via Phabricator via lldb-commits
shafik added inline comments. Comment at: lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp:5215 +uint32_t num_pointee_children = 0; +if (pointee_clang_type.IsAggregateType()) + num_pointee_children = I am curious what cases are pointers

[Lldb-commits] [PATCH] D80257: [lldb] Allows customizing libxml2 for darwin

2020-05-19 Thread Haibo Huang via Phabricator via lldb-commits
hhb updated this revision to Diff 265082. hhb added a comment. rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80257/new/ https://reviews.llvm.org/D80257 Files: lldb/cmake/modules/LLDBConfig.cmake lldb/source/Host/CMakeLists.txt

[Lldb-commits] [PATCH] D80257: [lldb] Allows customizing libxml2 for darwin

2020-05-19 Thread Haibo Huang via Phabricator via lldb-commits
hhb updated this revision to Diff 265080. hhb added a comment. Remove endif() Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80257/new/ https://reviews.llvm.org/D80257 Files: lldb/cmake/modules/LLDBConfig.cmake lldb/source/Host/CMakeLists.txt

[Lldb-commits] [PATCH] D80150: [lldb/DataFormatter] Check for overflow when finding NSDate epoch

2020-05-19 Thread Vedant Kumar via Phabricator via lldb-commits
vsk marked 3 inline comments as done. vsk added a comment. @labath Agreed on all points, I've addressed the feedback in 82dbf4aca84 by moving "DataFormatters/Mock.h" to "Plugins/Language/ObjC/Utilities.h", and adding a

[Lldb-commits] [PATCH] D80257: [lldb] Allows customizing libxml2 for darwin

2020-05-19 Thread Haibo Huang via Phabricator via lldb-commits
hhb created this revision. Herald added subscribers: lldb-commits, mgorny. Herald added a project: LLDB. hhb updated this revision to Diff 265080. hhb added a comment. hhb edited the summary of this revision. hhb updated this revision to Diff 265082. Remove endif() hhb added a comment. rebase

[Lldb-commits] [lldb] a3609b0 - Add terminateCommands to lldb-vscode protocol

2020-05-19 Thread António Afonso via lldb-commits
Author: António Afonso Date: 2020-05-19T16:18:05-07:00 New Revision: a3609b0ec68522cb417ffe36ce9eb2e25ca61578 URL: https://github.com/llvm/llvm-project/commit/a3609b0ec68522cb417ffe36ce9eb2e25ca61578 DIFF:

[Lldb-commits] [lldb] 82dbf4a - [lldb/test] Move "DataFormatters/Mock.h" to "Plugins/Language/ObjC/Utilities.h"

2020-05-19 Thread Vedant Kumar via lldb-commits
Author: Vedant Kumar Date: 2020-05-19T16:09:42-07:00 New Revision: 82dbf4aca84ec889d0dc390674ff44e30441bcfd URL: https://github.com/llvm/llvm-project/commit/82dbf4aca84ec889d0dc390674ff44e30441bcfd DIFF: https://github.com/llvm/llvm-project/commit/82dbf4aca84ec889d0dc390674ff44e30441bcfd.diff

[Lldb-commits] [PATCH] D80254: Prevent GetNumChildren from transitively walking pointer chains

2020-05-19 Thread Jaroslav Sevcik via Phabricator via lldb-commits
jarin created this revision. jarin added reviewers: teemperor, jingham. jarin added a project: LLDB. Herald added a subscriber: lldb-commits. This is an attempt to fix https://bugs.llvm.org/show_bug.cgi?id=45988, where SBValue::GetNumChildren returns 2, but SBValue::GetChildAtIndex(1) returns an

[Lldb-commits] [PATCH] D80253: [lldb] Cleans up system_libs

2020-05-19 Thread Haibo Huang via Phabricator via lldb-commits
hhb created this revision. Herald added subscribers: lldb-commits, mgorny. Herald added a project: LLDB. Long long ago system_libs was appended to LLDB_SYSTEM_LIBS in cmake/LLDBDependencies.cmake. After that file was removed, system_libs is orphaned. Currently the only user is source/Utility.

[Lldb-commits] [PATCH] D79308: [lldb-server] Reset stop reason of all threads when resuming

2020-05-19 Thread Jaroslav Sevcik via Phabricator via lldb-commits
jarin added a comment. Pavel, could you possibly land this for us? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D79308/new/ https://reviews.llvm.org/D79308 ___ lldb-commits mailing list lldb-commits@lists.llvm.org

[Lldb-commits] [lldb] c7dddaa - [lldb/Reproducers] Update GDB remote client tests for passive replay

2020-05-19 Thread Jonas Devlieghere via lldb-commits
Author: Jonas Devlieghere Date: 2020-05-19T13:07:50-07:00 New Revision: c7dddaa89ff17d2b191bd70de5a01712590b317d URL: https://github.com/llvm/llvm-project/commit/c7dddaa89ff17d2b191bd70de5a01712590b317d DIFF:

[Lldb-commits] [PATCH] D80226: [lldb/Driver] Error out when encountering unknown arguments

2020-05-19 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: labath, jingham, clayborg. There appears to be consensus in D80165 that this is the desired behavior and I personally agree. Repository: rLLDB LLDB https://reviews.llvm.org/D80226 Files:

[Lldb-commits] [PATCH] D68003: [dotest] Support specifying a version for skipIfOutOfTreeDebugserver

2020-05-19 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere abandoned this revision. JDevlieghere added a comment. Abandoning this for the reason mentioned above. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68003/new/ https://reviews.llvm.org/D68003 ___

[Lldb-commits] [PATCH] D80159: [lldb/Properties] Move OSPluginReportsAllThreads from Target to Process

2020-05-19 Thread Jonas Devlieghere via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG018e5a96eebd: [lldb/Properties] Move OSPluginReportsAllThreads from Target to Process (authored by JDevlieghere). Herald added a project: LLDB. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[Lldb-commits] [PATCH] D80159: [lldb/Properties] Move OSPluginReportsAllThreads from Target to Process

2020-05-19 Thread Jim Ingham via Phabricator via lldb-commits
jingham accepted this revision. jingham added a comment. This revision is now accepted and ready to land. Excellent! Thanks for doing this. Repository: rLLDB LLDB CHANGES SINCE LAST ACTION https://reviews.llvm.org/D80159/new/ https://reviews.llvm.org/D80159

[Lldb-commits] [lldb] 018e5a9 - [lldb/Properties] Move OSPluginReportsAllThreads from Target to Process

2020-05-19 Thread Jonas Devlieghere via lldb-commits
Author: Jonas Devlieghere Date: 2020-05-19T11:26:39-07:00 New Revision: 018e5a96eebd712dd743589327354d528865f9b6 URL: https://github.com/llvm/llvm-project/commit/018e5a96eebd712dd743589327354d528865f9b6 DIFF:

[Lldb-commits] [lldb] 15ee8a3 - Silence warnings around int/float conversions.

2020-05-19 Thread Eric Christopher via lldb-commits
Author: Eric Christopher Date: 2020-05-19T10:56:18-07:00 New Revision: 15ee8a3a58223b48afbe33cb60084f864ef20889 URL: https://github.com/llvm/llvm-project/commit/15ee8a3a58223b48afbe33cb60084f864ef20889 DIFF:

[Lldb-commits] [PATCH] D80224: [lldb/Reproducers] Support reproducers for PlatformRemoteGDBServer

2020-05-19 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere created this revision. JDevlieghere added reviewers: labath, teemperor. Add reproducer support to PlatformRemoteGDBServer. The logic is essentially the same as for ProcessGDBRemote. During capture we record the GDB packets and during replay we connect to a replay server. This

[Lldb-commits] [PATCH] D80165: [lldb/Driver] Fix handling on positional arguments

2020-05-19 Thread Jim Ingham via Phabricator via lldb-commits
jingham added a comment. I agree with Pavel, reporting an error and exiting seems like a good behavior. When I mistype a command line option, I generally immediately quit lldb, up-arrow and fix and resubmit, so this would save keystrokes. We should make sure if we do exit that we don't

[Lldb-commits] [lldb] 225f241 - [lldb/Reproducers] Move connection logic into replay server (NFC)

2020-05-19 Thread Jonas Devlieghere via lldb-commits
Author: Jonas Devlieghere Date: 2020-05-19T10:55:35-07:00 New Revision: 225f241c84469d7d29c9ef626558c302134c48ff URL: https://github.com/llvm/llvm-project/commit/225f241c84469d7d29c9ef626558c302134c48ff DIFF:

[Lldb-commits] [PATCH] D80165: [lldb/Driver] Fix handling on positional arguments

2020-05-19 Thread Pavel Labath via Phabricator via lldb-commits
labath added inline comments. Comment at: lldb/test/Shell/Driver/TestPositionalArgs.test:27-28 + +UNKNOWN: warning: ignoring unknown option: -baz +UNKNOWN: warning: ignoring unknown option: --quux +UNKNOWN: Current executable set to {{.*}}foo JDevlieghere wrote:

[Lldb-commits] [PATCH] D80165: [lldb/Driver] Fix handling on positional arguments

2020-05-19 Thread Jonas Devlieghere via Phabricator via lldb-commits
JDevlieghere marked 2 inline comments as done. JDevlieghere added inline comments. Comment at: lldb/test/Shell/Driver/TestPositionalArgs.test:27-28 + +UNKNOWN: warning: ignoring unknown option: -baz +UNKNOWN: warning: ignoring unknown option: --quux +UNKNOWN: Current executable

[Lldb-commits] [PATCH] D80165: [lldb/Driver] Fix handling on positional arguments

2020-05-19 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. Thanks for fixing this. I remember running into this a couple of times, but I never got around to do something about it... Comment at: lldb/test/Shell/Driver/TestPositionalArgs.test:27-28 + +UNKNOWN: warning: ignoring unknown option: -baz +UNKNOWN:

[Lldb-commits] [PATCH] D80150: [lldb/DataFormatter] Check for overflow when finding NSDate epoch

2020-05-19 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. While I think the idea in this patch is good, the actual implementation seems problematic to me, on several levels. The name `Mock.h` is overly generic, and inappropriate. According to wikipedia , mock (object)s are "simulated

[Lldb-commits] [PATCH] D77043: Fix process gdb-remote usage of value_regs/invalidate_regs

2020-05-19 Thread Muhammad Omair Javaid via Phabricator via lldb-commits
omjavaid added a comment. In D77043#2041844 , @labath wrote: > Sorry, about the delay. I had to find some time to dig into the code again, > to understand all of the interactions. > > It seems that we (lldb) have painted ourselves in the corner somewhat.

[Lldb-commits] [lldb] 2bb16f9 - [lldb/Test] Skip TestPositionalArgs with lldb-repro

2020-05-19 Thread Jonas Devlieghere via lldb-commits
Author: Jonas Devlieghere Date: 2020-05-18T22:59:55-07:00 New Revision: 2bb16f94128ecd4cfcbbb8fe2db57490043a805f URL: https://github.com/llvm/llvm-project/commit/2bb16f94128ecd4cfcbbb8fe2db57490043a805f DIFF: