OSDN Git Service

glsl: Add missing check for whether an expression is an add operation
authorNeil Roberts <neil@linux.intel.com>
Sat, 4 Jul 2015 21:40:58 +0000 (22:40 +0100)
committerNeil Roberts <neil@linux.intel.com>
Mon, 6 Jul 2015 18:52:10 +0000 (11:52 -0700)
commit18039078e0254c7cb5e15b7186be05e2e4c10f38
treefa142afbb8051437d9d22914145d0ff00c73618f
parentd9ab95b365f058a46bc43a8cb96b6fff10a13faf
glsl: Add missing check for whether an expression is an add operation

There is a piece of code that is trying to match expressions of the
form (mul (floor (add (abs x) 0.5) (sign x))). However the check for
the add expression wasn't checking whether it had the expected
operation. It looks like this was just an oversight because it doesn't
match the pattern for the rest of the code snippet. The existing line
to check whether add_expr!=NULL was added as part of a coverity fix in
3384179f.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=91226
Cc: Matt Turner <mattst88@gmail.com>
Cc: "10.6" <mesa-stable@lists.freedesktop.org>
Reviewed-by: Matt Turner <mattst88@gmail.com>
src/glsl/opt_algebraic.cpp