fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-05 03:08:05 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

vc4: Use the SSA defs list for figuring out eligible MOVs for copy prop.

Browse source 
I thought I'd converted this over previously.  It was copy propagating
MOVs badly with the new destination packing flags.
This commit is contained in:
Eric Anholt 2015-08-21 09:22:32 -07:00
parent 20746c2e7d
commit 2002438c91
1 changed files with 10 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
src/gallium/drivers/vc4/vc4_opt_copy_propagation.c
View file

@ -39,21 +39,27 @@ qir_opt_copy_propagation(struct vc4_compile *c)
{
bool progress = false;
bool debug = false;
struct qreg *movs = calloc(c->num_temps, sizeof(struct qreg));
list_for_each_entry(struct qinst, inst, &c->instructions, link) {
for (int i = 0; i < qir_get_op_nsrc(inst->op); i++) {
int index = inst->src[i].index;
if (inst->src[i].file == QFILE_TEMP &&
(movs[index].file == QFILE_TEMP ||
movs[index].file == QFILE_UNIF)) {
c->defs[index] &&
c->defs[index]->op == QOP_MOV &&
(c->defs[index]->src[0].file == QFILE_TEMP ||
c->defs[index]->src[0].file == QFILE_UNIF)) {
/* If it has a pack, it shouldn't be an SSA
* def.
*/
assert(!c->defs[index]->dst.pack);
if (debug) {
fprintf(stderr, "Copy propagate: ");
qir_dump_inst(c, inst);
fprintf(stderr, "\n");
}
inst->src[i] = movs[index];
inst->src[i] = c->defs[index]->src[0];
if (debug) {
fprintf(stderr, "to: ");
@ -64,14 +70,6 @@ qir_opt_copy_propagation(struct vc4_compile *c)
progress = true;
}
}
if (inst->op == QOP_MOV &&
inst->dst.file == QFILE_TEMP &&
inst->src[0].file != QFILE_VPM) {
movs[inst->dst.index] = inst->src[0];
}
}
free(movs);
return progress;
}