diff options
| author | Matt Turner <[email protected]> | 2015-09-21 13:58:19 -0700 |
|---|---|---|
| committer | Matt Turner <[email protected]> | 2015-09-22 21:20:29 -0700 |
| commit | 10da96887c785930c2553b2d5bde91e52b8b034a (patch) | |
| tree | 7fe21c647ec686f68b1fc0cc78e3179d5894bf46 /src/mesa/drivers/dri/i965 | |
| parent | e7496fed2a2428b704b8aeab5f42df8e6a2f7ae1 (diff) | |
i965/vec4: Detect and delete useless MOVs.
With NIR:
instructions in affected programs: 111508 -> 109193 (-2.08%)
helped: 507
Without NIR:
instructions in affected programs: 28763 -> 28474 (-1.00%)
helped: 186
Reviewed-by: Jason Ekstrand <[email protected]>
Reviewed-by: Kenneth Graunke <[email protected]>
Diffstat (limited to 'src/mesa/drivers/dri/i965')
| -rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4.cpp | 22 |
1 files changed, 22 insertions, 0 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp b/src/mesa/drivers/dri/i965/brw_vec4.cpp index ed49cd38987..6e52d79e053 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4.cpp @@ -1021,6 +1021,28 @@ vec4_visitor::opt_register_coalesce() inst->src[0].abs || inst->src[0].negate || inst->src[0].reladdr) continue; + /* Remove no-op MOVs */ + if (inst->dst.file == inst->src[0].file && + inst->dst.reg == inst->src[0].reg && + inst->dst.reg_offset == inst->src[0].reg_offset) { + bool is_nop_mov = true; + + for (unsigned c = 0; c < 4; c++) { + if ((inst->dst.writemask & (1 << c)) == 0) + continue; + + if (BRW_GET_SWZ(inst->src[0].swizzle, c) != c) { + is_nop_mov = false; + break; + } + } + + if (is_nop_mov) { + inst->remove(block); + continue; + } + } + bool to_mrf = (inst->dst.file == MRF); /* Can't coalesce this GRF if someone else was going to |