Mesa (master): aco: sink get_alu_src() in bfe lowering
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Wed Aug 26 13:57:10 UTC 2020
Module: Mesa
Branch: master
Commit: d2cf6a8399e38f2c26564aeb6d0646c6c6198518
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=d2cf6a8399e38f2c26564aeb6d0646c6c6198518
Author: Rhys Perry <pendingchaos02 at gmail.com>
Date: Fri Aug 21 13:25:45 2020 +0100
aco: sink get_alu_src() in bfe lowering
Signed-off-by: Rhys Perry <pendingchaos02 at gmail.com>
Reviewed-by: Daniel Schürmann <daniel at schuermann.dev>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/6424>
---
src/amd/compiler/aco_instruction_selection.cpp | 13 ++++++++-----
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/src/amd/compiler/aco_instruction_selection.cpp b/src/amd/compiler/aco_instruction_selection.cpp
index f156acc535a..16c209a4daa 100644
--- a/src/amd/compiler/aco_instruction_selection.cpp
+++ b/src/amd/compiler/aco_instruction_selection.cpp
@@ -2757,21 +2757,24 @@ void visit_alu_instr(isel_context *ctx, nir_alu_instr *instr)
}
case nir_op_ubfe:
case nir_op_ibfe: {
- Temp base = get_alu_src(ctx, instr->src[0]);
- Temp offset = get_alu_src(ctx, instr->src[1]);
- Temp bits = get_alu_src(ctx, instr->src[2]);
-
if (dst.bytes() != 4)
unreachable("Unsupported BFE bit size");
if (dst.type() == RegType::sgpr) {
+ Temp base = get_alu_src(ctx, instr->src[0]);
+
nir_const_value* const_offset = nir_src_as_const_value(instr->src[1].src);
nir_const_value* const_bits = nir_src_as_const_value(instr->src[2].src);
if (const_offset && const_bits) {
uint32_t extract = (const_bits->u32 << 16) | (const_offset->u32 & 0x1f);
aco_opcode opcode = instr->op == nir_op_ubfe ? aco_opcode::s_bfe_u32 : aco_opcode::s_bfe_i32;
bld.sop2(opcode, Definition(dst), bld.def(s1, scc), base, Operand(extract));
- } else if (instr->op == nir_op_ubfe) {
+ break;
+ }
+
+ Temp offset = get_alu_src(ctx, instr->src[1]);
+ Temp bits = get_alu_src(ctx, instr->src[2]);
+ if (instr->op == nir_op_ubfe) {
Temp mask = bld.sop2(aco_opcode::s_bfm_b32, bld.def(s1), bits, offset);
Temp masked = bld.sop2(aco_opcode::s_and_b32, bld.def(s1), bld.def(s1, scc), base, mask);
bld.sop2(aco_opcode::s_lshr_b32, Definition(dst), bld.def(s1, scc), masked, offset);
More information about the mesa-commit
mailing list