fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-06-03 17:38:25 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

nv50_ir_ra: Add constraint movs for split-of-split

Browse source 
This handles this case similarly to the merge-of-merge fix from c86563c2

Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/24541>
This commit is contained in:
M Henning 2023-08-06 16:11:15 -04:00 • committed by Marge Bot
parent 7e653b2eb1
commit a46942ce04
1 changed files with 16 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/gallium/drivers/nouveau/codegen/nv50_ir_ra.cpp
View file

@ -2435,11 +2435,24 @@ RegAlloc::InsertConstraintsPass::insertConstraintMoves()
Instruction *cst = *it;
Instruction *mov;
if (cst->op == OP_SPLIT && false) {
// spilling splits is annoying, just make sure they're separate
// Note tha RA cannot handle when a split/merge/union consumes the result
// of a split/merge/union, so we must always insert a constraint move in
// this case
if (cst->op == OP_SPLIT) {
for (int d = 0; cst->defExists(d); ++d) {
if (!cst->getDef(d)->refCount())
bool needs_mov = false;
for (ValueRef *consumer_ref: cst->getDef(d)->uses) {
operation consumer_op = consumer_ref->getInsn()->op;
needs_mov = needs_mov ||
consumer_op == OP_SPLIT ||
consumer_op == OP_MERGE ||
consumer_op == OP_UNION;
if (needs_mov)
break;
}
if (!needs_mov)
continue;
LValue *lval = new_LValue(func, cst->def(d).getFile());
const uint8_t size = cst->def(d).getSize();
lval->reg.size = size;
@ -2451,7 +2464,6 @@ RegAlloc::InsertConstraintsPass::insertConstraintMoves()
cst->bb->insertAfter(cst, mov);
cst->getSrc(0)->asLValue()->noSpill = 1;
mov->getSrc(0)->asLValue()->noSpill = 1;
}
} else
if (cst->op == OP_MERGE || cst->op == OP_UNION) {