diff options
author | Matt Turner <[email protected]> | 2015-01-28 18:37:32 -0800 |
---|---|---|
committer | Matt Turner <[email protected]> | 2015-02-03 12:25:14 -0800 |
commit | e87928a494a7cf0985a9d1cd78bda8729d17c614 (patch) | |
tree | 41570e0360460c38a13ac5f6de5d066cdee8d1eb | |
parent | cfa21656420c52a8c99e146314675f691e277c42 (diff) |
i965/fs: Add support for constant propagating into sources with modifiers.
All but 16 of the programs helped were ARB fp programs.
total instructions in shared programs: 5949286 -> 5945470 (-0.06%)
instructions in affected programs: 275162 -> 271346 (-1.39%)
helped: 1197
GAINED: 1
Reviewed-by: Jason Ekstrand <[email protected]>
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp index 5dd72559298..68a266c7331 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_copy_propagation.cpp @@ -448,16 +448,22 @@ fs_visitor::try_constant_propagate(fs_inst *inst, acp_entry *entry) (entry->dst.reg_offset + entry->regs_written) * 32) continue; - /* Don't bother with cases that should have been taken care of by the - * GLSL compiler's constant folding pass. - */ - if (inst->src[i].negate || inst->src[i].abs) - continue; - fs_reg val = entry->src; val.effective_width = inst->src[i].effective_width; val.type = inst->src[i].type; + if (inst->src[i].abs) { + if (!brw_abs_immediate(val.type, &val.fixed_hw_reg)) { + continue; + } + } + + if (inst->src[i].negate) { + if (!brw_negate_immediate(val.type, &val.fixed_hw_reg)) { + continue; + } + } + switch (inst->opcode) { case BRW_OPCODE_MOV: case SHADER_OPCODE_LOAD_PAYLOAD: |