On Sun, Feb 5, 2012 at 11:17 AM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="word-wrap:break-word"><div class="im"><br><div><div>On Feb 4, 2012, at 11:16 PM, Talin wrote:</div><br><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" target="_blank">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><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></blockquote>

<div><br></div><div>OK it's in.</div><div><br></div><div>It would be possible to refactor the hashing logic - I was thinking of introducing a Hashing.h to ADT. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div style="word-wrap:break-word"><div><br></div><font color="#888888"><div>-Chris</div><br></font></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>-- Talin<br>