<div dir="ltr">On 18 November 2013 20:16, Yuchen Wu <span dir="ltr"><<a href="mailto:yuchenericwu@hotmail.com" target="_blank">yuchenericwu@hotmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<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">Hi all,<br>
<br>
I've updated the patch so that the file checksum is the crc32 of arc destinations. Please take a look!<br>
<br>
Function and line checksums coming soon.<br></blockquote><div><br></div><div>Thanks!</div><div><br></div><div><div>+    void setCfgChecksum(uint32_t Checksum) {<br></div></div><div><div>+      CfgChecksum = Checksum;</div>

<div>+    }</div></div><div><br></div><div>This function appears to be unused?</div><div><br></div><div><div>+    FileChecksum = zlib::crc32(EdgeDestinations);</div></div><div><br></div><div>Oof. So, if we don't link in zlib when building llvm, this is undefined behaviour on the spot (it isn't even guaranteed to terminate, it uses llvm_unreachable).</div>

<div><br></div><div>There's no need to use a crc32 here, just use whatever is in ADT/Hashing.h.</div><div><br></div><div>LGTM with above fixed.</div><div><br></div><div>Nick</div><div><br></div></div></div></div>