intel/fs/gfx12+: Drop redundant handling of SHADER_OPCODE_BROADCAST in exec pipe...
authorFrancisco Jerez <currojerez@riseup.net>
Wed, 4 Jan 2023 20:53:18 +0000 (12:53 -0800)
committerMarge Bot <emma+marge@anholt.net>
Thu, 19 Jan 2023 06:14:03 +0000 (06:14 +0000)
Commit c80c0ed9433035c2f4dc6 introduced handling of
SHADER_OPCODE_BROADCAST into inferred_exec_pipe(), but it was already
being handled, drop the redundant handling.  Shouldn't lead to any
functional changes.

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/20543>

src/intel/compiler/brw_fs_scoreboard.cpp

index 64db8f9..2418357 100644 (file)
@@ -133,9 +133,6 @@ namespace {
                (inst->opcode == SHADER_OPCODE_SEL_EXEC &&
                 type_sz(inst->dst.type) > 4))
          return TGL_PIPE_INT;
-      else if (inst->opcode == SHADER_OPCODE_BROADCAST &&
-               !devinfo->has_64bit_float && type_sz(t) >= 8)
-         return TGL_PIPE_INT;
       else if (inst->opcode == FS_OPCODE_PACK_HALF_2x16_SPLIT)
          return TGL_PIPE_FLOAT;
       else if (type_sz(inst->dst.type) >= 8 || type_sz(t) >= 8 ||