[PATCH] D33785: [CodeView] Fix alignment / padding when writing symbol records

[Bob Haarman via Phabricator via llvm-commits]

inglorion added a comment.

> I guess I'm saying it seems nice to always write aligned records until we come up with some reason to not write aligned records.

@zturner and I had talked about that a little bit before. I mentioned that unaligned records may save some space. They may be slightly less efficient to process, but that could be a good trade-off in object files. I'm not sure how large these records usually are or what the effect on size or speed would be if we always aligned or always packed or how much leeway we have in simplifying the code while being interoperable with other tools. For now, I prefer to focus on getting usable PDBs out of lld; we can perhaps revisit the alignment issue after that if it seems warranted.


https://reviews.llvm.org/D33785