Mesa (master): glsl: Strip arrayness from ir_type_dereference_variable too

Ian Romanick idr at kemper.freedesktop.org
Fri Sep 26 14:58:15 UTC 2014


Module: Mesa
Branch: master
Commit: f79583435b5fd0bc0c82d3538eeb5a74de3d9bff
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=f79583435b5fd0bc0c82d3538eeb5a74de3d9bff

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Wed Sep 10 15:19:43 2014 -0700

glsl: Strip arrayness from ir_type_dereference_variable too

If the thing being dereferenced is a record or an array of records, it
should be treated as row-major.  The ir_type_derference_record path
already does this, and I think I intended to do the same for this path
in b17a4d5d.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=83741
Cc: mesa-stable at lists.freedesktop.org

---

 src/glsl/lower_ubo_reference.cpp |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/glsl/lower_ubo_reference.cpp b/src/glsl/lower_ubo_reference.cpp
index f3920b3..43dd067 100644
--- a/src/glsl/lower_ubo_reference.cpp
+++ b/src/glsl/lower_ubo_reference.cpp
@@ -111,7 +111,7 @@ is_dereferenced_thing_row_major(const ir_dereference *deref)
          case GLSL_MATRIX_LAYOUT_COLUMN_MAJOR:
             return false;
          case GLSL_MATRIX_LAYOUT_ROW_MAJOR:
-            return matrix || deref->type->is_record();
+            return matrix || deref->type->without_array()->is_record();
          }
 
          unreachable("invalid matrix layout");




More information about the mesa-commit mailing list