summaryrefslogtreecommitdiffstats
path: root/src/glsl/ir_validate.cpp
diff options
context:
space:
mode:
authorIan Romanick <ian.d.romanick@intel.com>2014-05-23 18:57:36 -0700
committerIan Romanick <ian.d.romanick@intel.com>2014-09-30 13:34:42 -0700
commit21df0169028d600b17ab73795da2838e92ba9038 (patch)
tree10cc2800c15be2af1b17aabd33a5c0c00cb697a4 /src/glsl/ir_validate.cpp
parent8afe6efa218f87158c35857fad41926b404a2cbd (diff)
downloadexternal_mesa3d-21df0169028d600b17ab73795da2838e92ba9038.zip
external_mesa3d-21df0169028d600b17ab73795da2838e92ba9038.tar.gz
external_mesa3d-21df0169028d600b17ab73795da2838e92ba9038.tar.bz2
glsl: Make ir_variable::max_ifc_array_access private
The payoff for this will come in a few more patches. No change Valgrind massif results for a trimmed apitrace of dota2. Signed-off-by: Ian Romanick <ian.d.romanick@intel.com> Reviewed-by: Matt Turner <mattst88@gmail.com> Reviewed-by: Tapani Pälli <tapani.palli@intel.com>
Diffstat (limited to 'src/glsl/ir_validate.cpp')
-rw-r--r--src/glsl/ir_validate.cpp9
1 files changed, 7 insertions, 2 deletions
diff --git a/src/glsl/ir_validate.cpp b/src/glsl/ir_validate.cpp
index 5ed8abb..96dd7bd 100644
--- a/src/glsl/ir_validate.cpp
+++ b/src/glsl/ir_validate.cpp
@@ -682,10 +682,15 @@ ir_validate::visit(ir_variable *ir)
ir->get_interface_type()->fields.structure;
for (unsigned i = 0; i < ir->get_interface_type()->length; i++) {
if (fields[i].type->array_size() > 0) {
- if (ir->max_ifc_array_access[i] >= fields[i].type->length) {
+ const unsigned *const max_ifc_array_access =
+ ir->get_max_ifc_array_access();
+
+ assert(max_ifc_array_access != NULL);
+
+ if (max_ifc_array_access[i] >= fields[i].type->length) {
printf("ir_variable has maximum access out of bounds for "
"field %s (%d vs %d)\n", fields[i].name,
- ir->max_ifc_array_access[i], fields[i].type->length);
+ max_ifc_array_access[i], fields[i].type->length);
ir->print();
abort();
}