From: Samuel Iglesias Gonsalvez <sigles...@igalia.com> Signed-off-by: Samuel Iglesias Gonsalvez <sigles...@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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev