Browse Source

mesa: Fix the Mesa IR copy propagation to not read past writes to the reg.

Fixes glsl-vs-post-increment-01.

Reviewed-by: José Fonseca <jfonseca@vmware.com>
tags/android-x86-2.2-r2
Eric Anholt 14 years ago
parent
commit
76857e8954
1 changed files with 40 additions and 7 deletions
  1. 40
    7
      src/mesa/program/ir_to_mesa.cpp

+ 40
- 7
src/mesa/program/ir_to_mesa.cpp View File

@@ -2742,13 +2742,46 @@ ir_to_mesa_visitor::copy_propagate(void)
/* Continuing the block, clear any written channels from
* the ACP.
*/
if (inst->dst_reg.file == PROGRAM_TEMPORARY) {
if (inst->dst_reg.reladdr) {
memset(acp, 0, sizeof(*acp) * this->next_temp * 4);
} else {
for (int i = 0; i < 4; i++) {
if (inst->dst_reg.writemask & (1 << i)) {
acp[4 * inst->dst_reg.index + i] = NULL;
if (inst->dst_reg.file == PROGRAM_TEMPORARY && inst->dst_reg.reladdr) {
/* Any temporary might be written, so no copy propagation
* across this instruction.
*/
memset(acp, 0, sizeof(*acp) * this->next_temp * 4);
} else if (inst->dst_reg.file == PROGRAM_OUTPUT &&
inst->dst_reg.reladdr) {
/* Any output might be written, so no copy propagation
* from outputs across this instruction.
*/
for (int r = 0; r < this->next_temp; r++) {
for (int c = 0; c < 4; c++) {
if (acp[4 * r + c]->src_reg[0].file == PROGRAM_OUTPUT)
acp[4 * r + c] = NULL;
}
}
} else if (inst->dst_reg.file == PROGRAM_TEMPORARY ||
inst->dst_reg.file == PROGRAM_OUTPUT) {
/* Clear where it's used as dst. */
if (inst->dst_reg.file == PROGRAM_TEMPORARY) {
for (int c = 0; c < 4; c++) {
if (inst->dst_reg.writemask & (1 << c)) {
acp[4 * inst->dst_reg.index + c] = NULL;
}
}
}

/* Clear where it's used as src. */
for (int r = 0; r < this->next_temp; r++) {
for (int c = 0; c < 4; c++) {
if (!acp[4 * r + c])
continue;

int src_chan = GET_SWZ(acp[4 * r + c]->src_reg[0].swizzle, c);

if (acp[4 * r + c]->src_reg[0].file == inst->dst_reg.file &&
acp[4 * r + c]->src_reg[0].index == inst->dst_reg.index &&
inst->dst_reg.writemask & (1 << src_chan))
{
acp[4 * r + c] = NULL;
}
}
}

Loading…
Cancel
Save