[Mesa-dev] [PATCH] i965/fs: Add pass to rename registers to break live ranges.
Eric Anholt
eric at anholt.net
Mon May 12 13:50:46 PDT 2014
Eric Anholt <eric at anholt.net> writes:
> Matt Turner <mattst88 at gmail.com> writes:
>
>> From: Kenneth Graunke <kenneth at whitecape.org>
>>
>> The pass breaks live ranges of virtual registers by allocating new
>> registers when it sees an assignment to a virtual GRF it's already seen
>> written.
>>
>> total instructions in shared programs: 1656292 -> 1651898 (-0.27%)
>> instructions in affected programs: 466836 -> 462442 (-0.94%)
>> GAINED: 173
>> LOST: 3
>> ---
>> [mattst88]: Perform renaming at control flow level 0, rather than bailing
>> at first sight of control flow.
>
> I think this needs to handle renaming the regs in special[] in
> compact_virtual_grfs(). Either that or we *don't* have to actually do
> that work and we should drop it from that code, too.
Now that special[] isn't special any more, this patch is:
Reviewed-by: Eric Anholt <eric at anholt.net>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20140512/be42bd00/attachment.sig>
More information about the mesa-dev
mailing list