[Mesa-dev] [PATCH V7 04/24] glsl: dont allow gl_PerVertex to be redeclared as an array of arrays

Timothy Arceri t_arceri at yahoo.com.au
Tue Oct 6 15:47:32 PDT 2015


V3: move patch after fixes to ast for AoA and add const to helper
as suggested by Ian

V2: move single dimensional array detection into a helper

Signed-off-by: Timothy Arceri <t_arceri at yahoo.com.au>
Reviewed-by: Ian Romanick <ian.d.romanick at intel.com>
---
 src/glsl/ast.h          | 6 ++++++
 src/glsl/ast_to_hir.cpp | 3 ++-
 2 files changed, 8 insertions(+), 1 deletion(-)

diff --git a/src/glsl/ast.h b/src/glsl/ast.h
index b43be24..e5e3d4e 100644
--- a/src/glsl/ast.h
+++ b/src/glsl/ast.h
@@ -330,6 +330,12 @@ public:
       array_dimensions.push_tail(&dim->link);
    }
 
+   const bool is_single_dimension()
+   {
+      return this->array_dimensions.tail_pred->prev != NULL &&
+             this->array_dimensions.tail_pred->prev->is_head_sentinel();
+   }
+
    virtual void print(void) const;
 
    /* This list contains objects of type ast_node containing the
diff --git a/src/glsl/ast_to_hir.cpp b/src/glsl/ast_to_hir.cpp
index 3dfa3e6..5f15dbc 100644
--- a/src/glsl/ast_to_hir.cpp
+++ b/src/glsl/ast_to_hir.cpp
@@ -6186,7 +6186,8 @@ ast_interface_block::hir(exec_list *instructions,
                              _mesa_shader_stage_to_string(state->stage));
          }
          if (this->instance_name == NULL ||
-             strcmp(this->instance_name, "gl_in") != 0 || this->array_specifier == NULL) {
+             strcmp(this->instance_name, "gl_in") != 0 || this->array_specifier == NULL ||
+             !this->array_specifier->is_single_dimension()) {
             _mesa_glsl_error(&loc, state,
                              "gl_PerVertex input must be redeclared as "
                              "gl_in[]");
-- 
2.4.3



More information about the mesa-dev mailing list