diff options
| author | Pavel Labath <labath@google.com> | 2018-05-15 13:42:26 +0000 |
|---|---|---|
| committer | Pavel Labath <labath@google.com> | 2018-05-15 13:42:26 +0000 |
| commit | 245dd2e0c76648b8f875a2f45003ffd8614eb6d6 (patch) | |
| tree | be88759a81c898b69e633104c714d6ded212540f /lldb/source/Plugins/Platform | |
| parent | 349506a926c78af36cdcac75d44ac7522f168d06 (diff) | |
| download | bcm5719-llvm-245dd2e0c76648b8f875a2f45003ffd8614eb6d6.tar.gz bcm5719-llvm-245dd2e0c76648b8f875a2f45003ffd8614eb6d6.zip | |
Reapply "Remove Process references from the Host module"
This re-lands r332250/D46395, after fixing Mac build errors.
llvm-svn: 332353
Diffstat (limited to 'lldb/source/Plugins/Platform')
| -rw-r--r-- | lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp b/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp index 9b2c86a5f68..eeff8f3518a 100644 --- a/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp +++ b/lldb/source/Plugins/Platform/POSIX/PlatformPOSIX.cpp @@ -868,11 +868,12 @@ PlatformPOSIX::DebugProcess(ProcessLaunchInfo &launch_info, Debugger &debugger, if (IsHost()) { // We are going to hand this process off to debugserver which will be in - // charge of setting the exit status. We still need to reap it from lldb - // but if we let the monitor thread also set the exit status, we set up a - // race between debugserver & us for who will find out about the debugged - // process's death. - launch_info.GetFlags().Set(eLaunchFlagDontSetExitStatus); + // charge of setting the exit status. However, we still need to reap it + // from lldb. So, make sure we use a exit callback which does not set exit + // status. + const bool monitor_signals = false; + launch_info.SetMonitorProcessCallback( + &ProcessLaunchInfo::NoOpMonitorCallback, monitor_signals); process_sp = Platform::DebugProcess(launch_info, debugger, target, error); } else { if (m_remote_platform_sp) |

