AlexIndustrial/mesa
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

i965/fs: Change try_opt_frontfacing_ternary to eliminate asserts

Browse Source 
If we check for the case that is actually necessary, the asserts
become superfluous.

Signed-off-by: Ian Romanick <ian.d.romanick@intel.com>
Reviewed-by: Matt Turner <mattst88@gmail.com>
This commit is contained in:
Ian Romanick
2015-02-28 08:26:37 -08:00
parent ce3f46397d
commit 4a53445b0d
1 changed files with 2 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
+2 -5
View File
@@ -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;