From: Caio Marcelo de Oliveira Filho Date: Fri, 3 May 2019 19:42:39 +0000 (-0700) Subject: spirv: Add vtn_mode_to_address_format() X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=672a3f42d91534a4a77ba5da10a66871734d7953;p=mesa.git spirv: Add vtn_mode_to_address_format() Handles all the modes and we can use it in combination with nir_address_format_to_glsl_type() to replace the vtn_ptr_type_for_mode() helper. Since the new helper is more generic, moved the assertions from the old one to the call sites. Reviewed-by: Jason Ekstrand Reviewed-by: Bas Nieuwenhuizen --- diff --git a/src/compiler/spirv/vtn_private.h b/src/compiler/spirv/vtn_private.h index 55cd0c2786c..a66f2c02fbd 100644 --- a/src/compiler/spirv/vtn_private.h +++ b/src/compiler/spirv/vtn_private.h @@ -813,6 +813,9 @@ enum vtn_variable_mode vtn_storage_class_to_mode(struct vtn_builder *b, struct vtn_type *interface_type, nir_variable_mode *nir_mode_out); +nir_address_format vtn_mode_to_address_format(struct vtn_builder *b, + enum vtn_variable_mode); + static inline uint32_t vtn_align_u32(uint32_t v, uint32_t a) { diff --git a/src/compiler/spirv/vtn_variables.c b/src/compiler/spirv/vtn_variables.c index 0d7a6b54766..11d3111c516 100644 --- a/src/compiler/spirv/vtn_variables.c +++ b/src/compiler/spirv/vtn_variables.c @@ -96,23 +96,6 @@ vk_desc_type_for_mode(struct vtn_builder *b, enum vtn_variable_mode mode) } } -static const struct glsl_type * -vtn_ptr_type_for_mode(struct vtn_builder *b, enum vtn_variable_mode mode) -{ - nir_address_format addr_format; - switch (mode) { - case vtn_variable_mode_ubo: - addr_format = b->options->ubo_addr_format; - break; - case vtn_variable_mode_ssbo: - addr_format = b->options->ssbo_addr_format; - break; - default: - vtn_fail("Invalid mode for vulkan_resource_index"); - } - return nir_address_format_to_glsl_type(addr_format); -} - static nir_ssa_def * vtn_variable_resource_index(struct vtn_builder *b, struct vtn_variable *var, nir_ssa_def *desc_array_index) @@ -130,9 +113,14 @@ vtn_variable_resource_index(struct vtn_builder *b, struct vtn_variable *var, nir_intrinsic_set_binding(instr, var->binding); nir_intrinsic_set_desc_type(instr, vk_desc_type_for_mode(b, var->mode)); + vtn_fail_if(var->mode != vtn_variable_mode_ubo && + var->mode != vtn_variable_mode_ssbo, + "Invalid mode for vulkan_resource_index"); + + nir_address_format addr_format = vtn_mode_to_address_format(b, var->mode); const struct glsl_type *index_type = b->options->lower_ubo_ssbo_access_to_offsets ? - glsl_uint_type() : vtn_ptr_type_for_mode(b, var->mode); + glsl_uint_type() : nir_address_format_to_glsl_type(addr_format); instr->num_components = glsl_get_vector_elements(index_type); nir_ssa_dest_init(&instr->instr, &instr->dest, instr->num_components, @@ -153,9 +141,13 @@ vtn_resource_reindex(struct vtn_builder *b, enum vtn_variable_mode mode, instr->src[1] = nir_src_for_ssa(offset_index); nir_intrinsic_set_desc_type(instr, vk_desc_type_for_mode(b, mode)); + vtn_fail_if(mode != vtn_variable_mode_ubo && mode != vtn_variable_mode_ssbo, + "Invalid mode for vulkan_resource_reindex"); + + nir_address_format addr_format = vtn_mode_to_address_format(b, mode); const struct glsl_type *index_type = b->options->lower_ubo_ssbo_access_to_offsets ? - glsl_uint_type() : vtn_ptr_type_for_mode(b, mode); + glsl_uint_type() : nir_address_format_to_glsl_type(addr_format); instr->num_components = glsl_get_vector_elements(index_type); nir_ssa_dest_init(&instr->instr, &instr->dest, instr->num_components, @@ -175,7 +167,12 @@ vtn_descriptor_load(struct vtn_builder *b, enum vtn_variable_mode mode, desc_load->src[0] = nir_src_for_ssa(desc_index); nir_intrinsic_set_desc_type(desc_load, vk_desc_type_for_mode(b, mode)); - const struct glsl_type *ptr_type = vtn_ptr_type_for_mode(b, mode); + vtn_fail_if(mode != vtn_variable_mode_ubo && mode != vtn_variable_mode_ssbo, + "Invalid mode for load_vulkan_descriptor"); + + nir_address_format addr_format = vtn_mode_to_address_format(b, mode); + const struct glsl_type *ptr_type = + nir_address_format_to_glsl_type(addr_format); desc_load->num_components = glsl_get_vector_elements(ptr_type); nir_ssa_dest_init(&desc_load->instr, &desc_load->dest, @@ -1794,6 +1791,43 @@ vtn_storage_class_to_mode(struct vtn_builder *b, return mode; } +nir_address_format +vtn_mode_to_address_format(struct vtn_builder *b, enum vtn_variable_mode mode) +{ + switch (mode) { + case vtn_variable_mode_ubo: + return b->options->ubo_addr_format; + + case vtn_variable_mode_ssbo: + return b->options->ssbo_addr_format; + + case vtn_variable_mode_phys_ssbo: + return b->options->phys_ssbo_addr_format; + + case vtn_variable_mode_push_constant: + return b->options->push_const_addr_format; + + case vtn_variable_mode_workgroup: + return b->options->shared_addr_format; + + case vtn_variable_mode_cross_workgroup: + return b->options->global_addr_format; + + case vtn_variable_mode_function: + if (b->physical_ptrs) + return b->options->temp_addr_format; + /* Fall through. */ + + case vtn_variable_mode_private: + case vtn_variable_mode_uniform: + case vtn_variable_mode_input: + case vtn_variable_mode_output: + return nir_address_format_logical; + } + + unreachable("Invalid variable mode"); +} + nir_ssa_def * vtn_pointer_to_ssa(struct vtn_builder *b, struct vtn_pointer *ptr) {