[Mesa-dev] [PATCH 7/8] glsl: Minor clean ups in constant_referenced
Ian Romanick
idr at freedesktop.org
Wed Mar 12 15:49:26 PDT 2014
From: Ian Romanick <ian.d.romanick at intel.com>
These could probably be squashed into one of the previous commits.
Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
---
src/glsl/ir_constant_expression.cpp | 17 ++++++-----------
1 file changed, 6 insertions(+), 11 deletions(-)
diff --git a/src/glsl/ir_constant_expression.cpp b/src/glsl/ir_constant_expression.cpp
index ea9e84a..e4f8a58 100644
--- a/src/glsl/ir_constant_expression.cpp
+++ b/src/glsl/ir_constant_expression.cpp
@@ -408,41 +408,36 @@ constant_referenced(const ir_dereference *deref,
const ir_dereference_array *const da =
(const ir_dereference_array *) deref;
- ir_constant *index_c =
+ ir_constant *const index_c =
da->array_index->constant_expression_value(variable_context);
if (!index_c || !index_c->type->is_scalar() || !index_c->type->is_integer())
break;
- int index = index_c->type->base_type == GLSL_TYPE_INT ?
+ const int index = index_c->type->base_type == GLSL_TYPE_INT ?
index_c->get_int_component(0) :
index_c->get_uint_component(0);
ir_constant *substore;
int suboffset;
- const ir_dereference *deref = da->array->as_dereference();
+ const ir_dereference *const deref = da->array->as_dereference();
if (!deref)
break;
if (!constant_referenced(deref, variable_context, substore, suboffset))
break;
- const glsl_type *vt = da->array->type;
+ const glsl_type *const vt = da->array->type;
if (vt->is_array()) {
store = substore->get_array_element(index);
offset = 0;
- break;
- }
- if (vt->is_matrix()) {
+ } else if (vt->is_matrix()) {
store = substore;
offset = index * vt->vector_elements;
- break;
- }
- if (vt->is_vector()) {
+ } else if (vt->is_vector()) {
store = substore;
offset = suboffset + index;
- break;
}
break;
--
1.8.1.4
More information about the mesa-dev
mailing list