[Lldb-commits] [PATCH] D12888: Add first tests for mini-dump debugging.

[Zachary Turner via lldb-commits]

zturner added a comment.

In http://reviews.llvm.org/D12888#247195, @amccarth wrote:

> > Instead of test/functionalities/minidump, I would probably call this test/functionalities/core-file/windows.
>
>
> Hmm...
>
> 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.
>
> How about
>
>   test/functionalities/crash_files/core-files
>
> and
>
>   test/functionalities/crash_files/mini-dumps
>
> ?


Are the "mini dumps" you're referring to that can be created on a Mac or Linux host the same file format?  I think we need some way to capture the distinction between the two file formats.  All of the different formats have one thing in common though.  They support *postmortem* debugging.  So maybe it could be:

test/functionalities/postmortem/elf

test/functionalities/postmortem/windows

I don't think having it be called "windows" precludes being able to use the functionality on non-windows hosts (provided we ever write a library that can process them on non-Windows), I just can't think of a better way to describe the format itself.


http://reviews.llvm.org/D12888