[Lldb-commits] [PATCH] D12888: Add first tests for mini-dump debugging.
Adrian McCarthy via lldb-commits
lldb-commits at lists.llvm.org
Wed Sep 16 10:32:21 PDT 2015
amccarth marked 2 inline comments as done.
amccarth added a comment.
> Instead of test/functionalities/minidump, I would probably call this test/functionalities/core-file/windows.
There is talk of (eventually) debugging minidumps on other platforms. Also some crash capture tools can make minidumps, even on Mac and Linux hosts. I could imagine you might also want to be able to debug a core file from a non-Windows host on a Windows host. So core-file/windows could end up being confusing/misleading in the long run.
Comment at: test/functionalities/minidump/TestMiniDump.py:29
@@ +28,3 @@
+ stop_description = thread.GetStopDescription(256);
+ self.assertTrue("0xc0000005" in stop_description);
> Do we not have a way to get the exception code?
It looks like you can, because the API has methods for getting extra values associated with the stop reason (SBThread::GetStopReasonDataCount and SBThread::GetStopReasonDataAtIndex), but no data about exceptions is not plumbed through--the process plugins just generate a string description and attach it to the stop reason. I spent quite a bit of time discovering this.
I would like to make the process plugins provide the information in a way that makes it available through those APIs as a separate patch.
More information about the lldb-commits