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