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

Zachary Turner via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 1 14:00:30 PDT 2017


Right, I'm just saying this patch shouldn't affect the records we write (I
haven't actually checked whether we write aligned or unaligned). If it does
turn out we're writing unaligned though, we should try to fix that in a
followup
On Thu, Jun 1, 2017 at 1:57 PM Reid Kleckner via Phabricator <
reviews at reviews.llvm.org> wrote:

> rnk 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. Having the
> flexibility to do either seems good. We definitely have to read the
> unaligned records.
>
>
> https://reviews.llvm.org/D33785
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170601/34f084c3/attachment.html>


More information about the llvm-commits mailing list