[Mesa-dev] [PATCH] glsl: fix packed varyings interface type and add default case

Ilia Mirkin imirkin at alum.mit.edu
Fri Sep 25 00:31:24 PDT 2015


On Fri, Sep 25, 2015 at 3:28 AM, Tapani Pälli <tapani.palli at intel.com> wrote:
> fixes Piglit test:
>    arb_program_interface_query/linker/query-varyings.shader_test
>
> Signed-off-by: Tapani Pälli <tapani.palli at intel.com>

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

> ---
>  src/glsl/linker.cpp | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
> index 53e276c..ab54e11 100644
> --- a/src/glsl/linker.cpp
> +++ b/src/glsl/linker.cpp
> @@ -3239,8 +3239,12 @@ add_packed_varyings(struct gl_shader_program *shProg, int stage)
>           switch (var->data.mode) {
>           case ir_var_shader_in:
>              iface = GL_PROGRAM_INPUT;
> +            break;
>           case ir_var_shader_out:
>              iface = GL_PROGRAM_OUTPUT;
> +            break;
> +         default:
> +            unreachable("unexpected type");
>           }
>           if (!add_program_resource(shProg, iface, var,
>                                     build_stageref(shProg, var->name,
> --
> 2.4.3
>


More information about the mesa-dev mailing list