[Lldb-commits] [PATCH] D128504: debugserver: spawn process in its own process group

aarzilli added a comment. > Do you have commit access to the repository? I do not. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128504/new/ https://reviews.llvm.org/D128504 ___ lldb-commits mailing lis

[Lldb-commits] [PATCH] D128504: debugserver: spawn process in its own process group

aarzilli added a comment. Ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128504/new/ https://reviews.llvm.org/D128504 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/

[Lldb-commits] [PATCH] D128504: debugserver: spawn process in its own process group

aarzilli added a comment. Ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128504/new/ https://reviews.llvm.org/D128504 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/

[Lldb-commits] [PATCH] D128504: debugserver: spawn process in its own process group

aarzilli added a comment. Ping? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128504/new/ https://reviews.llvm.org/D128504 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org

[Lldb-commits] [PATCH] D128504: debugserver: spawn process in its own process group

aarzilli added a comment. Ping? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128504/new/ https://reviews.llvm.org/D128504 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org

[Lldb-commits] [PATCH] D128504: debugserver: spawn process in its own process group

aarzilli added a comment. In D128504#3608530 , @jingham wrote: > Why is it desirable to have the debugger not see signals sent to the process? > > Jim Regardless of this patch, it is always possible to send signals directly to the target process. This i

[Lldb-commits] [PATCH] D128504: debugserver: spawn process in its own process group

aarzilli created this revision. aarzilli added reviewers: jasonmolenda, clayborg. aarzilli added a project: LLDB. Herald added a subscriber: JDevlieghere. Herald added a project: All. aarzilli requested review of this revision. Herald added a subscriber: lldb-commits. Change POSIX spawn launch fla

[Lldb-commits] [PATCH] D89315: [debugserver] Add option to propagate SIGSEGV to target process

aarzilli added a comment. In D89315#2400244 , @JDevlieghere wrote: > In D89315#2399805 , @aarzilli wrote: > >> I don't want to be overbearing but should I be doing something else? Is this >> going to be merged and

[Lldb-commits] [PATCH] D89315: [debugserver] Add option to propagate SIGSEGV to target process

aarzilli added a comment. I don't want to be overbearing but should I be doing something else? Is this going to be merged and, if it is, what timeline should I expect? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89315/new/ https://reviews.llvm.org/D89315

[Lldb-commits] [PATCH] D89315: [debugserver] Add option to propagate SIGSEGV to target process

aarzilli updated this revision to Diff 298319. aarzilli added a comment. Like this? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89315/new/ https://reviews.llvm.org/D89315 Files: lldb/tools/debugserver/source/DNB.cpp lldb/tools/debugserver/source/DNB.h lldb/tools/debugserver/sou

[Lldb-commits] [PATCH] D89315: [debugserver] Add option to propagate SIGSEGV to target process

aarzilli updated this revision to Diff 298077. aarzilli added a comment. > I wonder if the implementation would be less intrusive (and keep some of > these functions from adding to their already too many arguments) if you added > this flag to the RNBContext that RNBRemote holds onto. Done. > Y

[Lldb-commits] [PATCH] D89315: [debugserver] Add option to propagate SIGSEGV to target process

aarzilli created this revision. aarzilli added reviewers: clayborg, jasonmolenda. Herald added a project: LLDB. Herald added a subscriber: lldb-commits. aarzilli requested review of this revision. Herald added a subscriber: JDevlieghere. Adds a command line option that makes debugserver propagate