From: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Date: Tue, 26 May 2020 14:21:44 +0000 (+0200)
Subject: aco: sign-extend input/identity for 16-bit subgroup ops on GFX6-GFX7
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=77f08982af8f0807ab489d36a5c137661e10624d;p=mesa.git

aco: sign-extend input/identity for 16-bit subgroup ops on GFX6-GFX7

16-bit subgroup ops are implemented with 32-bit instructions
on GFX6-GFX7.

Signed-off-by: Samuel Pitoiset <samuel.pitoiset@gmail.com>
Reviewed-by: Rhys Perry <pendingchaos02@gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5227>
---

diff --git a/src/amd/compiler/aco_lower_to_hw_instr.cpp b/src/amd/compiler/aco_lower_to_hw_instr.cpp
index df7b571c529..40d466904ef 100644
--- a/src/amd/compiler/aco_lower_to_hw_instr.cpp
+++ b/src/amd/compiler/aco_lower_to_hw_instr.cpp
@@ -590,6 +590,9 @@ void emit_reduction(lower_context *ctx, aco_opcode op, ReduceOp reduce_op, unsig
             sdwa->sel[0] = sdwa_uword;
          sdwa->dst_sel = sdwa_udword;
          bld.insert(std::move(sdwa));
+      } else if (ctx->program->chip_class == GFX6 || ctx->program->chip_class == GFX7) {
+         bld.vop3(aco_opcode::v_bfe_i32, Definition(PhysReg{tmp}, v1),
+                  Operand(PhysReg{tmp}, v1), Operand(0u), Operand(16u));
       }
    }