diff options
author | Matt Turner <mattst88@gmail.com> | 2014-10-17 20:32:58 -0700 |
---|---|---|
committer | Matt Turner <mattst88@gmail.com> | 2014-10-29 21:35:46 -0700 |
commit | d056863b3c535aeebfe5fcfc9468eb33a06ddb60 (patch) | |
tree | 9718584e114426560451e5d0c075412775d83353 /src | |
parent | 26122e09a304061e46690e1b9da2e36a527b3e2e (diff) |
glsl: Drop constant 0.0 components from dot products.
Helps a small number of vertex shaders in the games Dungeon Defenders
and Shank, as well as an internal benchmark.
instructions in affected programs: 2801 -> 2719 (-2.93%)
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/glsl/opt_algebraic.cpp | 27 |
1 files changed, 27 insertions, 0 deletions
diff --git a/src/glsl/opt_algebraic.cpp b/src/glsl/opt_algebraic.cpp index 0cdb8ecfc0a..c8d1ba1e2f5 100644 --- a/src/glsl/opt_algebraic.cpp +++ b/src/glsl/opt_algebraic.cpp @@ -553,6 +553,33 @@ ir_algebraic_visitor::handle_expression(ir_expression *ir) } return new(mem_ctx) ir_swizzle(ir->operands[0], component, 0, 0, 0, 1); } + + for (int i = 0; i < 2; i++) { + if (!op_const[i]) + continue; + + unsigned components[4] = { 0 }, count = 0; + + for (unsigned c = 0; c < op_const[i]->type->vector_elements; c++) { + if (op_const[i]->value.f[c] == 0.0) + continue; + + components[count] = c; + count++; + } + + /* No channels had zero values; bail. */ + if (count >= op_const[i]->type->vector_elements) + break; + + /* Swizzle both operands to remove the channels that were zero. */ + return new(mem_ctx) + ir_expression(ir_binop_dot, glsl_type::float_type, + new(mem_ctx) ir_swizzle(ir->operands[0], + components, count), + new(mem_ctx) ir_swizzle(ir->operands[1], + components, count)); + } break; case ir_binop_less: |