Re: [1/5] wined3d: Add missing temporary variable declaration to ARB shader backend.

[Stefan Dösinger]

> +    shader_addline(buffer, "TEMP TB;\n");
> Which shader instruction uses this? Afair vertex shaders should only use TA, 
> but there's no proper infrastructure that manages that.

[1/5] wined3d: Add missing temporary variable declaration to ARB shader backend.

[Matteo Bruni]

2011/4/29 Stefan Dösinger <stefandoesinger(a)gmx.at>:
> > +    shader_addline(buffer, "TEMP TB;\n");
> > Which shader instruction uses this? Afair vertex shaders should only use TA,
> > but there's no proper infrastructure that manages that.

AFAICS, TB could get used in shader_hw_pow():

...
    shader_arb_get_src_param(ins, &ins->src[1], 1, src1);
...

where the '1' in the third argument means that
shader_arb_get_src_param() is allowed to use TB as temporary storage.
There may be other pieces of code with the same assumption, I didn't
look deeper.

[1/5] wined3d: Add missing temporary variable declaration to ARB shader backend.

[Stefan Dösinger]

Am 29.04.2011 um 00:39 schrieb Matteo Bruni:
> 2011/4/29 Stefan Dösinger <stefandoesinger(a)gmx.at>:
> > > +    shader_addline(buffer, "TEMP TB;\n");
> > > Which shader instruction uses this? Afair vertex shaders should only use TA,
> > > but there's no proper infrastructure that manages that.
>
> AFAICS, TB could get used in shader_hw_pow():
>
> ...
>    shader_arb_get_src_param(ins, &ins->src[1], 1, src1);
> ...
>
> where the '1' in the third argument means that
> shader_arb_get_src_param() is allowed to use TB as temporary storage.
> There may be other pieces of code with the same assumption, I didn't
> look deeper.
> Ok