clayborg requested changes to this revision.
clayborg added a comment.
This revision now requires changes to proceed.

We need to grab the address from the exception info and pass that along for 
linux.



================
Comment at: lldb/source/Plugins/Process/minidump/ProcessMinidump.cpp:264
+
     stop_info = StopInfo::CreateStopReasonWithSignal(
+        *stop_thread, signo);
----------------
Are exceptions on linux always just a signal number? Mac can be a mach 
exception or a signal. Might be worth looking at the breakpad sources to see if 
they ever fill in anything else for linux other than a signal. 

Also, there is more info in the exception record that needs to be passed along. 
Like for a SIGSEGV, the address should be in the exception info somewhere.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D68096



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to