diff options
author | Paul Berry <stereotype441@gmail.com> | 2013-03-29 13:34:51 -0700 |
---|---|---|
committer | Paul Berry <stereotype441@gmail.com> | 2013-04-02 12:24:08 -0700 |
commit | 7d4f1e6467a1b28ea6b0453ef9cd0254b3d57c19 (patch) | |
tree | 6562d56e7676ce280c6131e96527cb035b46da12 /src/glsl/ir_constant_expression.cpp | |
parent | 450950c57ac53d80a1c57f8ef5eb365db300fec2 (diff) | |
download | external_mesa3d-7d4f1e6467a1b28ea6b0453ef9cd0254b3d57c19.zip external_mesa3d-7d4f1e6467a1b28ea6b0453ef9cd0254b3d57c19.tar.gz external_mesa3d-7d4f1e6467a1b28ea6b0453ef9cd0254b3d57c19.tar.bz2 |
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
Diffstat (limited to 'src/glsl/ir_constant_expression.cpp')
-rw-r--r-- | src/glsl/ir_constant_expression.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/glsl/ir_constant_expression.cpp b/src/glsl/ir_constant_expression.cpp index c2d0dc4..c09e56a 100644 --- a/src/glsl/ir_constant_expression.cpp +++ b/src/glsl/ir_constant_expression.cpp @@ -1398,7 +1398,7 @@ ir_dereference_array::constant_referenced(struct hash_table *variable_context, return; } - const glsl_type *vt = substore->type; + const glsl_type *vt = array->type; if (vt->is_array()) { store = substore->get_array_element(index); offset = 0; |