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<br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 1, 2017 at 1:57 PM Reid Kleckner via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rnk added a comment.<br>
<br>
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.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D33785" rel="noreferrer" target="_blank">https://reviews.llvm.org/D33785</a><br>
<br>
<br>
<br>
</blockquote></div>