[PATCH] D33323: [llvm-pdbdump] Add the ability to merge PDBs

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 18 11:22:39 PDT 2017


zturner added a comment.

What would be the advantage of having it be a separate tool?  I kind of like the fact that I don't have to remember which tool to use when I want something, which kind of solves the problem of tools with overlapping responsibilities.  For example, I can never remember whether to use `llvm-objdump` or `llvm-readobj`.  Seems to me they should be the same tool.  Like "if you want to do something to a PDB, use `llvm-pdbdump`".  That said, I do agree it does more than just dumping now, so perhaps a new name is in order soon.  I wouldn't want to mix a change like that in with other changes though.


https://reviews.llvm.org/D33323





More information about the llvm-commits mailing list