[Lldb-commits] [PATCH] D18464: Implement `target modules dump headers`

Greg Clayton via lldb-commits lldb-commits at lists.llvm.org
Fri Apr 1 09:50:05 PDT 2016

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

I would limit this scope to dumping ObjectFiles since that seems to be what you want. So this command should probably change from:

  (lldb) target modules dump headers


  (lldb) target modules dump objfile

Then CommandObjectTargetModulesDump would just get the ObjectFile from any modules in the arguments and call ObjectFile::Dump(...) instead of module->Dump() since module->Dump() dumps the module itself, and the object file and the symbol file. We already have a "target modules dump symfile", so "target modules dump objfile" makes more sense here.


More information about the lldb-commits mailing list