diff options
author | Iago Toral Quiroga <[email protected]> | 2016-10-21 13:15:41 +0200 |
---|---|---|
committer | Iago Toral Quiroga <[email protected]> | 2016-10-24 15:49:53 +0200 |
commit | 537dce06ec8e0fa4becd42d5e4b3d07cf722387f (patch) | |
tree | d496d524d2216475ce17f81f2a3051acb0519f2b /src/mesa | |
parent | 3d6b5dee3a0c9c077d68e5567b95f22b627be07e (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/mesa')
-rw-r--r-- | src/mesa/main/mtypes.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h index 88397b93e57..79dd1c51df7 100644 --- a/src/mesa/main/mtypes.h +++ b/src/mesa/main/mtypes.h @@ -2577,6 +2577,7 @@ struct gl_uniform_block * cross-validating uniform blocks. */ enum gl_uniform_block_packing _Packing; + GLboolean _RowMajor; }; /** |