[PATCH] D22778: Add Loop Sink pass to reverse the LICM based of basic block frequency.

[Chandler Carruth via llvm-commits]

Thanks for that context.

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.

On Sat, Jul 30, 2016 at 10:37 PM Xinliang David Li via llvm-commits <
llvm-commits at lists.llvm.org> wrote:

> It seems phab did not have all the conversations recorded. Complete
> discussion can be found in llvm-commits archive
>
> On Sat, Jul 30, 2016 at 10:31 PM, Xinliang David Li <davidxl at google.com>
> wrote:
> > See discussions in https://reviews.llvm.org/D19950
> >
> > David
> >
> > On Sat, Jul 30, 2016 at 10:18 PM, Chandler Carruth <chandlerc at gmail.com>
> wrote:
> >> chandlerc added a subscriber: chandlerc.
> >> chandlerc added a comment.
> >>
> >> Have you talked to anyone about the design for this?
> >>
> >> 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.
> >>
> >>
> >> https://reviews.llvm.org/D22778
> >>
> >>
> >>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160731/9fd895f0/attachment.html>