diff options
author | Paul Berry <stereotype441@gmail.com> | 2013-07-31 10:15:49 -0700 |
---|---|---|
committer | Paul Berry <stereotype441@gmail.com> | 2013-08-01 20:19:31 -0700 |
commit | d5a333a06f4dd8e084666e82019dde7cb40f9643 (patch) | |
tree | b7a74a193c44f4eccd43a4aa1f70017dbb0e505d /src/glsl/ir_set_program_inouts.cpp | |
parent | 0b0dc03a31e13aa1cb4e0a6479dbd09cef612d44 (diff) | |
download | external_mesa3d-d5a333a06f4dd8e084666e82019dde7cb40f9643.zip external_mesa3d-d5a333a06f4dd8e084666e82019dde7cb40f9643.tar.gz external_mesa3d-d5a333a06f4dd8e084666e82019dde7cb40f9643.tar.bz2 |
glsl: Extract marking functions from ir_set_program_inouts.
This patch extracts the functions mark_whole_variable() and
try_mark_partial_variable() from the ir_set_program_inouts visitor
functions. This will make the code easier to follow when we add
geometry shader support.
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Diffstat (limited to 'src/glsl/ir_set_program_inouts.cpp')
-rw-r--r-- | src/glsl/ir_set_program_inouts.cpp | 59 |
1 files changed, 45 insertions, 14 deletions
diff --git a/src/glsl/ir_set_program_inouts.cpp b/src/glsl/ir_set_program_inouts.cpp index 351f65b..1ba624f 100644 --- a/src/glsl/ir_set_program_inouts.cpp +++ b/src/glsl/ir_set_program_inouts.cpp @@ -59,6 +59,10 @@ public: virtual ir_visitor_status visit_enter(ir_discard *); virtual ir_visitor_status visit(ir_dereference_variable *); +private: + void mark_whole_variable(ir_variable *var); + bool try_mark_partial_variable(ir_variable *var, ir_rvalue *index); + struct gl_program *prog; GLenum shader_type; }; @@ -103,6 +107,17 @@ mark(struct gl_program *prog, ir_variable *var, int offset, int len, } } +/** + * Mark an entire variable as used. Caller must ensure that the variable + * represents a shader input or output. + */ +void +ir_set_program_inouts_visitor::mark_whole_variable(ir_variable *var) +{ + mark(this->prog, var, 0, var->type->count_attribute_slots(), + this->shader_type == GL_FRAGMENT_SHADER); +} + /* Default handler: Mark all the locations in the variable as used. */ ir_visitor_status ir_set_program_inouts_visitor::visit(ir_dereference_variable *ir) @@ -110,17 +125,43 @@ ir_set_program_inouts_visitor::visit(ir_dereference_variable *ir) if (!is_shader_inout(ir->var)) return visit_continue; - mark(this->prog, ir->var, 0, ir->type->count_attribute_slots(), - this->shader_type == GL_FRAGMENT_SHADER); + mark_whole_variable(ir->var); return visit_continue; } +/** + * Try to mark a portion of the given variable as used. Caller must ensure + * that the variable represents a shader input or output which can be indexed + * into in array fashion (an array or matrix). + * + * If the index can't be interpreted as a constant, or some other problem + * occurs, then nothing will be marked and false will be returned. + */ +bool +ir_set_program_inouts_visitor::try_mark_partial_variable(ir_variable *var, + ir_rvalue *index) +{ + const glsl_type *type = var->type; + + ir_constant *index_as_constant = index->as_constant(); + if (!index_as_constant) + return false; + + int width = 1; + if (type->is_array() && type->fields.array->is_matrix()) { + width = type->fields.array->matrix_columns; + } + + mark(this->prog, var, index_as_constant->value.u[0] * width, width, + this->shader_type == GL_FRAGMENT_SHADER); + return true; +} + ir_visitor_status ir_set_program_inouts_visitor::visit_enter(ir_dereference_array *ir) { ir_dereference_variable *deref_var; - ir_constant *index = ir->array_index->as_constant(); deref_var = ir->array->as_dereference_variable(); ir_variable *var = deref_var ? deref_var->var : NULL; @@ -128,18 +169,8 @@ ir_set_program_inouts_visitor::visit_enter(ir_dereference_array *ir) if (!var || !is_shader_inout(var)) return visit_continue; - if (index) { - int width = 1; - - if (deref_var->type->is_array() && - deref_var->type->fields.array->is_matrix()) { - width = deref_var->type->fields.array->matrix_columns; - } - - mark(this->prog, var, index->value.i[0] * width, width, - this->shader_type == GL_FRAGMENT_SHADER); + if (try_mark_partial_variable(var, ir->array_index)) return visit_continue_with_parent; - } return visit_continue; } |