From: Ian Romanick Date: Sat, 28 Feb 2015 16:26:37 +0000 (-0800) Subject: i965/fs: Change try_opt_frontfacing_ternary to eliminate asserts X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=4a53445b0d199489b2d1ae7d8654791e42b16804;p=mesa.git i965/fs: Change try_opt_frontfacing_ternary to eliminate asserts If we check for the case that is actually necessary, the asserts become superfluous. Signed-off-by: Ian Romanick Reviewed-by: Matt Turner --- diff --git a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp index 5d4b166a8fb..91dd212d0f9 100644 --- a/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp +++ b/src/mesa/drivers/dri/i965/brw_fs_visitor.cpp @@ -2735,11 +2735,8 @@ fs_visitor::try_opt_frontfacing_ternary(ir_if *ir) if (!then_rhs || !else_rhs) return false; - if ((then_rhs->is_one() || then_rhs->is_negative_one()) && - (else_rhs->is_one() || else_rhs->is_negative_one())) { - assert(then_rhs->is_one() == else_rhs->is_negative_one()); - assert(else_rhs->is_one() == then_rhs->is_negative_one()); - + if ((then_rhs->is_one() && else_rhs->is_negative_one()) || + (else_rhs->is_one() && then_rhs->is_negative_one())) { then_assign->lhs->accept(this); fs_reg dst = this->result; dst.type = BRW_REGISTER_TYPE_D;