[PATCH] D20971: Dump MSF headers to YAML

[Zachary Turner via llvm-commits]

zturner added a comment.

I debated over whether it was worth it.  The reason for doing all this is to enable new tests, and in writing those new tests (once this is complete), we should find bugs in the yaml output format as well.

I don't mind adding one tests that just checks these 5 fields, but I don't think it should turn into something comprehensive like `pdbdump-headers.text` where we test every single possible output path, because it seems like a lot of work for little gain.


http://reviews.llvm.org/D20971