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

[Bob Haarman via Phabricator via llvm-commits]

inglorion added a comment.

I would like to change the changes to these option names to coincide with renaming llvm-pdbdump to llvm-pdbtool (or some such) so that we only break the interface once.

While we're at it - how much value is there in having both yaml and raw that is almost the same, but with slightly different syntax? I've added some code to dump things in raw format before we understood what they meant, but I think now that we can dump pretty much everything as YAML, we should consider removing raw. Again, that breaks the API, so the introduction of llvm-pdbtool seems like a good time to do it.



================
Comment at: llvm/test/DebugInfo/PDB/Inputs/debug-subsections.yaml:1
----
-DbiStream:
----------------
Why does this change?


https://reviews.llvm.org/D33996