This revision was automatically updated to reflect the committed changes. Closed by commit rGe4d6ed58a86e: [lldb] [llgs] Include process id in W/X stop reasons (authored by mgorny). Herald added a project: LLDB.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127191/new/ https://reviews.llvm.org/D127191 Files: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp lldb/test/API/tools/lldb-server/TestGdbRemoteFork.py Index: lldb/test/API/tools/lldb-server/TestGdbRemoteFork.py =================================================================== --- lldb/test/API/tools/lldb-server/TestGdbRemoteFork.py +++ lldb/test/API/tools/lldb-server/TestGdbRemoteFork.py @@ -74,7 +74,7 @@ # resume the parent self.test_sequence.add_log_lines([ "read packet: $c#00", - {"direction": "send", "regex": r"[$]W00#.*"}, + {"direction": "send", "regex": r"[$]W00;process:[0-9a-f]+#.*"}, ], True) self.expect_gdbremote_sequence() @@ -87,7 +87,7 @@ "read packet: $c#00", {"direction": "send", "regex": r"[$]T.*vforkdone.*"}, "read packet: $c#00", - {"direction": "send", "regex": r"[$]W00#.*"}, + {"direction": "send", "regex": r"[$]W00;process:[0-9a-f]+#.*"}, ], True) self.expect_gdbremote_sequence() @@ -135,7 +135,7 @@ # resume the child self.test_sequence.add_log_lines([ "read packet: $c#00", - {"direction": "send", "regex": r"[$]W00#.*"}, + {"direction": "send", "regex": r"[$]W00;process:[0-9a-f]+#.*"}, ], True) self.expect_gdbremote_sequence() Index: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp =================================================================== --- lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp +++ lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp @@ -474,6 +474,8 @@ StreamGDBRemote response; response.Format("{0:g}", *wait_status); + if (bool(m_extensions_supported & NativeProcessProtocol::Extension::multiprocess)) + response.Format(";process:{0:x-}", process->GetID()); return SendPacketNoLock(response.GetString()); }
Index: lldb/test/API/tools/lldb-server/TestGdbRemoteFork.py =================================================================== --- lldb/test/API/tools/lldb-server/TestGdbRemoteFork.py +++ lldb/test/API/tools/lldb-server/TestGdbRemoteFork.py @@ -74,7 +74,7 @@ # resume the parent self.test_sequence.add_log_lines([ "read packet: $c#00", - {"direction": "send", "regex": r"[$]W00#.*"}, + {"direction": "send", "regex": r"[$]W00;process:[0-9a-f]+#.*"}, ], True) self.expect_gdbremote_sequence() @@ -87,7 +87,7 @@ "read packet: $c#00", {"direction": "send", "regex": r"[$]T.*vforkdone.*"}, "read packet: $c#00", - {"direction": "send", "regex": r"[$]W00#.*"}, + {"direction": "send", "regex": r"[$]W00;process:[0-9a-f]+#.*"}, ], True) self.expect_gdbremote_sequence() @@ -135,7 +135,7 @@ # resume the child self.test_sequence.add_log_lines([ "read packet: $c#00", - {"direction": "send", "regex": r"[$]W00#.*"}, + {"direction": "send", "regex": r"[$]W00;process:[0-9a-f]+#.*"}, ], True) self.expect_gdbremote_sequence() Index: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp =================================================================== --- lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp +++ lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationServerLLGS.cpp @@ -474,6 +474,8 @@ StreamGDBRemote response; response.Format("{0:g}", *wait_status); + if (bool(m_extensions_supported & NativeProcessProtocol::Extension::multiprocess)) + response.Format(";process:{0:x-}", process->GetID()); return SendPacketNoLock(response.GetString()); }
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits