diff options
author | Pavel Labath <pavel@labath.sk> | 2025-06-26 13:51:14 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-06-26 13:51:14 +0200 |
commit | b77114b723eb68563a0900846df5bd1b454edc2f (patch) | |
tree | 36ed38d9b16c9b517d056562d1d1780f233ed4dd /lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp | |
parent | f64d5df3c2524d92b3257ac50cfd941393159f98 (diff) | |
download | llvm-b77114b723eb68563a0900846df5bd1b454edc2f.zip llvm-b77114b723eb68563a0900846df5bd1b454edc2f.tar.gz llvm-b77114b723eb68563a0900846df5bd1b454edc2f.tar.bz2 |
[lldb] Remove child_process_inherit argument from Pipe (#145516)
It's not necessary on posix platforms as of #126935 and it's ignored on
windows as of #138896. For both platforms, we have a better way of
inheriting FDs/HANDLEs.
Diffstat (limited to 'lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp')
-rw-r--r-- | lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp index 2ca7099..4bb85f5 100644 --- a/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp +++ b/lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunication.cpp @@ -944,7 +944,7 @@ Status GDBRemoteCommunication::StartDebugserverProcess( #if defined(__APPLE__) // Using a named pipe as debugserver does not support --pipe. Status error = socket_pipe.CreateWithUniqueName("debugserver-named-pipe", - false, named_pipe_path); + named_pipe_path); if (error.Fail()) { LLDB_LOG(log, "named pipe creation failed: {0}", error); return error; @@ -953,7 +953,7 @@ Status GDBRemoteCommunication::StartDebugserverProcess( debugserver_args.AppendArgument(named_pipe_path); #else // Using an unnamed pipe as it's simpler. - Status error = socket_pipe.CreateNew(true); + Status error = socket_pipe.CreateNew(); if (error.Fail()) { LLDB_LOG(log, "unnamed pipe creation failed: {0}", error); return error; @@ -961,7 +961,7 @@ Status GDBRemoteCommunication::StartDebugserverProcess( pipe_t write = socket_pipe.GetWritePipe(); debugserver_args.AppendArgument(llvm::StringRef("--pipe")); debugserver_args.AppendArgument(llvm::to_string(write)); - launch_info.AppendCloseFileAction(socket_pipe.GetReadFileDescriptor()); + launch_info.AppendDuplicateFileAction((int64_t)write, (int64_t)write); #endif } @@ -1044,7 +1044,7 @@ Status GDBRemoteCommunication::StartDebugserverProcess( Status error; if (named_pipe_path.size() > 0) { - error = socket_pipe.OpenAsReader(named_pipe_path, false); + error = socket_pipe.OpenAsReader(named_pipe_path); if (error.Fail()) { LLDB_LOG(log, "failed to open named pipe {0} for reading: {1}", named_pipe_path, error); |