[PATCH] D19500: Parse and dump PDB DBI Stream Header Information

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 25 17:43:26 PDT 2016


zturner added inline comments.

================
Comment at: lib/DebugInfo/PDB/Raw/PDBDbiStream.cpp:68
@@ +67,3 @@
+PDBDbiStream::PDBDbiStream(PDBFile &File) : Pdb(File), Stream(3, File) {
+  static_assert(sizeof(HeaderInfo) == 64, "Invalid HeaderInfo size!");
+}
----------------
ruiu wrote:
> Can you write this outside of the constructor?
The class is forward declared as private in `PDBDbiStream.h`, so it's inaccessible outside the constructor.  I could declare it in the public section of the class, but it seemed more appropriate to be private.  I don't feel strongly either way, so let me know if you do.


http://reviews.llvm.org/D19500





More information about the llvm-commits mailing list