[Mesa-dev] [PATCH 44/74] glsl: add std430 interface packing support to ssbo writes and unsized array length

Iago Toral Quiroga itoral at igalia.com
Thu May 14 07:06:47 PDT 2015


From: Samuel Iglesias Gonsalvez <siglesias at igalia.com>

Signed-off-by: Samuel Iglesias Gonsalvez <siglesias at igalia.com>
---
 src/glsl/lower_ubo_reference.cpp | 65 ++++++++++++++++++++++++++++++----------
 1 file changed, 50 insertions(+), 15 deletions(-)

diff --git a/src/glsl/lower_ubo_reference.cpp b/src/glsl/lower_ubo_reference.cpp
index a5aaf07..55f3d87 100644
--- a/src/glsl/lower_ubo_reference.cpp
+++ b/src/glsl/lower_ubo_reference.cpp
@@ -163,7 +163,7 @@ public:
                         unsigned write_mask);
    void emit_ssbo_writes(ir_dereference *deref, ir_variable *base_offset,
                          unsigned int deref_offset, bool row_major,
-                         int matrix_columns, unsigned write_mask);
+                         int matrix_columns, bool is_std430, unsigned write_mask);
    ir_ssbo_store *ssbo_write(ir_rvalue *deref, ir_rvalue *offset,
                              unsigned write_mask);
 
@@ -343,8 +343,15 @@ lower_ubo_reference_visitor::setup_for_load_or_write(ir_variable *var,
             const bool array_row_major =
                is_dereferenced_thing_row_major(deref_array);
 
-            array_stride = deref_array->type->std140_size(array_row_major);
-            array_stride = glsl_align(array_stride, 16);
+            /* The array type will give the correct interface packing
+             * information
+             */
+            if (deref_array->array->type->interface_packing == GLSL_INTERFACE_PACKING_STD430) {
+               array_stride = deref_array->type->std430_size(array_row_major);
+            } else {
+               array_stride = deref_array->type->std140_size(array_row_major);
+               array_stride = glsl_align(array_stride, 16);
+            }
          }
 
          ir_rvalue *array_index = deref_array->array_index;
@@ -380,7 +387,12 @@ lower_ubo_reference_visitor::setup_for_load_or_write(ir_variable *var,
 
             ralloc_free(field_deref);
 
-            unsigned field_align = type->std140_base_alignment(field_row_major);
+            unsigned field_align = 0;
+
+            if (struct_type->interface_packing == GLSL_INTERFACE_PACKING_STD430)
+               field_align = type->std430_base_alignment(field_row_major);
+            else
+               field_align = type->std140_base_alignment(field_row_major);
 
             intra_struct_offset = glsl_align(intra_struct_offset, field_align);
 
@@ -388,7 +400,10 @@ lower_ubo_reference_visitor::setup_for_load_or_write(ir_variable *var,
                        deref_record->field) == 0)
                break;
 
