<div dir="ltr">Hi Gerolf,<div><br></div><div>The implementation looks good to me, but the testing case will not work since r206429 was reverted.</div><div>If it is not possible to come up with a testing case, please try your best to explain that in the commit log.</div>
<div><br></div><div>Thanks,</div><div>Manman</div><div><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Apr 17, 2014 at 2:23 PM, Gerolf Hoflehner <span dir="ltr"><<a href="mailto:ghoflehner@apple.com" target="_blank">ghoflehner@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">any opinions?<br>
<div><div class="h5"><br>
On Apr 14, 2014, at 3:29 PM, Gerolf Hoflehner <<a href="mailto:ghoflehner@apple.com">ghoflehner@apple.com</a>> wrote:<br>
<br>
> Hi<br>
><br>
> In some cases a reference to a cloned block is not available since it has not been cloned yet. The net effect is that some block may not get inlined. The attached patch solves this timing problem by first collecting all candidate inline blocks and computing the references as necessary, and then proceeding with the actual cloning.<br>

><br>
> Gerolf<br>
><br>
</div></div>> <r16427209.patch>_______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>