[Mesa-dev] [PATCH 5/5] i965/fs: Remove hand-coded 64-bit packing optimizations
Jason Ekstrand
jason at jlekstrand.net
Wed Feb 15 07:29:51 UTC 2017
The optimization in unpack_64 is clearly subsumed with the opt_algebraic
optimizations in the previous commit. The pack optimization may not be
quite handled by opt_algebraic but opt_algebraic should get the really
bad cases. Also, it's been broken since it was merged and we've never
noticed so it must not be doing anything.
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 50 --------------------------------
1 file changed, 50 deletions(-)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index 91c14eb..9647300 100644
--- a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
+++ b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
@@ -1213,61 +1213,11 @@ fs_visitor::nir_emit_alu(const fs_builder &bld, nir_alu_instr *instr)
break;
case nir_op_pack_64_2x32_split:
- /* Optimize the common case where we are re-packing a double with
- * the result of a previous double unpack. In this case we can take the
- * 32-bit value to use in the re-pack from the original double and bypass
- * the unpack operation.
- */
- for (int i = 0; i < 2; i++) {
- if (!instr->src[i].src.is_ssa)
- continue;
-
- const nir_instr *parent_instr = instr->src[i].src.ssa->parent_instr;
- if (parent_instr->type == nir_instr_type_alu)
- continue;
-
- const nir_alu_instr *alu_parent = nir_instr_as_alu(parent_instr);
- if (alu_parent->op == nir_op_unpack_64_2x32_split_x ||
- alu_parent->op == nir_op_unpack_64_2x32_split_y)
- continue;
-
- if (!alu_parent->src[0].src.is_ssa)
- continue;
-
- op[i] = get_nir_src(alu_parent->src[0].src);
- op[i] = offset(retype(op[i], BRW_REGISTER_TYPE_DF), bld,
- alu_parent->src[0].swizzle[channel]);
- if (alu_parent->op == nir_op_unpack_64_2x32_split_y)
- op[i] = subscript(op[i], BRW_REGISTER_TYPE_UD, 1);
- else
- op[i] = subscript(op[i], BRW_REGISTER_TYPE_UD, 0);
- }
bld.emit(FS_OPCODE_PACK, result, op[0], op[1]);
break;
case nir_op_unpack_64_2x32_split_x:
case nir_op_unpack_64_2x32_split_y: {
- /* Optimize the common case where we are unpacking from a double we have
- * previously packed. In this case we can just bypass the pack operation
- * and source directly from its arguments.
- */
- unsigned index = (instr->op == nir_op_unpack_64_2x32_split_x) ? 0 : 1;
- if (instr->src[0].src.is_ssa) {
- nir_instr *parent_instr = instr->src[0].src.ssa->parent_instr;
- if (parent_instr->type == nir_instr_type_alu) {
- nir_alu_instr *alu_parent = nir_instr_as_alu(parent_instr);
- if (alu_parent->op == nir_op_pack_64_2x32_split &&
- alu_parent->src[index].src.is_ssa) {
- op[0] = retype(get_nir_src(alu_parent->src[index].src),
- BRW_REGISTER_TYPE_UD);
- op[0] =
- offset(op[0], bld, alu_parent->src[index].swizzle[channel]);
- bld.MOV(result, op[0]);
- break;
- }
- }
- }
-
if (instr->op == nir_op_unpack_64_2x32_split_x)
bld.MOV(result, subscript(op[0], BRW_REGISTER_TYPE_UD, 0));
else
--
2.5.0.400.gff86faf
More information about the mesa-dev
mailing list