[PATCH] D26374: Add documentation for the PDB file format

[David Majnemer via llvm-commits]

majnemer added inline comments.


================
Comment at: docs/PDB/MsfFile.rst:24
+    ulittle32_t Unknown;
+    ulittle32_t BlockMapAddr;
+  };
----------------
zturner wrote:
> majnemer wrote:
> > Looking at the microsoft-pdb sources, isn't this technically an array of addresses?
> > https://github.com/Microsoft/microsoft-pdb/blob/master/PDB/msf/msf.cpp#L954
> It's more like a pointer to an array.  It's a single uint32 which is the index of a block, and at that block is an array.
My reading of the PDB sources makes me thing that its an array of pointers to arrays. I don't think we ever encountered this in the wild because our PDBs were not large enough.


https://reviews.llvm.org/D26374