fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-07 02:48:06 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

i965: Avoid applying negate to wrong MAD source.

Browse source 
For some given GLSL IR like (+ (neg x) (* 1.2 x)), the try_emit_mad
function would see that one of the +'s sources was a negate expression
and set mul_negate = true without confirming that it was actually a
multiply.

Cc: 10.5 <mesa-stable@lists.freedesktop.org>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89315
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=89095
Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
(cherry picked from commit d528907fd2)
[Emil Velikov: drop the changes in brw_vec4_visitor.cpp]
Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>

Conflicts:
	src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
	src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp
This commit is contained in:
Matt Turner 2015-02-27 10:59:17 -08:00 • committed by Emil Velikov
parent 0cd8e357e3
commit 31fcb21ef5
1 changed files with 13 additions and 15 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp
View file

@ -428,21 +428,16 @@ fs_visitor::try_emit_mad(ir_expression *ir)
if (ir->type != glsl_type::float_type)
return false;
ir_rvalue *nonmul = ir->operands[1];
ir_expression *mul = ir->operands[0]->as_expression();
ir_rvalue *nonmul;
ir_expression *mul;
bool mul_negate, mul_abs;
bool mul_negate = false, mul_abs = false;
if (mul && mul->operation == ir_unop_abs) {
mul = mul->operands[0]->as_expression();
mul_abs = true;
} else if (mul && mul->operation == ir_unop_neg) {
mul = mul->operands[0]->as_expression();
mul_negate = true;
}
for (int i = 0; i < 2; i++) {
mul_negate = false;
mul_abs = false;
if (!mul || mul->operation != ir_binop_mul) {
nonmul = ir->operands[0];
mul = ir->operands[1]->as_expression();
mul = ir->operands[i]->as_expression();
nonmul = ir->operands[1 - i];
if (mul && mul->operation == ir_unop_abs) {
mul = mul->operands[0]->as_expression();
@ -452,10 +447,13 @@ fs_visitor::try_emit_mad(ir_expression *ir)
mul_negate = true;
}
if (!mul || mul->operation != ir_binop_mul)
return false;
if (mul && mul->operation == ir_binop_mul)
break;
}
if (!mul || mul->operation != ir_binop_mul)
return false;
if (nonmul->as_constant() ||
mul->operands[0]->as_constant() ||
mul->operands[1]->as_constant())