[PATCH] D19570: [PDB] Parse module information from the DBI stream

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 26 18:46:49 PDT 2016


zturner updated this revision to Diff 55153.
zturner added a comment.

Fixed alignment issue by adding 2 bytes of padding.  When I was using a pointer it was getting 4-byte aligned, but when I changed to a ulittle32_t it was getting only 2-byte aligned, messing up the rest of the structure.  This should be correct now.  Alternatively I could have used an aligned_ulittle32_t, but the name is so long it made the comment formatting really ugly.


http://reviews.llvm.org/D19570

Files:
  include/llvm/DebugInfo/PDB/Raw/ModInfo.h
  include/llvm/DebugInfo/PDB/Raw/PDBDbiStream.h
  lib/DebugInfo/PDB/CMakeLists.txt
  lib/DebugInfo/PDB/Raw/ModInfo.cpp
  lib/DebugInfo/PDB/Raw/PDBDbiStream.cpp
  test/DebugInfo/PDB/pdbdump-headers.test
  tools/llvm-pdbdump/llvm-pdbdump.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19570.55153.patch
Type: text/x-patch
Size: 13098 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160427/2fc691bc/attachment.bin>


More information about the llvm-commits mailing list