aboutsummaryrefslogtreecommitdiffstats
path: root/src/compiler/glsl/tests/varyings_test.cpp
diff options
context:
space:
mode:
authorIago Toral Quiroga <[email protected]>2016-10-21 13:15:41 +0200
committerIago Toral Quiroga <[email protected]>2016-10-24 15:49:53 +0200
commit537dce06ec8e0fa4becd42d5e4b3d07cf722387f (patch)
treed496d524d2216475ce17f81f2a3051acb0519f2b /src/compiler/glsl/tests/varyings_test.cpp
parent3d6b5dee3a0c9c077d68e5567b95f22b627be07e (diff)
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 <[email protected]> (v1)
Diffstat (limited to 'src/compiler/glsl/tests/varyings_test.cpp')
-rw-r--r--src/compiler/glsl/tests/varyings_test.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/compiler/glsl/tests/varyings_test.cpp b/src/compiler/glsl/tests/varyings_test.cpp
index 7643e1bbbf9..eb134c9e8f2 100644
--- a/src/compiler/glsl/tests/varyings_test.cpp
+++ b/src/compiler/glsl/tests/varyings_test.cpp
@@ -83,6 +83,7 @@ link_varyings::link_varyings()
glsl_type::get_interface_instance(f,
ARRAY_SIZE(f),
GLSL_INTERFACE_PACKING_STD140,
+ false,
"simple_interface");
}