[Lldb-commits] [PATCH] D68096: ProcessMinidump: inject SIGSTOP on Linux if no thread has a signal

Joseph Tremoulet via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Sep 27 12:09:37 PDT 2019

JosephTremoulet marked 2 inline comments as done.
JosephTremoulet added inline comments.

Comment at: lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp:219-237
+  if (arch.GetTriple().isOSLinux()) {
+    SetUnixSignals(UnixSignals::Create(GetArchitecture()));
+    if (!m_thread_list.empty() &&
+        (!m_active_exception ||
JosephTremoulet wrote:
> labath wrote:
> > Would it be possible to move this code (except maybe the SetUnixSignals bit) into the `RefreshStateAfterStop` function? Would be less confusing and it would avoid the need for extra member variables...
> Yeah, happy to.  I wasn't sure what the implications of doing so would be, but I guess for a dump file there will only ever be one stop?
I've moved it.  In doing so I left the early-out for `!m_active_exception`, which is a behavioral difference from my previous iteration, but this covers my use-case so I think better to keep the change focused like this.

  rG LLVM Github Monorepo



More information about the lldb-commits mailing list