[PATCH] D28707: PDB: Add a class to create the /names stream contents.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 13 15:49:54 PST 2017


ruiu added a comment.

I will eventually do that, but that's not doable at the moment. In order to test this output with YAM, we need to plug this in to other parts so that PDB files get this string table. Currently doing that doesn't make much sense because no one is using the string table, and we have no code for that.

Also I think this unit test is useful by itself. This is compact and does only what it should do.


https://reviews.llvm.org/D28707





More information about the llvm-commits mailing list