[Mesa-dev] [PATCH] glsl: Drop bogus is_vertex_input from add_shader_variable().

Alejandro Piñeiro apinheiro at igalia.com
Mon Dec 19 12:12:41 UTC 2016


lgtm

Reviewed-by: Alejandro Piñeiro <apinheiro at igalia.com>

On 17/12/16 03:46, Kenneth Graunke wrote:
> stage_mask is a bitmask of shader stages, so the proper comparison would
> be (1 << MESA_SHADER_VERTEX), not MESA_SHADER_VERTEX itself.
>
> But we only care for structure types, and VS inputs cannot be structs.
> So we can just drop this entirely.
>
> Signed-off-by: Kenneth Graunke <kenneth at whitecape.org>
> ---
>  src/compiler/glsl/linker.cpp | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
>
> Sorry, I forgot I split this into two patches...apply this one before
> [PATCH] glsl: Fix program interface queries relating to interface blocks.
>
> diff --git a/src/compiler/glsl/linker.cpp b/src/compiler/glsl/linker.cpp
> index 3660257..5066014 100644
> --- a/src/compiler/glsl/linker.cpp
> +++ b/src/compiler/glsl/linker.cpp
> @@ -3735,10 +3735,6 @@ add_shader_variable(const struct gl_context *ctx,
>                      bool use_implicit_location, int location,
>                      const glsl_type *outermost_struct_type = NULL)
>  {
> -   const bool is_vertex_input =
> -      programInterface == GL_PROGRAM_INPUT &&
> -      stage_mask == MESA_SHADER_VERTEX;
> -
>     switch (type->base_type) {
>     case GLSL_TYPE_STRUCT: {
>        /* The ARB_program_interface_query spec says:
> @@ -3764,8 +3760,7 @@ add_shader_variable(const struct gl_context *ctx,
>                                    outermost_struct_type))
>              return false;
>  
> -         field_location +=
> -            field->type->count_attribute_slots(is_vertex_input);
> +         field_location += field->type->count_attribute_slots(false);
>        }
>        return true;
>     }



More information about the mesa-dev mailing list