aco: don't check usesModifiers for pseudo instructions
authorGeorg Lehmann <dadschoorse@gmail.com>
Fri, 24 Mar 2023 12:41:11 +0000 (13:41 +0100)
committerMarge Bot <emma+marge@anholt.net>
Mon, 27 Mar 2023 14:22:07 +0000 (14:22 +0000)
This can't happen.

Reviewed-by: Rhys Perry <pendingchaos02@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/22106>

src/amd/compiler/aco_optimizer.cpp

index cca6b37..68f72ca 100644 (file)
@@ -1803,8 +1803,6 @@ label_instruction(opt_ctx& ctx, aco_ptr<Instruction>& instr)
    case aco_opcode::p_as_uniform:
       if (instr->definitions[0].isFixed()) {
          /* don't copy-propagate copies into fixed registers */
-      } else if (instr->usesModifiers()) {
-         // TODO
       } else if (instr->operands[0].isConstant()) {
          ctx.info[instr->definitions[0].tempId()].set_constant(
             ctx.program->gfx_level, instr->operands[0].constantValue64());