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/Host/macosx | |
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/Host/macosx')
-rw-r--r-- | lldb/source/Host/macosx/Host.mm | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/lldb/source/Host/macosx/Host.mm b/lldb/source/Host/macosx/Host.mm index d87c6be915c..848cafacf82 100644 --- a/lldb/source/Host/macosx/Host.mm +++ b/lldb/source/Host/macosx/Host.mm @@ -57,6 +57,7 @@ #include "lldb/Host/ConnectionFileDescriptor.h" #include "lldb/Host/HostInfo.h" #include "lldb/Host/ThreadLauncher.h" +#include "lldb/Target/ProcessLaunchInfo.h" #include "lldb/Target/Process.h" #include "lldb/Utility/ArchSpec.h" #include "lldb/Utility/CleanUp.h" @@ -68,6 +69,7 @@ #include "lldb/Utility/NameMatches.h" #include "lldb/Utility/StreamString.h" #include "lldb/Utility/StructuredData.h" +#include "lldb/lldb-defines.h" #include "llvm/Support/FileSystem.h" #include "llvm/Support/Errno.h" @@ -1497,15 +1499,9 @@ Status Host::LaunchProcess(ProcessLaunchInfo &launch_info) { launch_info.SetProcessID(pid); // Make sure we reap any processes we spawn or we will have zombies. - if (!launch_info.MonitorProcess()) { - const bool monitor_signals = false; - Host::MonitorChildProcessCallback callback = nullptr; - - if (!launch_info.GetFlags().Test(lldb::eLaunchFlagDontSetExitStatus)) - callback = Process::SetProcessExitStatus; - - StartMonitoringChildProcess(callback, pid, monitor_signals); - } + bool monitoring = launch_info.MonitorProcess(); + UNUSED_IF_ASSERT_DISABLED(monitoring); + assert(monitoring); } else { // Invalid process ID, something didn't go well if (error.Success()) |