[PATCH] D58200: [RegAllocGreedy] Take last chance recoloring into account in evicting.

Mikael Holmén via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 18 06:45:23 PST 2019



On 2/14/19 6:46 PM, Quentin Colombet via llvm-commits wrote:
> Hi Mikael,
> 
>> On Feb 14, 2019, at 12:47 AM, Mikael Holmén via Phabricator via llvm-commits <llvm-commits at lists.llvm.org> wrote:
>>
>> uabelho added a comment.
>>
>> Hi!
>>
>> Nice to see this fix get in because we've seen the need for such a change for our out-of-tree target as well.
>>
>> However, we've added a FixedRegisters argument to tryAssign and trySplit too.
>> It was a while since we did that change locally but would it make sense to push that into trunk too?
> 
> Yes, please submit a patch.
> 

Alright, will do tomorrow.

Thanks!
/Mikael

> Thanks,
> -Quentin
> 
>>
>>
>> Repository:
>>   rL LLVM
>>
>> CHANGES SINCE LAST ACTION
>>   https://reviews.llvm.org/D58200/new/
>>
>> https://reviews.llvm.org/D58200
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> 



More information about the llvm-commits mailing list