[Mesa-dev] [PATCH] mesa: allow unlimited temporaries

[Ian Romanick]

On Sep 5, 2010, at 3:38 AM, Christoph Bumiller <e0425955 at student.tuwien.ac.at 
 > wrote:

> On 09/05/2010 01:59 AM, Luca Barbieri wrote:
>> Currently Mesa has a limit of 1024 temporaries.
>>
>> Unfortunately, some GLSL shaders such as an SSAO fragment
>> post-processing shader in Unigine Tropics, go over this limit at
>> least before program optimizations are applied.

By the time we generate Mesa IR all the optimizations should be done.

>>
>> Instead of just enlarging the limit, remove it completely and
>> replace all arrays using it with dynamic allocation code.
>>
>> If no one objects, I'll push this.
>> ---
>
> How the ... is it using 1000 TEMPs ?
> Wait, does it have large arrays and are they accessed indirectly ?
>
> Can you implement* an INDEXABLE_TEMPORARY file instead so we can use
> TGSI_FILE_TEMPORARY_ARRAY so that I can finally support this in nv50
> without having to spill my whole GPR file to memory because I don't
> know what part of TEMP can be accessed by the indirect load/store ?

This is the right way to go. It should be handled at a higher level  
IR. It will be a lot easier to do this before generating the low-level  
Mesa IR.

Sent from my phone.

> Kthxbye :-)
>
> * or wait for someone (possibly me, but unlikely) to implement ...
> might be nicer, but, I'm not really objecting to the patch