[PATCH] D26552: [PDB] Add documentation for the PDB DBI Stream.

[Zachary Turner via llvm-commits]

zturner added inline comments.


================
Comment at: docs/PDB/DbiStream.rst:87
+  uint16_t MajorVersion : 7;
+  uint16_t NewVersionFormat : 1;
+
----------------
amccarth wrote:
> Bit-field syntax is ambiguous because bit-fields can be allocated left-to-right or right-to-left (implementation defined).  There are also implementation-defined padding and alignment issues.
While true, I need a concise way to express this since there are multiple bitfields defined throughout this file and in other files (some of which have not yet been written).  If it was easy to insert images into RestructuredText, I could just draw it out, but I was hoping this could be understood to mean "the first 8 bits are X, the next 7 bits are Y, the final bit is Z", etc.


https://reviews.llvm.org/D26552