kastiglione created this revision.
kastiglione added a reviewer: jingham.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
Delete unused `EnableTracer()` and `SetTracer()` functions on `Thread`. By
deleting
these, their `ThreadP
omjavaid added a comment.
This patch looks quite neat overall I have a few minor questions, comments and
suggestions.
1. To get the review going consider further splitting up these patches into
separate chunks containing, for example MemoryTagHandler API and related unit
tests, GDB remote addo
Author: Michał Górny
Date: 2021-02-14T22:34:25+01:00
New Revision: 011791dda43c55cd9d5e53b6119b0cf40c183c48
URL:
https://github.com/llvm/llvm-project/commit/011791dda43c55cd9d5e53b6119b0cf40c183c48
DIFF:
https://github.com/llvm/llvm-project/commit/011791dda43c55cd9d5e53b6119b0cf40c183c48.diff
emaste added a comment.
Not sure if you're missing something, but removing the old plugin LGTM
Comment at: lldb/tools/lldb-server/CMakeLists.txt:13
if(CMAKE_SYSTEM_NAME MATCHES "FreeBSD")
- list(APPEND LLDB_PLUGINS
-lldbPluginProcessFreeBSDRemote
-lldbPluginProcessFre
emaste added inline comments.
Comment at: lldb/tools/lldb-server/CMakeLists.txt:13
if(CMAKE_SYSTEM_NAME MATCHES "FreeBSD")
- list(APPEND LLDB_PLUGINS
-lldbPluginProcessFreeBSDRemote
-lldbPluginProcessFreeBSD)
+ list(APPEND LLDB_PLUGINS lldbPluginProcessFreeBSDRemote)
aadsm created this revision.
aadsm added reviewers: clayborg, labath, wallace, kusmour.
aadsm requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
VSCode was not being informed whenever a location had been resolved (after
being initated as non-
kastiglione created this revision.
kastiglione added a reviewer: jingham.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
`GetRealStopInfo` has only one call site, and in that call site a reference to
the
concrete thread plan i
kastiglione created this revision.
kastiglione added a reviewer: jingham.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
Correct `RestoreThreadState` to a `void` return type. Also, update the
signature of its
callee, `Thread::
kastiglione created this revision.
kastiglione added a reviewer: jingham.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
Remove `ShouldAutoContinue`, implemented in only one subclass.
In the `ShouldAutoContinue::ShouldAutoCont