[Mesa-dev] [PATCH 1/2] gallium: comment that INSTANCEID doesn't include start_instance

Christian König deathsimple at vodafone.de
Thu Sep 5 07:47:50 PDT 2013


I was aware that this was only correct in one use case, but didn't know 
which one. Probably should have added a comment about that.

Thanks for fixing it, both patches are: Reviewed-by: Christian König 
<christian.koenig at amd.com>

Am 05.09.2013 16:41, schrieb Marek Olšák:
> ---
>   src/gallium/include/pipe/p_shader_tokens.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/include/pipe/p_shader_tokens.h b/src/gallium/include/pipe/p_shader_tokens.h
> index 872dfe9..1beec05 100644
> --- a/src/gallium/include/pipe/p_shader_tokens.h
> +++ b/src/gallium/include/pipe/p_shader_tokens.h
> @@ -153,7 +153,7 @@ struct tgsi_declaration_interp
>   #define TGSI_SEMANTIC_FACE       7
>   #define TGSI_SEMANTIC_EDGEFLAG   8
>   #define TGSI_SEMANTIC_PRIMID     9
> -#define TGSI_SEMANTIC_INSTANCEID 10
> +#define TGSI_SEMANTIC_INSTANCEID 10 /**< doesn't include start_instance */
>   #define TGSI_SEMANTIC_VERTEXID   11
>   #define TGSI_SEMANTIC_STENCIL    12
>   #define TGSI_SEMANTIC_CLIPDIST   13



More information about the mesa-dev mailing list