[PATCH] D19824: [codeview] Isolate type dumping from object file state

[Zachary Turner via llvm-commits]

zturner added a comment.

In http://reviews.llvm.org/D19824#419096, @rnk wrote:

> In http://reviews.llvm.org/D19824#419069, @zturner wrote:
>
> > I wonder if `llvm-pdbdump` will want to use this eventually.  Although I don't have a good idea where to put it in common code so that it can be reused.
>
>
> Agreed. I think maybe it's best for llvm-pdbdump to keep testing the interface you have that abstracts over DIA.


It should certainly continue to test that, but I think it should also dump the raw properties of a file, such as the stuff I've been working on the past week.  Low level file structure type stuff.


http://reviews.llvm.org/D19824