From 073b87653943b95fb286f41fbbff301230fb678c Mon Sep 17 00:00:00 2001 From: Francisco Jerez Date: Wed, 4 Jan 2023 12:53:06 -0800 Subject: [PATCH] intel/fs/xe2+: Don't special case SEL_EXEC in inferred_exec_pipe(). This is lowered to 32-bit integer execution type by the regioning lowering pass now, so the existing special casing is redudant for Gfx12 and buggy for Xe2+, since SEL_EXEC is now emitted without lowering for 64-bit integers. Reviewed-by: Lionel Landwerlin Part-of: --- src/intel/compiler/brw_fs_scoreboard.cpp | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/intel/compiler/brw_fs_scoreboard.cpp b/src/intel/compiler/brw_fs_scoreboard.cpp index 65b09813ac5..ab9e324c045 100644 --- a/src/intel/compiler/brw_fs_scoreboard.cpp +++ b/src/intel/compiler/brw_fs_scoreboard.cpp @@ -131,9 +131,7 @@ namespace { return TGL_PIPE_MATH; else if (inst->opcode == SHADER_OPCODE_MOV_INDIRECT || inst->opcode == SHADER_OPCODE_BROADCAST || - inst->opcode == SHADER_OPCODE_SHUFFLE || - (inst->opcode == SHADER_OPCODE_SEL_EXEC && - type_sz(inst->dst.type) > 4)) + inst->opcode == SHADER_OPCODE_SHUFFLE) return TGL_PIPE_INT; else if (inst->opcode == FS_OPCODE_PACK_HALF_2x16_SPLIT) return TGL_PIPE_FLOAT;