[PATCH] D20971: Dump MSF headers to YAML

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 3 11:09:08 PDT 2016

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.


More information about the llvm-commits mailing list