[Lldb-commits] [PATCH] D45700: Improve LLDB's handling of non-local minidumps
Leonard Mosescu via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Apr 17 12:09:26 PDT 2018
lemo marked 2 inline comments as done.
lemo added inline comments.
Comment at: packages/Python/lldbsuite/test/functionalities/postmortem/minidump/TestMiniDump.py:53
+ for module in self.target.modules:
> lemo wrote:
> > labath wrote:
> > > Could we strengthen these assertions a bit. Given that this is static data you are loading, I don't see why you couldn't hard-code the names of the modules you should expect.
> > Done.
> > Just curious, do we have any support for golden output files? (it would be great for tests like this)
> If you're feeling adventurous, you can try rewriting this as a lit test. You should be able to just do a `lldb -c my-core -o "image list"` and FileCheck the output.
Thanks for the tip, I'll keep this in mind for the future.
More information about the lldb-commits