[Mesa-dev] [PATCH 1/6] i965/fs: Make the register allocation class_sizes[] choice static.

Eric Anholt eric at anholt.net
Thu Oct 4 12:52:42 PDT 2012


Kenneth Graunke <kenneth at whitecape.org> writes:

> On 10/02/2012 07:52 PM, Eric Anholt wrote:
>> Based on split_virtual_grfs(), we choose the same set every time, so set it in
>> stone.  This will help us avoid regenerating the somewhat expensive
>> class/register set setup every compile.
>> ---
>>   src/mesa/drivers/dri/i965/brw_fs.h                |    1 +
>>   src/mesa/drivers/dri/i965/brw_fs_reg_allocate.cpp |  101 +++++++++------------
>>   2 files changed, 42 insertions(+), 60 deletions(-)
>
> Hmm.  I guess we don't have SEND from GRFs expressed at the visitor 
> level...only a couple of instances at the generate_code level.  And 
> those don't use more than 4 payload registers.
>
> In that case, this seems reasonable.

Yeah, if we need bigger vgrfs (like the branch I have laying around),
then I think we'll just statically add those other sizes.  They're small
numbers.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/mesa-dev/attachments/20121004/aa63414f/attachment.pgp>


More information about the mesa-dev mailing list