[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:56:15 PDT 2016


zturner added a comment.

Gah, is it better to use the aligned types, or to keep manually adding padding?  The latest update also breaks the definition of `SCBytes` because:

  uint16_t x;
  uint32_t y;

is different from

  ulittle16_t x;
  ulittle32_t y;

Should I just switch to using aligned types everywhere?


http://reviews.llvm.org/D19570





More information about the llvm-commits mailing list