<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Feb 4, 2012, at 11:16 PM, Talin wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">This patch improves performance and memory usage for uniquing of ConstantStruct, ConstantArray, and ConstantVector. It's been verified to work both by myself and Meador Inge.<div><br></div><div>I'm posting it here because I wasn't sure if attaching it to the bug (<a href="http://llvm.org/bugs/show_bug.cgi?id=1210">http://llvm.org/bugs/show_bug.cgi?id=1210</a>) was sufficient to get approval for submission.<br clear="all">

</div></blockquote><br></div><div>Looks great to me.  It is unfortuanate that you have to manually write out the hashing logic like that, but it's fine with me.  2x memory reduction is huge!</div><div><br></div><div>-Chris</div><br></body></html>