diff options
author | Ian Romanick <[email protected]> | 2012-12-14 12:00:14 -0800 |
---|---|---|
committer | Ian Romanick <[email protected]> | 2013-01-25 09:07:34 -0500 |
commit | 13be1f4a103802ed936f2374d72b2f6979dafd58 (patch) | |
tree | 636053ac19725220449afbcb65310ec99e1c1ddf /src/glsl | |
parent | 17e6f1904444be019cafa208495e3fe479cb86fd (diff) |
glsl: Add ir_variable::is_in_uniform_block predicate
The way a variable is tested for this property is about to change, and
this makes the code easier to modify.
Signed-off-by: Ian Romanick <[email protected]>
Reviewed-by: Carl Worth <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/glsl')
-rw-r--r-- | src/glsl/ir.h | 8 | ||||
-rw-r--r-- | src/glsl/link_uniforms.cpp | 4 | ||||
-rw-r--r-- | src/glsl/linker.cpp | 2 | ||||
-rw-r--r-- | src/glsl/lower_ubo_reference.cpp | 2 | ||||
-rw-r--r-- | src/glsl/opt_dead_code.cpp | 2 |
5 files changed, 13 insertions, 5 deletions
diff --git a/src/glsl/ir.h b/src/glsl/ir.h index 9fdfb18e25b..14212dfb7a2 100644 --- a/src/glsl/ir.h +++ b/src/glsl/ir.h @@ -350,6 +350,14 @@ public: glsl_interp_qualifier determine_interpolation_mode(bool flat_shade); /** + * Determine whether or not a variable is part of a uniform block. + */ + inline bool is_in_uniform_block() const + { + return this->mode == ir_var_uniform && this->uniform_block != -1; + } + + /** * Declared type of the variable */ const struct glsl_type *type; diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp index 25bba15bdbe..c639a3d16a4 100644 --- a/src/glsl/link_uniforms.cpp +++ b/src/glsl/link_uniforms.cpp @@ -228,7 +228,7 @@ public: ir_variable *var) { ubo_var = NULL; - if (var->uniform_block != -1) { + if (var->is_in_uniform_block()) { struct gl_uniform_block *block = &shader->UniformBlocks[var->uniform_block]; @@ -442,7 +442,7 @@ link_update_uniform_buffer_variables(struct gl_shader *shader) foreach_list(node, shader->ir) { ir_variable *const var = ((ir_instruction *) node)->as_variable(); - if ((var == NULL) || (var->uniform_block == -1)) + if ((var == NULL) || !var->is_in_uniform_block()) continue; assert(var->mode == ir_var_uniform); diff --git a/src/glsl/linker.cpp b/src/glsl/linker.cpp index 1d4e2f6e793..a480dd0523f 100644 --- a/src/glsl/linker.cpp +++ b/src/glsl/linker.cpp @@ -1077,7 +1077,7 @@ update_array_sizes(struct gl_shader_program *prog) * will not be eliminated. Since we always do std140, just * don't resize arrays in UBOs. */ - if (var->uniform_block != -1) + if (var->is_in_uniform_block()) continue; unsigned int size = var->max_array_access; diff --git a/src/glsl/lower_ubo_reference.cpp b/src/glsl/lower_ubo_reference.cpp index e8d2c4742d6..1d08009a34e 100644 --- a/src/glsl/lower_ubo_reference.cpp +++ b/src/glsl/lower_ubo_reference.cpp @@ -78,7 +78,7 @@ lower_ubo_reference_visitor::handle_rvalue(ir_rvalue **rvalue) return; ir_variable *var = deref->variable_referenced(); - if (!var || var->uniform_block == -1) + if (!var || !var->is_in_uniform_block()) return; mem_ctx = ralloc_parent(*rvalue); diff --git a/src/glsl/opt_dead_code.cpp b/src/glsl/opt_dead_code.cpp index dad307af779..78d31cfcb3f 100644 --- a/src/glsl/opt_dead_code.cpp +++ b/src/glsl/opt_dead_code.cpp @@ -106,7 +106,7 @@ do_dead_code(exec_list *instructions, bool uniform_locations_assigned) if (entry->var->mode == ir_var_uniform && (uniform_locations_assigned || entry->var->constant_value || - entry->var->uniform_block != -1)) + entry->var->is_in_uniform_block())) continue; entry->var->remove(); |