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

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 14 09:46:00 PST 2019


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.

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



More information about the llvm-commits mailing list