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

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 7 16:57:50 PST 2016


zturner added inline comments.


================
Comment at: docs/PDB/MsfFile.rst:55
+  be stitched together accordingly.  The number of ``ulittle32_t``'s in this array
+  is given by ``⌈NumDirectoryBytes / BlockSize⌉``.
+  
----------------
amccarth wrote:
> Did you mean ``[NumDirectoryBytes / 4]``?
No, this is correct.  For starters, that's intentionally a mathematical ceiling operator and not a square bracket :)  That aside though, Suppose your directory is magically exactly 4KiB and your block size is also 4KiB.  Then your directory fits on a single block, and thus you only need a single `ulittle32` to describe the list of blocks that it lies on.


https://reviews.llvm.org/D26374





More information about the llvm-commits mailing list