aco/assembler: Fix s_bitreplicate_b64_b32 on GFX9.
authorGeorg Lehmann <dadschoorse@gmail.com>
Thu, 7 Jul 2022 20:10:09 +0000 (22:10 +0200)
committerMarge Bot <emma+marge@anholt.net>
Fri, 8 Jul 2022 10:09:19 +0000 (10:09 +0000)
This seems to be a relic from before aco added per generation opcodes.

Signed-off-by: Georg Lehmann <dadschoorse@gmail.com>
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17405>

src/amd/compiler/aco_assembler.cpp

index 789ea65..4bccd57 100644 (file)
@@ -149,10 +149,6 @@ emit_instruction(asm_context& ctx, std::vector<uint32_t>& out, Instruction* inst
    }
    case Format::SOP1: {
       uint32_t encoding = (0b101111101 << 23);
-      if (opcode >= 55 && ctx.gfx_level <= GFX9) {
-         assert(ctx.gfx_level == GFX9 && opcode < 60);
-         opcode = opcode - 4;
-      }
       encoding |= !instr->definitions.empty() ? instr->definitions[0].physReg() << 16 : 0;
       encoding |= opcode << 8;
       encoding |= !instr->operands.empty() ? instr->operands[0].physReg() : 0;