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

aco/spill: don't allocate extra spill_id for phi operands in add_coupling_code()

Browse source 
We can re-use the phi definition's spill_id.

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27774>
This commit is contained in:
Daniel Schürmann 2024-02-28 10:41:12 +01:00 • committed by Marge Bot
parent b1468fc952
commit 1a1d6aef56
1 changed files with 4 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/amd/compiler/aco_spill.cpp
View file

@ -911,17 +911,14 @@ add_coupling_code(spill_ctx& ctx, Block* block, unsigned block_idx)
}
}
uint32_t spill_id = ctx.allocate_spill_id(phi->definitions[0].regClass());
/* add interferences and affinity */
/* add interferences */
for (std::pair<Temp, uint32_t> pair : ctx.spills_exit[pred_idx])
ctx.add_interference(spill_id, pair.second);
ctx.add_affinity(def_spill_id, spill_id);
ctx.add_interference(def_spill_id, pair.second);
aco_ptr<Pseudo_instruction> spill{
create_instruction<Pseudo_instruction>(aco_opcode::p_spill, Format::PSEUDO, 2, 0)};
spill->operands[0] = spill_op;
spill->operands[1] = Operand::c32(spill_id);
spill->operands[1] = Operand::c32(def_spill_id);
Block& pred = ctx.program->blocks[pred_idx];
unsigned idx = pred.instructions.size();
do {
@ -934,7 +931,7 @@ add_coupling_code(spill_ctx& ctx, Block* block, unsigned block_idx)
/* Add the original name to predecessor's spilled variables */
if (spill_op.isTemp())
ctx.spills_exit[pred_idx][phi->operands[i].getTemp()] = spill_id;
ctx.spills_exit[pred_idx][phi->operands[i].getTemp()] = def_spill_id;
}
/* remove phi from instructions */