diff options
author | Francisco Jerez <[email protected]> | 2015-02-05 22:39:33 +0200 |
---|---|---|
committer | Francisco Jerez <[email protected]> | 2015-02-10 19:09:24 +0200 |
commit | de666fc102b805707c7033b203c5b76ccbbcef8d (patch) | |
tree | b6adbf07c41ffd69499bd5a3f62b501dbd586bca /src/mesa | |
parent | 8ad486077e122c19b603750e19dd678bb7793d5b (diff) |
i965/vec4: Fix the scheduler to take into account reads and writes of multiple registers.
v2: Avoid nested ternary operators in vec4_instruction::regs_read(). (Matt)
Reviewed-by: Matt Turner <[email protected]>
Diffstat (limited to 'src/mesa')
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_ir_vec4.h | 1 | ||||
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp | 15 | ||||
-rw-r--r-- | src/mesa/drivers/dri/i965/brw_vec4.cpp | 18 |
3 files changed, 29 insertions, 5 deletions
diff --git a/src/mesa/drivers/dri/i965/brw_ir_vec4.h b/src/mesa/drivers/dri/i965/brw_ir_vec4.h index f11a2d21d1e..941086ffd28 100644 --- a/src/mesa/drivers/dri/i965/brw_ir_vec4.h +++ b/src/mesa/drivers/dri/i965/brw_ir_vec4.h @@ -171,6 +171,7 @@ public: unsigned sol_vertex; /**< gen6: used for setting dst index in SVB header */ bool is_send_from_grf(); + unsigned regs_read(unsigned arg) const; bool can_reswizzle(int dst_writemask, int swizzle, int swizzle_mask); void reswizzle(int dst_writemask, int swizzle); bool can_do_source_mods(struct brw_context *brw); diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp index 88feeca8f83..2b22b2c45fc 100644 --- a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp +++ b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp @@ -1063,7 +1063,8 @@ vec4_instruction_scheduler::calculate_deps() /* read-after-write deps. */ for (int i = 0; i < 3; i++) { if (inst->src[i].file == GRF) { - add_dep(last_grf_write[inst->src[i].reg], n); + for (unsigned j = 0; j < inst->regs_read(i); ++j) + add_dep(last_grf_write[inst->src[i].reg + j], n); } else if (inst->src[i].file == HW_REG && (inst->src[i].fixed_hw_reg.file == BRW_GENERAL_REGISTER_FILE)) { @@ -1103,8 +1104,10 @@ vec4_instruction_scheduler::calculate_deps() /* write-after-write deps. */ if (inst->dst.file == GRF) { - add_dep(last_grf_write[inst->dst.reg], n); - last_grf_write[inst->dst.reg] = n; + for (unsigned j = 0; j < inst->regs_written; ++j) { + add_dep(last_grf_write[inst->dst.reg + j], n); + last_grf_write[inst->dst.reg + j] = n; + } } else if (inst->dst.file == MRF) { add_dep(last_mrf_write[inst->dst.reg], n); last_mrf_write[inst->dst.reg] = n; @@ -1156,7 +1159,8 @@ vec4_instruction_scheduler::calculate_deps() /* write-after-read deps. */ for (int i = 0; i < 3; i++) { if (inst->src[i].file == GRF) { - add_dep(n, last_grf_write[inst->src[i].reg]); + for (unsigned j = 0; j < inst->regs_read(i); ++j) + add_dep(n, last_grf_write[inst->src[i].reg + j]); } else if (inst->src[i].file == HW_REG && (inst->src[i].fixed_hw_reg.file == BRW_GENERAL_REGISTER_FILE)) { @@ -1194,7 +1198,8 @@ vec4_instruction_scheduler::calculate_deps() * can mark this as WAR dependency. */ if (inst->dst.file == GRF) { - last_grf_write[inst->dst.reg] = n; + for (unsigned j = 0; j < inst->regs_written; ++j) + last_grf_write[inst->dst.reg + j] = n; } else if (inst->dst.file == MRF) { last_mrf_write[inst->dst.reg] = n; } else if (inst->dst.file == HW_REG && diff --git a/src/mesa/drivers/dri/i965/brw_vec4.cpp b/src/mesa/drivers/dri/i965/brw_vec4.cpp index 48b2a6902f6..cdd8a446286 100644 --- a/src/mesa/drivers/dri/i965/brw_vec4.cpp +++ b/src/mesa/drivers/dri/i965/brw_vec4.cpp @@ -262,6 +262,24 @@ vec4_instruction::is_send_from_grf() } } +unsigned +vec4_instruction::regs_read(unsigned arg) const +{ + if (src[arg].file == BAD_FILE) + return 0; + + switch (opcode) { + case SHADER_OPCODE_SHADER_TIME_ADD: + return arg == 0 ? mlen : 1; + + case VS_OPCODE_PULL_CONSTANT_LOAD_GEN7: + return arg == 1 ? mlen : 1; + + default: + return 1; + } +} + bool vec4_instruction::can_do_source_mods(struct brw_context *brw) { |