
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Jul 28, 2014, at 12:58 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com">chandlerc@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jul 28, 2014 at 12:46 PM, Quentin Colombet <span dir="ltr"><<a href="mailto:qcolombet@apple.com" target="_blank">qcolombet@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto;"><div id=":4nm" class="a3s" style="overflow:hidden">I think we still have a bug in the current implementation.<br>

See my inline comments.<br></div></blockquote><div><br></div><div>Seems likely. Can you craft a test case to demonstrate it?</div></div></div></div></blockquote><div><br></div><div>I will see if I can come up with something.</div><br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto;">

<div id=":4nm" class="a3s" style="overflow:hidden">

<br>

Also, I think it would be nice to update the comments on the masks value instead of removing them.<br>

When I reversed engineered this code, it took me ages to figure out what were these magic values.</div></blockquote></div><br>I'm not sure that's going to help. The problem with comments is that they can get out of sync. I also don't think that little endian offsets are surprising or need lots of clarification here? The 1 instead of 0 thing you point out screams wrong to me at a glance, and I don't think a comment would really make it better (or worse, unless it got out of date).</div>

</div>

</blockquote></div><br><div>Fair enough!</div><div><br></div><div>-Quentin</div></body></html>