[Mesa-dev] [PATCH] glsl: add image_format check in cross_validate_globals()

Tapani Pälli tapani.palli at intel.com
Mon Jan 11 02:06:12 PST 2016


Reviewed-by: Tapani Pälli <tapani.palli at intel.com>

On 01/05/2016 03:32 PM, Samuel Iglesias Gonsálvez wrote:
> Fixes CTS test:
>
> ES31-CTS.shader_image_load_store.negative-linkErrors
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93410
>
> Signed-off-by: Samuel Iglesias Gonsálvez <siglesias at igalia.com>
> ---
>   src/glsl/linker.cpp | 6 ++++++
>   1 file changed, 6 insertions(+)
>
> diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp
> index a6e81b4..cf4d68d 100644
> --- a/src/glsl/linker.cpp
> +++ b/src/glsl/linker.cpp
> @@ -1133,6 +1133,12 @@ cross_validate_globals(struct gl_shader_program *prog,
>                               mode_string(var), var->name);
>                  return;
>               }
> +            if (existing->data.image_format != var->data.image_format) {
> +               linker_error(prog, "declarations for %s `%s` have "
> +                            "mismatching image format qualifiers\n",
> +                            mode_string(var), var->name);
> +               return;
> +            }
>   	 } else
>   	    variables.add_variable(var);
>         }
>


More information about the mesa-dev mailing list