[PATCH] D33996: Improve consistency and flexibility with llvm-pdbdump subcommands

Zachary Turner via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 8 10:34:59 PDT 2017


zturner added a comment.

I don't mind submitting it as a followup, but I think it's a bad idea to submit it at the same time.  While you're right that it only breaks the interface once to do it together, I don't think anyone is really depending on the interface that much, and changing the name is already going to be a rather huge change that is like 100% boilerplate



================
Comment at: llvm/test/DebugInfo/PDB/Inputs/debug-subsections.yaml:1
----
-DbiStream:
----------------
inglorion wrote:
> Why does this change?
It shouldn't, that looks like a copy paste error.


https://reviews.llvm.org/D33996





More information about the llvm-commits mailing list