From 09461082985cbc27b4cbfaf95d02ac9030a3d465 Mon Sep 17 00:00:00 2001
From: Ian Romanick <ian.d.romanick@intel.com>
Date: Fri, 18 Aug 2023 16:12:47 -0700
Subject: [PATCH] intel/fs: Simplify check in can_propagate_from

The larger predicate here already requires that inst->opcode must be
BRW_OPCODE_MOV, so it can't BRW_OPCODE_SEL. With that removed, the
other simplifications are pretty straight forward.

Reviewed-by: Kenneth Graunke <kenneth@whitecape.org>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/25091>
---
 src/intel/compiler/brw_fs_copy_propagation.cpp | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/intel/compiler/brw_fs_copy_propagation.cpp b/src/intel/compiler/brw_fs_copy_propagation.cpp
index f39107fc549..e365495960a 100644
--- a/src/intel/compiler/brw_fs_copy_propagation.cpp
+++ b/src/intel/compiler/brw_fs_copy_propagation.cpp
@@ -1140,8 +1140,7 @@ can_propagate_from(fs_inst *inst)
            inst->src[0].type == inst->dst.type &&
            !inst->saturate &&
            /* Subset of !is_partial_write() conditions. */
-           !((inst->predicate && inst->opcode != BRW_OPCODE_SEL) ||
-             !inst->dst.is_contiguous())) ||
+           !inst->predicate && inst->dst.is_contiguous()) ||
           is_identity_payload(FIXED_GRF, inst);
 }