[Lldb-commits] [PATCH] D126259: Add the ability to specify signal actions (pass, stop or notify) for a signal before a process is created

[Jim Ingham via Phabricator via lldb-commits]

jingham marked 3 inline comments as done.
jingham added a comment.

In D126259#3534729 <https://reviews.llvm.org/D126259#3534729>, @clayborg wrote:

> So the "process handle" command allows us to set signals by signal number as well. Does this patch support this? It seems like it wouldn't be too hard to do if we wanted to handle this. Lemme know what you think, other than that LGTM.

Ah, I forgot about specifying the signal by number.  Before you have a process I don't think we should allow signals by number.  The mapping signal number -> "signal name for any platform" is not 1-1 so we couldn't guarantee we were doing the right thing here.  I'll put in a check for "specified by number with no process" and error out.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D126259/new/

https://reviews.llvm.org/D126259