[all-commits] [llvm/llvm-project] 156cb4: [lldb] Remove non-stop mode code

Pavel Labath via All-commits all-commits at lists.llvm.org
Tue Sep 28 05:14:18 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 156cb4cc64bec2b72aad9848f8181b338ff19ebc
      https://github.com/llvm/llvm-project/commit/156cb4cc64bec2b72aad9848f8181b338ff19ebc
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2021-09-28 (Tue, 28 Sep 2021)

  Changed paths:
    M lldb/include/lldb/Target/Target.h
    M lldb/source/Commands/CommandObjectThread.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteClientBase.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteClientBase.h
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp
    M lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.h
    M lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp
    M lldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.h
    M lldb/source/Target/Target.cpp
    M lldb/source/Target/TargetProperties.td

  Log Message:
  -----------
  [lldb] Remove non-stop mode code

We added some support for this mode back in 2015, but the feature was
never productionized. It is completely untested, and there are known
major structural lldb issues that need to be resolved before this
feature can really be supported.

It also complicates making further changes to stop reply packet
handling, which is what I am about to do.

Differential Revision: https://reviews.llvm.org/D110553


  Commit: 9413ead7bcbaf5d8876ee484256df65c2846c5c9
      https://github.com/llvm/llvm-project/commit/9413ead7bcbaf5d8876ee484256df65c2846c5c9
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2021-09-28 (Tue, 28 Sep 2021)

  Changed paths:
    M lldb/packages/Python/lldbsuite/test/lldbtest.py
    M lldb/test/API/functionalities/load_after_attach/TestLoadAfterAttach.py
    M lldb/test/API/functionalities/load_after_attach/main.cpp

  Log Message:
  -----------
  [lldb/test] Add ability to specify environment when spawning processes

We only had that ability for regular debugger launches. This meant that
it was not possible to use the normal dlopen patterns in attach tests.
This fixes that.


Compare: https://github.com/llvm/llvm-project/compare/6359a4cdbfb8...9413ead7bcba


More information about the All-commits mailing list