glsl: Fix array indexing when constant folding built-in functions.
authorPaul Berry <stereotype441@gmail.com>
Fri, 29 Mar 2013 20:34:51 +0000 (13:34 -0700)
committerPaul Berry <stereotype441@gmail.com>
Tue, 2 Apr 2013 19:24:08 +0000 (12:24 -0700)
commit7d4f1e6467a1b28ea6b0453ef9cd0254b3d57c19
tree6562d56e7676ce280c6131e96527cb035b46da12
parent450950c57ac53d80a1c57f8ef5eb365db300fec2
glsl: Fix array indexing when constant folding built-in functions.

Mesa constant-folds built-in functions by using a miniature GLSL
interpreter (see
ir_function_signature::constant_expression_evaluate_expression_list()).
This interpreter had a bug in its handling of array indexing, which
caused expressions like "m[i][j]" (where m is a matrix) to be handled
incorrectly.  Specifically, it incorrectly treated j as indexing into
the whole matrix (rather than indexing just into the vector m[i]); as
a result the offset computed for m[i] was lost and m[i][j] was treated
as m[j][0].

Fixes piglit tests inverse-mat[234].{vert,frag}.

NOTE: This is a candidate for the 9.1 and 9.0 branches.

Reviewed-by: Ian Romanick <ian.d.romanick@intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=57436
src/glsl/ir_constant_expression.cpp