glsl: add matrix layout information to interface block types
authorIago Toral Quiroga <itoral@igalia.com>
Fri, 21 Oct 2016 11:15:41 +0000 (13:15 +0200)
committerIago Toral Quiroga <itoral@igalia.com>
Mon, 24 Oct 2016 13:49:53 +0000 (15:49 +0200)
commit537dce06ec8e0fa4becd42d5e4b3d07cf722387f
treed496d524d2216475ce17f81f2a3051acb0519f2b
parent3d6b5dee3a0c9c077d68e5567b95f22b627be07e
glsl: add matrix layout information to interface block types

So far we have been checking that interface block definitions had matching
matrix layouts by comparing the definitions of their fields, however, this
does not cover the case where the interface blocks are defined with
mismatching matrix layouts but don't define any field with a matrix type.
In this case Mesa will not fail to link because none of the fields will
inherit the mismatching layout qualifier.

This patch fixes the problem in the same way we fixed it for packing layout
information: we add the the layout information to the interface type and then
we check it matches during the uniform block linking process.

v2: Fix unit tests so they pass the new parameter to
    glsl_type::get_interface_instance()

Fixes:
dEQP-GLES31.functional.shaders.linkage.uniform.block.layout_qualifier_mismatch_3

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=98245
Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com> (v1)
src/compiler/glsl/ast_to_hir.cpp
src/compiler/glsl/builtin_variables.cpp
src/compiler/glsl/link_uniform_blocks.cpp
src/compiler/glsl/linker.cpp
src/compiler/glsl/tests/general_ir_test.cpp
src/compiler/glsl/tests/varyings_test.cpp
src/compiler/glsl_types.cpp
src/compiler/glsl_types.h
src/mesa/main/mtypes.h