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

Xinliang David Li via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 30 22:37:48 PDT 2016


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
>>
>>
>>


More information about the llvm-commits mailing list