From: Francisco Jerez <currojerez@riseup.net>
Date: Wed, 18 Mar 2015 13:51:45 +0000 (+0200)
Subject: i965/vec4: Add argument index and type checks to SEL saturate propagation.
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=7526ee36bcb89c867f4ec94f4585952195429841;p=mesa.git

i965/vec4: Add argument index and type checks to SEL saturate propagation.

SEL saturate propagation already implicitly relies on these
assumptions.

Reviewed-by: Abdiel Janulgue <abdiel.janulgue@linux.intel.com>
---

diff --git a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp
index 73b4f53f04b..478ae09b677 100644
--- a/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp
+++ b/src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp
@@ -343,7 +343,10 @@ try_copy_propagate(struct brw_context *brw, vec4_instruction *inst,
        */
       switch(inst->opcode) {
       case BRW_OPCODE_SEL:
-         if (inst->src[1].file != IMM ||
+         if (arg != 0 ||
+             inst->src[0].type != BRW_REGISTER_TYPE_F ||
+             inst->src[1].file != IMM ||
+             inst->src[1].type != BRW_REGISTER_TYPE_F ||
              inst->src[1].fixed_hw_reg.dw1.f < 0.0 ||
              inst->src[1].fixed_hw_reg.dw1.f > 1.0) {
             return false;