[PATCH vkd3d v2 2/6] vkd3d-shader: Include register ranges in descriptor binding error messages.
Conor McCarthy
conor.mccarthy6 at bigpond.com
Sun Jul 18 23:47:41 CDT 2021
From: Conor McCarthy <cmccarthy at codeweavers.com>
Signed-off-by: Conor McCarthy <cmccarthy at codeweavers.com>
---
libs/vkd3d-shader/spirv.c | 35 +++++++++++++++++++++++++++++++----
1 file changed, 31 insertions(+), 4 deletions(-)
diff --git a/libs/vkd3d-shader/spirv.c b/libs/vkd3d-shader/spirv.c
index 6576aca4..991ff82f 100644
--- a/libs/vkd3d-shader/spirv.c
+++ b/libs/vkd3d-shader/spirv.c
@@ -2212,6 +2212,8 @@ struct vkd3d_dxbc_compiler
struct vkd3d_shader_spec_constant *spec_constants;
size_t spec_constants_size;
enum vkd3d_shader_compile_option_formatting_flags formatting;
+
+ struct vkd3d_string_buffer_cache string_buffers;
};
static bool is_control_point_phase(const struct vkd3d_shader_phase *phase)
@@ -2334,6 +2336,8 @@ struct vkd3d_dxbc_compiler *vkd3d_dxbc_compiler_create(const struct vkd3d_shader
compiler->scan_descriptor_info = scan_descriptor_info;
+ vkd3d_string_buffer_cache_init(&compiler->string_buffers);
+
vkd3d_dxbc_compiler_emit_initial_declarations(compiler);
return compiler;
@@ -2460,6 +2464,26 @@ static void VKD3D_PRINTF_FUNC(3, 4) vkd3d_dxbc_compiler_error(struct vkd3d_dxbc_
compiler->failed = true;
}
+static struct vkd3d_string_buffer *vkd3d_dxbc_compiler_binding_error(struct vkd3d_dxbc_compiler *compiler,
+ enum vkd3d_shader_descriptor_type descriptor_type, const struct vkd3d_shader_register_range *range)
+{
+ struct vkd3d_string_buffer *buffer = vkd3d_string_buffer_get(&compiler->string_buffers);
+ char range_string[24];
+
+ if (!buffer)
+ return NULL;
+
+ if (range->last != ~0u)
+ sprintf(range_string, "[%u:%u]", range->first, range->last);
+ else
+ sprintf(range_string, "[%u:*]", range->first);
+
+ vkd3d_string_buffer_printf(buffer, "Could not find descriptor binding for type %#x, space %u, registers %s, "
+ "shader type %#x.\n", descriptor_type, range->space, range_string, compiler->shader_type);
+
+ return buffer;
+}
+
static struct vkd3d_shader_descriptor_binding vkd3d_dxbc_compiler_get_descriptor_binding(
struct vkd3d_dxbc_compiler *compiler, const struct vkd3d_shader_register *reg,
const struct vkd3d_shader_register_range *range, enum vkd3d_shader_resource_type resource_type,
@@ -2552,11 +2576,12 @@ static struct vkd3d_shader_descriptor_binding vkd3d_dxbc_compiler_get_descriptor
}
if (shader_interface->binding_count)
{
- FIXME("Could not find binding for type %#x, space %u, register %u, shader type %#x.\n",
- descriptor_type, range->space, range->first, compiler->shader_type);
+ struct vkd3d_string_buffer *buffer = vkd3d_dxbc_compiler_binding_error(compiler, descriptor_type, range);
+ if (buffer)
+ FIXME(buffer->buffer);
vkd3d_dxbc_compiler_error(compiler, VKD3D_SHADER_ERROR_SPV_DESCRIPTOR_BINDING_NOT_FOUND,
- "Could not find descriptor binding for type %#x, space %u, register %u, shader type %#x.",
- descriptor_type, range->space, range->first, compiler->shader_type);
+ buffer ? buffer->buffer : "");
+ vkd3d_string_buffer_release(&compiler->string_buffers, buffer);
}
}
@@ -9470,5 +9495,7 @@ void vkd3d_dxbc_compiler_destroy(struct vkd3d_dxbc_compiler *compiler)
vkd3d_free(compiler->shader_phases);
vkd3d_free(compiler->spec_constants);
+ vkd3d_string_buffer_cache_cleanup(&compiler->string_buffers);
+
vkd3d_free(compiler);
}
--
2.31.1
More information about the wine-devel
mailing list