diff options
author | Ilia Mirkin <[email protected]> | 2016-01-27 13:52:41 -0500 |
---|---|---|
committer | Ilia Mirkin <[email protected]> | 2016-01-28 11:31:19 -0500 |
commit | 089f60543930fcdab3848d59e6182abcaaeb1b86 (patch) | |
tree | 9c3c05b5ebb6b8ba514f428b969b6ad2c35cb2ae | |
parent | dda7a849868d5a4be6cec9d28c86a52aba62b32b (diff) |
glsl: disallow implicit conversions in ESSL shaders
Signed-off-by: Ilia Mirkin <[email protected]>
Reviewed-by: Timothy Arceri <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
-rw-r--r-- | src/compiler/glsl/ast_to_hir.cpp | 4 | ||||
-rw-r--r-- | src/compiler/glsl_types.cpp | 7 |
2 files changed, 11 insertions, 0 deletions
diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp index dfd31966eb0..3fca18a5087 100644 --- a/src/compiler/glsl/ast_to_hir.cpp +++ b/src/compiler/glsl/ast_to_hir.cpp @@ -291,6 +291,10 @@ apply_implicit_conversion(const glsl_type *to, ir_rvalue * &from, if (!state->is_version(120, 0)) return false; + /* ESSL does not allow implicit conversions */ + if (state->es_shader) + return false; + /* From page 27 (page 33 of the PDF) of the GLSL 1.50 spec: * * "There are no implicit array or structure conversions. For diff --git a/src/compiler/glsl_types.cpp b/src/compiler/glsl_types.cpp index 17ebf07acbc..ef6c3c6e3b7 100644 --- a/src/compiler/glsl_types.cpp +++ b/src/compiler/glsl_types.cpp @@ -1139,6 +1139,13 @@ glsl_type::can_implicitly_convert_to(const glsl_type *desired, if (this == desired) return true; + /* ESSL does not allow implicit conversions. If there is no state, we're + * doing intra-stage function linking where these checks have already been + * done. + */ + if (state && state->es_shader) + return false; + /* There is no conversion among matrix types. */ if (this->matrix_columns > 1 || desired->matrix_columns > 1) return false; |