diff --git a/src/amd/compiler/aco_register_allocation.cpp b/src/amd/compiler/aco_register_allocation.cpp index 9e3d796dc2f..3b4100cddd2 100644 --- a/src/amd/compiler/aco_register_allocation.cpp +++ b/src/amd/compiler/aco_register_allocation.cpp @@ -881,7 +881,15 @@ void handle_pseudo(ra_ctx& ctx, break; } } - if (!writes_sgpr) + /* if all operands are constant, no need to care either */ + bool reads_sgpr = false; + for (Operand& op : instr->operands) { + if (op.isTemp() && op.getTemp().type() == RegType::sgpr) { + reads_sgpr = true; + break; + } + } + if (!(writes_sgpr && reads_sgpr)) return; Pseudo_instruction *pi = (Pseudo_instruction *)instr; diff --git a/src/amd/compiler/aco_spill.cpp b/src/amd/compiler/aco_spill.cpp index 276dcbd7c5b..952094f40d0 100644 --- a/src/amd/compiler/aco_spill.cpp +++ b/src/amd/compiler/aco_spill.cpp @@ -265,7 +265,7 @@ aco_ptr do_reload(spill_ctx& ctx, Temp tmp, Temp new_name, uint32_t } else if (instr->format == Format::SOP1) { res.reset(create_instruction(instr->opcode, instr->format, instr->operands.size(), instr->definitions.size())); } else if (instr->format == Format::PSEUDO) { - res.reset(create_instruction(instr->opcode, instr->format, instr->operands.size(), instr->definitions.size())); + res.reset(create_instruction(instr->opcode, instr->format, instr->operands.size(), instr->definitions.size())); } for (unsigned i = 0; i < instr->operands.size(); i++) { res->operands[i] = instr->operands[i];