diff --git a/.pick_status.json b/.pick_status.json index 805a72e3058..7350f387f2c 100644 --- a/.pick_status.json +++ b/.pick_status.json @@ -2578,7 +2578,7 @@ "description": "aco/optimizer_postRA: Don't assume all operand registers were written by same instr.", "nominated": true, "nomination_type": 1, - "resolution": 0, + "resolution": 1, "main_sha": null, "because_sha": "0e4747d3fb7ec15f8c1d6b971b1352249e7d95c6" }, diff --git a/src/amd/compiler/aco_optimizer_postRA.cpp b/src/amd/compiler/aco_optimizer_postRA.cpp index 0b4a3a71ea7..a7f0c78b367 100644 --- a/src/amd/compiler/aco_optimizer_postRA.cpp +++ b/src/amd/compiler/aco_optimizer_postRA.cpp @@ -127,16 +127,7 @@ last_writer_idx(pr_opt_ctx& ctx, const Operand& op) if (op.isConstant() || op.isUndefined()) return const_or_undef; - assert(op.physReg().reg() < max_reg_cnt); - Idx instr_idx = ctx.instr_idx_by_regs[ctx.current_block->index][op.physReg().reg()]; - -#ifndef NDEBUG - /* Debug mode: */ - instr_idx = last_writer_idx(ctx, op.physReg(), op.regClass()); - assert(instr_idx != written_by_multiple_instrs); -#endif - - return instr_idx; + return last_writer_idx(ctx, op.physReg(), op.regClass()); } bool