diff options
Diffstat (limited to 'src/compiler')
-rw-r--r-- | src/compiler/glsl/ast_to_hir.cpp | 12 | ||||
-rw-r--r-- | src/compiler/glsl_types.cpp | 13 | ||||
-rw-r--r-- | src/compiler/glsl_types.h | 6 |
3 files changed, 31 insertions, 0 deletions
diff --git a/src/compiler/glsl/ast_to_hir.cpp b/src/compiler/glsl/ast_to_hir.cpp index 1114d9e4595..33490c831cb 100644 --- a/src/compiler/glsl/ast_to_hir.cpp +++ b/src/compiler/glsl/ast_to_hir.cpp @@ -5825,6 +5825,18 @@ ast_function::hir(exec_list *instructions, "sized", name); } + /* From Section 6.1 (Function Definitions) of the GLSL 1.00 spec: + * + * "Arrays are allowed as arguments, but not as the return type. [...] + * The return type can also be a structure if the structure does not + * contain an array." + */ + if (state->language_version == 100 && return_type->contains_array()) { + YYLTYPE loc = this->get_location(); + _mesa_glsl_error(& loc, state, + "function `%s' return type contains an array", name); + } + /* From section 4.1.7 of the GLSL 4.40 spec: * * "[Opaque types] can only be declared as function parameters diff --git a/src/compiler/glsl_types.cpp b/src/compiler/glsl_types.cpp index 43c0188e47f..0fd12305b7d 100644 --- a/src/compiler/glsl_types.cpp +++ b/src/compiler/glsl_types.cpp @@ -220,6 +220,19 @@ glsl_type::contains_sampler() const } } +bool +glsl_type::contains_array() const +{ + if (this->is_record() || this->is_interface()) { + for (unsigned int i = 0; i < this->length; i++) { + if (this->fields.structure[i].type->contains_array()) + return true; + } + return false; + } else { + return this->is_array(); + } +} bool glsl_type::contains_integer() const diff --git a/src/compiler/glsl_types.h b/src/compiler/glsl_types.h index 67c152119f8..9da0cbcc3b0 100644 --- a/src/compiler/glsl_types.h +++ b/src/compiler/glsl_types.h @@ -563,6 +563,12 @@ struct glsl_type { bool contains_sampler() const; /** + * Query whether or not type is an array or for struct, interface and + * array types, contains an array. + */ + bool contains_array() const; + + /** * Get the Mesa texture target index for a sampler type. */ gl_texture_index sampler_index() const; |