-            intra_struct_offset += type->std140_size(field_row_major);
+            if (struct_type->interface_packing == GLSL_INTERFACE_PACKING_STD430)
+               intra_struct_offset += type->std430_size(field_row_major);
+            else
+               intra_struct_offset += type->std140_size(field_row_major);
 
             /* If the field just examined was itself a structure, apply rule
              * #9:
@@ -633,6 +648,7 @@ lower_ubo_reference_visitor::emit_ssbo_writes(ir_dereference *deref,
                                               unsigned int deref_offset,
                                               bool row_major,
                                               int matrix_columns,
+                                              bool is_std430,
                                               unsigned write_mask)
 {
    if (deref->type->is_record()) {
@@ -651,7 +667,7 @@ lower_ubo_reference_visitor::emit_ssbo_writes(ir_dereference *deref,
 
          emit_ssbo_writes(field_deref, base_offset,
                           deref_offset + field_offset,
-                          row_major, 1,
+                          row_major, 1, is_std430,
                           writemask_for_size(field_deref->type->vector_elements));
 
 	      field_offset += field->type->std140_size(row_major);
@@ -660,7 +676,8 @@ lower_ubo_reference_visitor::emit_ssbo_writes(ir_dereference *deref,
    }
 
    if (deref->type->is_array()) {
-      unsigned array_stride =
+      unsigned array_stride = is_std430 ?
+         deref->type->fields.array->std430_size(row_major) :
          glsl_align(deref->type->fields.array->std140_size(row_major), 16);
 
       for (unsigned i = 0; i < deref->type->length; i++) {
@@ -670,7 +687,7 @@ lower_ubo_reference_visitor::emit_ssbo_writes(ir_dereference *deref,
                                               element);
          emit_ssbo_writes(element_deref, base_offset,
                           deref_offset + i * array_stride,
-                          row_major, 1,
+                          row_major, 1, is_std430,
                           writemask_for_size(element_deref->type->vector_elements));
       }
       return;
@@ -689,7 +706,7 @@ lower_ubo_reference_visitor::emit_ssbo_writes(ir_dereference *deref,
             int size_mul = deref->type->is_double() ? 8 : 4;
             emit_ssbo_writes(col_deref, base_offset,
                              deref_offset + i * size_mul,
-                             row_major, deref->type->matrix_columns,
+                             row_major, deref->type->matrix_columns, is_std430,
                              writemask_for_size(col_deref->type->vector_elements));
          } else {
             /* std140 always rounds the stride of arrays (and matrices) to a
@@ -698,9 +715,15 @@ lower_ubo_reference_visitor::emit_ssbo_writes(ir_dereference *deref,
              */
             int size_mul = (deref->type->is_double() &&
                             deref->type->vector_elements > 2) ? 32 : 16;
+            /* This is not the case for std430 matrices of two vector components */
+            if (is_std430 && deref->type->vector_elements <= 2) {
+               size_mul = deref->type->is_double() ?
+                           8 * deref->type->vector_elements :
+                           4 * deref->type->vector_elements;
+            }
             emit_ssbo_writes(col_deref, base_offset,
                              deref_offset + i * size_mul,
-                             row_major, deref->type->matrix_columns,
+                             row_major, deref->type->matrix_columns, is_std430,
                              writemask_for_size(col_deref->type->vector_elements));
          }
       }
@@ -751,6 +774,7 @@ lower_ubo_reference_visitor::write_to_memory(ir_dereference *deref,
    bool row_major;
    int matrix_columns;
    bool is_buffer;
+   bool is_std430 = var->type->interface_packing == GLSL_INTERFACE_PACKING_STD430;
 
    /* Compute the offset to the start if the dereference as well as other
     * information we need to configure the write
@@ -773,7 +797,9 @@ lower_ubo_reference_visitor::write_to_memory(ir_dereference *deref,
 
    deref = new(mem_ctx) ir_dereference_variable(write_var);
    emit_ssbo_writes(deref, write_offset, const_offset,
-                    row_major, matrix_columns, write_mask);
+                    row_major, matrix_columns,
+                    is_std430,
+                    write_mask);
 }
 
 ir_visitor_status
@@ -882,8 +908,12 @@ lower_ubo_reference_visitor::calculate_unsized_array_stride(ir_dereference *dere
       const bool array_row_major =
          is_dereferenced_thing_row_major(deref_var);
 
-      array_stride = unsized_array_type->std140_size(array_row_major);
-      array_stride = glsl_align(array_stride, 16);
+      if (deref->type->interface_packing == GLSL_INTERFACE_PACKING_STD430) {
+         array_stride = unsized_array_type->std430_size(array_row_major);
+      } else {
+         array_stride = unsized_array_type->std140_size(array_row_major);
+         array_stride = glsl_align(array_stride, 16);
+      }
       break;
    }
    case ir_type_dereference_record:
@@ -898,8 +928,13 @@ lower_ubo_reference_visitor::calculate_unsized_array_stride(ir_dereference *dere
 
       const bool array_row_major =
          is_dereferenced_thing_row_major(deref_record);
-      array_stride = unsized_array_type->std140_size(array_row_major);
-      array_stride = glsl_align(array_stride, 16);
+
+      if (deref->type->interface_packing == GLSL_INTERFACE_PACKING_STD430) {
+         array_stride = unsized_array_type->std430_size(array_row_major);
+      } else {
+         array_stride = unsized_array_type->std140_size(array_row_major);
+         array_stride = glsl_align(array_stride, 16);
+      }
       break;
    }
    default:
-- 
1.9.1



More information about the mesa-dev mailing list