New Iteration of subregister liveness patches

Quentin Colombet qcolombet at apple.com
Fri Dec 5 13:23:03 PST 2014


> On Dec 5, 2014, at 1:20 PM, Renato Golin <renato.golin at linaro.org> wrote:
> 
> On 5 December 2014 at 19:19, Tom Stellard <tom at stellard.net> wrote:
>> I tested these patches a few months ago and there were some regressions
>> on R600.  I'm not sure when I'll get a chance to test them again, but I
>> am in favor of merging these patches as soon as possible, since sub-reg
>> liveness is hidden behind a subtarget flag and it's easier to fully test
>> once it is in ToT.
> 
> If this is not the default behaviour, and it can be turned on by just
> adding a flag, then I don't see why not, after addressing Quentin's
> comments. I agree it'll be much easier to test this once it lands.

Same here. And I would add, my concerns can be addressed by subsequent patches.

Cheers,
Q.

> 
> cheers,
> --renato





More information about the llvm-commits mailing list