<div dir="ltr">Thanks for that context.<div><br></div><div>However, the conclusion there seemed to be to improve the MachineSinking pass rather than produce an IR level sinking pass, and I don't see anywhere that this direction change is explained.</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Jul 30, 2016 at 10:37 PM Xinliang David Li via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">It seems phab did not have all the conversations recorded. Complete<br>
discussion can be found in llvm-commits archive<br>
<br>
On Sat, Jul 30, 2016 at 10:31 PM, Xinliang David Li <<a href="mailto:davidxl@google.com" target="_blank">davidxl@google.com</a>> wrote:<br>
> See discussions in <a href="https://reviews.llvm.org/D19950" rel="noreferrer" target="_blank">https://reviews.llvm.org/D19950</a><br>
><br>
> David<br>
><br>
> On Sat, Jul 30, 2016 at 10:18 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>> wrote:<br>
>> chandlerc added a subscriber: chandlerc.<br>
>> chandlerc added a comment.<br>
>><br>
>> Have you talked to anyone about the design for this?<br>
>><br>
>> I know Daniel Jasper, Quentin, and several others have looked at similar things before. Previous attempts have focused on using MachineLICM to do sinking as well as hoisting. While I don't have a strong opinion about one design over the other, we should be consistent about the plan here, and possibly consolidate some of the logic.<br>
>><br>
>><br>
>> <a href="https://reviews.llvm.org/D22778" rel="noreferrer" target="_blank">https://reviews.llvm.org/D22778</a><br>
>><br>
>><br>
>><br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>