From: Ilia Mirkin Date: Wed, 1 Oct 2014 03:27:25 +0000 (-0400) Subject: freedreno/ir3: avoid fan-in sources referring to same instruction X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=3dd9a0d6fdea1ff5b1fe903fce206bf1d1515400;p=mesa.git freedreno/ir3: avoid fan-in sources referring to same instruction Since the RA has to be done s.t. each one gets its own (adjacent) register, it would complicate matters if instructions were allowed to be repeated. This enables copy-propagation use in situations where previously that might have happened. Signed-off-by: Ilia Mirkin Signed-off-by: Rob Clark --- diff --git a/src/gallium/drivers/freedreno/ir3/ir3_cp.c b/src/gallium/drivers/freedreno/ir3/ir3_cp.c index 73c2a27c6eb..83bcb7a742b 100644 --- a/src/gallium/drivers/freedreno/ir3/ir3_cp.c +++ b/src/gallium/drivers/freedreno/ir3/ir3_cp.c @@ -70,7 +70,7 @@ static void walk_children(struct ir3_instruction *instr, bool keep) static struct ir3_instruction * instr_cp_fanin(struct ir3_instruction *instr) { - unsigned i; + unsigned i, j; /* we need to handle fanin specially, to detect cases * when we need to keep a mov @@ -92,7 +92,15 @@ instr_cp_fanin(struct ir3_instruction *instr) if (is_meta(cand) && (cand->opc == OPC_META_FO)) cand = instr_cp(src->instr, true); - src->instr = cand; + /* we can't have 2 registers referring to the same instruction, so + * go through and check if any already refer to the candidate + * instruction. if so, don't do the propagation. + */ + for (j = 1; j < instr->regs_count; j++) + if (instr->regs[j]->instr == cand) + break; + if (j == instr->regs_count) + src->instr = cand; } }