From 7526ee36bcb89c867f4ec94f4585952195429841 Mon Sep 17 00:00:00 2001 From: Francisco Jerez Date: Wed, 18 Mar 2015 15:51:45 +0200 Subject: [PATCH] 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 --- src/mesa/drivers/dri/i965/brw_vec4_copy_propagation.cpp | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) 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; -- 2.30.2