<div dir="ltr">In r189621 and r189622.<div><br></div><div>Manman </div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 29, 2013 at 4:03 PM, Manman Ren <span dir="ltr"><<a href="mailto:manman.ren@gmail.com" target="_blank">manman.ren@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Increased from 4 to 16, and returning SmallString instead of passing in a reference.<span class="HOEnZb"><font color="#888888"><div>
<br></div><div>Manman<br><div><br></div></div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Aug 29, 2013 at 3:53 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">One more comment on top of Dave's:<br>
<br>
+  SmallVector<Value *, 4> RetainValues;<br>
+  // Both createForwardDecl and createStruct|Class|etc|Type can add the same<br>
+  // type to AllRetainTypes. A set is used to avoid duplication.<br>
+  SmallPtrSet<Value *, 4> RetainSet;<br>
<br>
4 seems a bit low.<br>
<span><font color="#888888"><br>
-eric<br>
</font></span><div><br>
<br>
On Thu, Aug 29, 2013 at 2:33 PM, Manman Ren <<a href="mailto:manman.ren@gmail.com" target="_blank">manman.ren@gmail.com</a>> wrote:<br>
</div><div><div>> Hi,<br>
><br>
> I think it is a good idea to start a new thread :)<br>
> Attached please find 3 patches: one for llvm (update DIBuilder to retain<br>
> types with unique<br>
> identifier), one for clang (to generate the unique identifier), the last for<br>
> updating testing cases.<br>
><br>
> Thanks,<br>
> Manman<br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>