[Mesa-dev] [PATCH] R600/SI: allow 4 more input SGPRs to a shader

Christian König deathsimple at vodafone.de
Tue Apr 29 06:26:30 PDT 2014


> Also, I think it would be a good idea to add the maximum possible number or registers
> to the calling convention (i.e. the number or register we would use if all inputs
> are enabled and all user sgprs are being used).
Well that's what we did previously, unfortunately it wasn't adopted when 
we added streamout to the mix.

With Mareks change we are probably there again and only need to worry 
about it again when we add support for the trap handlers.

Christian.

Am 28.04.2014 21:14, schrieb Tom Stellard:
> On Mon, Apr 28, 2014 at 08:27:07PM +0200, Marek Olšák wrote:
>> Please see my Mesa patch which moves index_bias to an SGPR. Doing it
>> conditionally would be messy.
>>
> An alternative solution would be to give the function a target specific
> calling convention and then create and implement that calling convention
> in the driver.  This way you would only need to put an ifdef around setting
> the function's calling convention.
>
> Also, I think it would be a good idea to add the maximum possible number or registers
> to the calling convention (i.e. the number or register we would use if all inputs
> are enabled and all user sgprs are being used).  We can always add an optimized
> calling convention later that loads some of the parameters from a constant buffer.
>
> -Tom
>
>> Marek
>>
>> On Mon, Apr 28, 2014 at 11:40 AM, Michel Dänzer <michel at daenzer.net> wrote:
>>> On 26.04.2014 22:36, Marek Olšák wrote:
>>>> From: Marek Olšák <marek.olsak at amd.com>
>>>>
>>>> Mesa needs one more SGPR, let's add 5 then, because it's a recurring issue.
>>>>
>>>> This is silly. Why cannot Mesa specify the calling convention?
>>>> Now I have to break transform feedback to support ARB_draw_indirect until
>>>> this patch is released. Or I'll have to wait until the release and then
>>>> merge ARB_draw_indirect support.
>>> Can't you do it the way we did several times before: Land the LLVM
>>> change, then land the Mesa change with the new functionality only
>>> enabled if built against LLVM >= 3.5?
>>>
>>>
>>> --
>>> Earthling Michel Dänzer            |                  http://www.amd.com
>>> Libre software enthusiast          |                Mesa and X developer
>> _______________________________________________
>> mesa-dev mailing list
>> mesa-dev at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev



More information about the mesa-dev mailing list