<div dir="ltr">Increased from 4 to 16, and returning SmallString instead of passing in a reference.<div><br></div><div>Manman<br><div><br></div></div></div><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 class="HOEnZb"><font color="#888888"><br>
-eric<br>
</font></span><div class="im HOEnZb"><br>
<br>
On Thu, Aug 29, 2013 at 2:33 PM, Manman Ren <<a href="mailto:manman.ren@gmail.com">manman.ren@gmail.com</a>> wrote:<br>
</div><div class="HOEnZb"><div class="h5">> 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>