On 12 February 2016 at 22:10, Axel Davy <[email protected]> wrote:
> POSITION0 input is forbidden for PS3.0 apparently.
>
> Besides for depth input we weren't using the system value
> version if needed
>
> Signed-off-by: Axel Davy <[email protected]>
>
> ---
> Could be better having it separated into two patches perhaps.
Yes, pretty please. Lets leave the nine_get_position_input() helper out.
> It should get into Mesa 11.2, to prevent regression caused
> by the introduction of position as system value.
>
I'll keep an eye open, although feel free to slap the "Cc: mesa-stable..." tag.
> @@ -2048,9 +2052,22 @@ DECL_SPECIAL(DCL)
> unsigned interp_location = 0;
> /* SM3 only, SM2 input semantic determined by file */
> assert(sem.reg.idx < Elements(tx->regs.v));
> +
> + if (tgsi.Name == TGSI_SEMANTIC_POSITION) {
> + /* Position0/PositionT0 are forbidden (likely because vPos
> already does that)
s/PositionT0/PositionT/ ?
> + * is forbidden. */
Too much "forbidden" ? Just drop this line of the comment.
> + if (sem.usage == D3DDECLUSAGE_POSITION && sem.usage_idx == 0
> ||
> + sem.usage == D3DDECLUSAGE_POSITIONT)
> + return D3DERR_INVALIDCALL;
Throw an empty newline, to improve readability.
Thanks
Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
