[PATCH] D15302: [Greedy regalloc] Replace analyzeSiblingValues with something new [Part1]

Quentin Colombet via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 21 17:21:11 PDT 2016


> On Mar 21, 2016, at 5:15 PM, Wei Mi <wmi at google.com> wrote:
> 
> On Mon, Mar 21, 2016 at 5:07 PM, Quentin Colombet <qcolombet at apple.com> wrote:
>> qcolombet added a comment.
>> 
>> Hi Wei,
>> 
>> I think we will need to wait for Tom to double check what happened for AMDGPU.
> 
> Sure.

> 
>> 
>> One question though, this revision ended up being the combination of the 3 parts, right?
>> 
> 
> Yes, it is the combination of the 3 parts. It can save us some test
> change, but make the final patch larger.

Thanks for the confirmation.
That works for me.

You should close the other two diffs.

Cheers,
-Quentin

> 
> Wei.



More information about the llvm-commits mailing list