[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