From ecc5b59a7069ab080a892e3f6a413ef62d3afee2 Mon Sep 17 00:00:00 2001 From: Rhys Perry Date: Mon, 17 Aug 2020 11:36:24 +0100 Subject: [PATCH] aco: don't allow destination opsel for v_cvt_pknorm MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit It doesn't make sense to do this. Signed-off-by: Rhys Perry Reviewed-by: Daniel Schürmann Part-of: --- src/amd/compiler/aco_ir.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/amd/compiler/aco_ir.cpp b/src/amd/compiler/aco_ir.cpp index f794125972d..a258447b662 100644 --- a/src/amd/compiler/aco_ir.cpp +++ b/src/amd/compiler/aco_ir.cpp @@ -310,14 +310,14 @@ bool can_use_opsel(chip_class chip, aco_opcode op, int idx, bool high) case aco_opcode::v_sub_i16: case aco_opcode::v_add_u16_e64: case aco_opcode::v_sub_u16_e64: - case aco_opcode::v_cvt_pknorm_i16_f16: - case aco_opcode::v_cvt_pknorm_u16_f16: case aco_opcode::v_lshlrev_b16_e64: case aco_opcode::v_lshrrev_b16_e64: case aco_opcode::v_ashrrev_i16_e64: case aco_opcode::v_mul_lo_u16_e64: return true; case aco_opcode::v_pack_b32_f16: + case aco_opcode::v_cvt_pknorm_i16_f16: + case aco_opcode::v_cvt_pknorm_u16_f16: return idx != -1; case aco_opcode::v_mad_u32_u16: case aco_opcode::v_mad_i32_i16: