Signed-off-by: Henri Verbeet hverbeet@codeweavers.com --- include/vkd3d_shader.h | 14 ++++++------ libs/vkd3d-shader/dxbc.c | 41 ++++++++++++++++++----------------- libs/vkd3d-shader/vkd3d_shader_main.c | 4 ++-- libs/vkd3d/vkd3d_main.c | 15 +++++++------ tests/vkd3d_shader_api.c | 2 +- 5 files changed, 39 insertions(+), 37 deletions(-)
diff --git a/include/vkd3d_shader.h b/include/vkd3d_shader.h index 6eed196..24e2f24 100644 --- a/include/vkd3d_shader.h +++ b/include/vkd3d_shader.h @@ -568,17 +568,17 @@ struct vkd3d_root_signature_desc1 enum vkd3d_shader_root_signature_flags flags; };
-enum vkd3d_root_signature_version +enum vkd3d_shader_root_signature_version { - VKD3D_ROOT_SIGNATURE_VERSION_1_0 = 0x1, - VKD3D_ROOT_SIGNATURE_VERSION_1_1 = 0x2, + VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0 = 0x1, + VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1 = 0x2,
- VKD3D_FORCE_32_BIT_ENUM(VKD3D_ROOT_SIGNATURE_VERSION), + VKD3D_FORCE_32_BIT_ENUM(VKD3D_SHADER_ROOT_SIGNATURE_VERSION), };
struct vkd3d_versioned_root_signature_desc { - enum vkd3d_root_signature_version version; + enum vkd3d_shader_root_signature_version version; union { struct vkd3d_root_signature_desc v_1_0; @@ -737,7 +737,7 @@ int vkd3d_shader_serialize_root_signature(const struct vkd3d_versioned_root_sign struct vkd3d_shader_code *dxbc);
int vkd3d_shader_convert_root_signature(struct vkd3d_versioned_root_signature_desc *dst, - enum vkd3d_root_signature_version version, const struct vkd3d_versioned_root_signature_desc *src); + enum vkd3d_shader_root_signature_version version, const struct vkd3d_versioned_root_signature_desc *src);
int vkd3d_shader_scan_dxbc(const struct vkd3d_shader_code *dxbc, struct vkd3d_shader_scan_info *scan_info); @@ -767,7 +767,7 @@ typedef int (*PFN_vkd3d_shader_serialize_root_signature)( const struct vkd3d_versioned_root_signature_desc *root_signature, struct vkd3d_shader_code *dxbc);
typedef int (*PFN_vkd3d_shader_convert_root_signature)(struct vkd3d_versioned_root_signature_desc *dst, - enum vkd3d_root_signature_version version, const struct vkd3d_versioned_root_signature_desc *src); + enum vkd3d_shader_root_signature_version version, const struct vkd3d_versioned_root_signature_desc *src);
typedef int (*PFN_vkd3d_shader_scan_dxbc)(const struct vkd3d_shader_code *dxbc, struct vkd3d_shader_scan_info *scan_info); diff --git a/libs/vkd3d-shader/dxbc.c b/libs/vkd3d-shader/dxbc.c index ca9e7b8..e871963 100644 --- a/libs/vkd3d-shader/dxbc.c +++ b/libs/vkd3d-shader/dxbc.c @@ -2580,7 +2580,7 @@ static int shader_parse_root_signature(const char *data, unsigned int data_size,
read_dword(&ptr, &version); TRACE("Version %#x.\n", version); - if (version != VKD3D_ROOT_SIGNATURE_VERSION_1_0 && version != VKD3D_ROOT_SIGNATURE_VERSION_1_1) + if (version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0 && version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1) { FIXME("Unknown version %#x.\n", version); return VKD3D_ERROR_INVALID_ARGUMENT; @@ -2591,7 +2591,7 @@ static int shader_parse_root_signature(const char *data, unsigned int data_size, read_dword(&ptr, &offset); TRACE("Parameter count %u, offset %u.\n", count, offset);
- if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) { v_1_0->parameter_count = count; if (v_1_0->parameter_count) @@ -2608,7 +2608,7 @@ static int shader_parse_root_signature(const char *data, unsigned int data_size, { struct vkd3d_root_signature_desc1 *v_1_1 = &desc->u.v_1_1;
- assert(version == VKD3D_ROOT_SIGNATURE_VERSION_1_1); + assert(version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1);
v_1_1->parameter_count = count; if (v_1_1->parameter_count) @@ -2672,7 +2672,7 @@ int vkd3d_shader_parse_root_signature(const struct vkd3d_shader_code *dxbc,
static unsigned int versioned_root_signature_get_parameter_count(const struct vkd3d_versioned_root_signature_desc *desc) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) return desc->u.v_1_0.parameter_count; else return desc->u.v_1_1.parameter_count; @@ -2681,7 +2681,7 @@ static unsigned int versioned_root_signature_get_parameter_count(const struct vk static enum vkd3d_shader_root_parameter_type versioned_root_signature_get_parameter_type( const struct vkd3d_versioned_root_signature_desc *desc, unsigned int i) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) return desc->u.v_1_0.parameters[i].parameter_type; else return desc->u.v_1_1.parameters[i].parameter_type; @@ -2690,7 +2690,7 @@ static enum vkd3d_shader_root_parameter_type versioned_root_signature_get_parame static enum vkd3d_shader_visibility versioned_root_signature_get_parameter_shader_visibility( const struct vkd3d_versioned_root_signature_desc *desc, unsigned int i) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) return desc->u.v_1_0.parameters[i].shader_visibility; else return desc->u.v_1_1.parameters[i].shader_visibility; @@ -2699,7 +2699,7 @@ static enum vkd3d_shader_visibility versioned_root_signature_get_parameter_shade static const struct vkd3d_shader_root_constants *versioned_root_signature_get_root_constants( const struct vkd3d_versioned_root_signature_desc *desc, unsigned int i) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) return &desc->u.v_1_0.parameters[i].u.constants; else return &desc->u.v_1_1.parameters[i].u.constants; @@ -2707,7 +2707,7 @@ static const struct vkd3d_shader_root_constants *versioned_root_signature_get_ro
static unsigned int versioned_root_signature_get_static_sampler_count(const struct vkd3d_versioned_root_signature_desc *desc) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) return desc->u.v_1_0.static_sampler_count; else return desc->u.v_1_1.static_sampler_count; @@ -2716,7 +2716,7 @@ static unsigned int versioned_root_signature_get_static_sampler_count(const stru static const struct vkd3d_shader_static_sampler_desc *versioned_root_signature_get_static_samplers( const struct vkd3d_versioned_root_signature_desc *desc) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) return desc->u.v_1_0.static_samplers; else return desc->u.v_1_1.static_samplers; @@ -2724,7 +2724,7 @@ static const struct vkd3d_shader_static_sampler_desc *versioned_root_signature_g
static unsigned int versioned_root_signature_get_flags(const struct vkd3d_versioned_root_signature_desc *desc) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) return desc->u.v_1_0.flags; else return desc->u.v_1_1.flags; @@ -2939,7 +2939,7 @@ static int shader_write_root_parameters(struct root_signature_writer_context *co switch (versioned_root_signature_get_parameter_type(desc, i)) { case VKD3D_SHADER_ROOT_PARAMETER_TYPE_DESCRIPTOR_TABLE: - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) ret = shader_write_descriptor_table(context, &desc->u.v_1_0.parameters[i].u.descriptor_table); else ret = shader_write_descriptor_table1(context, &desc->u.v_1_1.parameters[i].u.descriptor_table); @@ -2950,7 +2950,7 @@ static int shader_write_root_parameters(struct root_signature_writer_context *co case VKD3D_SHADER_ROOT_PARAMETER_TYPE_CBV: case VKD3D_SHADER_ROOT_PARAMETER_TYPE_SRV: case VKD3D_SHADER_ROOT_PARAMETER_TYPE_UAV: - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) ret = shader_write_root_descriptor(context, &desc->u.v_1_0.parameters[i].u.descriptor); else ret = shader_write_root_descriptor1(context, &desc->u.v_1_1.parameters[i].u.descriptor); @@ -3120,7 +3120,7 @@ static int validate_root_signature_desc(const struct vkd3d_versioned_root_signat type = versioned_root_signature_get_parameter_type(desc, i); if (type == VKD3D_SHADER_ROOT_PARAMETER_TYPE_DESCRIPTOR_TABLE) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) ret = validate_descriptor_table_v_1_0(&desc->u.v_1_0.parameters[i].u.descriptor_table); else ret = validate_descriptor_table_v_1_1(&desc->u.v_1_1.parameters[i].u.descriptor_table); @@ -3143,8 +3143,8 @@ int vkd3d_shader_serialize_root_signature(const struct vkd3d_versioned_root_sign
TRACE("root_signature %p, dxbc %p.\n", root_signature, dxbc);
- if (root_signature->version != VKD3D_ROOT_SIGNATURE_VERSION_1_0 - && root_signature->version != VKD3D_ROOT_SIGNATURE_VERSION_1_1) + if (root_signature->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0 + && root_signature->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1) { WARN("Root signature version %#x not supported.\n", root_signature->version); return VKD3D_ERROR_INVALID_ARGUMENT; @@ -3422,7 +3422,7 @@ fail: }
int vkd3d_shader_convert_root_signature(struct vkd3d_versioned_root_signature_desc *dst, - enum vkd3d_root_signature_version version, const struct vkd3d_versioned_root_signature_desc *src) + enum vkd3d_shader_root_signature_version version, const struct vkd3d_versioned_root_signature_desc *src) { int ret;
@@ -3434,13 +3434,14 @@ int vkd3d_shader_convert_root_signature(struct vkd3d_versioned_root_signature_de return VKD3D_ERROR_INVALID_ARGUMENT; }
- if (version != VKD3D_ROOT_SIGNATURE_VERSION_1_0 && version != VKD3D_ROOT_SIGNATURE_VERSION_1_1) + if (version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0 && version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1) { WARN("Root signature version %#x not supported.\n", version); return VKD3D_ERROR_INVALID_ARGUMENT; }
- if (src->version != VKD3D_ROOT_SIGNATURE_VERSION_1_0 && src->version != VKD3D_ROOT_SIGNATURE_VERSION_1_1) + if (src->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0 + && src->version != VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1) { WARN("Root signature version %#x not supported.\n", src->version); return VKD3D_ERROR_INVALID_ARGUMENT; @@ -3449,13 +3450,13 @@ int vkd3d_shader_convert_root_signature(struct vkd3d_versioned_root_signature_de memset(dst, 0, sizeof(*dst)); dst->version = version;
- if (version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) { ret = convert_root_signature_to_v1_0(dst, src); } else { - assert(version == VKD3D_ROOT_SIGNATURE_VERSION_1_1); + assert(version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1); ret = convert_root_signature_to_v1_1(dst, src); }
diff --git a/libs/vkd3d-shader/vkd3d_shader_main.c b/libs/vkd3d-shader/vkd3d_shader_main.c index d4bd763..ff02eca 100644 --- a/libs/vkd3d-shader/vkd3d_shader_main.c +++ b/libs/vkd3d-shader/vkd3d_shader_main.c @@ -514,11 +514,11 @@ static void vkd3d_shader_free_root_signature_v_1_1(struct vkd3d_root_signature_d
void vkd3d_shader_free_root_signature(struct vkd3d_versioned_root_signature_desc *desc) { - if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) { vkd3d_shader_free_root_signature_v_1_0(&desc->u.v_1_0); } - else if (desc->version == VKD3D_ROOT_SIGNATURE_VERSION_1_1) + else if (desc->version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1) { vkd3d_shader_free_root_signature_v_1_1(&desc->u.v_1_1); } diff --git a/libs/vkd3d/vkd3d_main.c b/libs/vkd3d/vkd3d_main.c index 7775077..964c6f5 100644 --- a/libs/vkd3d/vkd3d_main.c +++ b/libs/vkd3d/vkd3d_main.c @@ -180,15 +180,15 @@ int vkd3d_parse_root_signature_v_1_0(const struct vkd3d_shader_code *dxbc, return ret; }
- if (desc.version == VKD3D_ROOT_SIGNATURE_VERSION_1_0) + if (desc.version == VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0) { *out_desc = desc; } else { - enum vkd3d_root_signature_version version = desc.version; + enum vkd3d_shader_root_signature_version version = desc.version;
- ret = vkd3d_shader_convert_root_signature(&converted_desc, VKD3D_ROOT_SIGNATURE_VERSION_1_0, &desc); + ret = vkd3d_shader_convert_root_signature(&converted_desc, VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0, &desc); vkd3d_shader_free_root_signature(&desc); if (ret < 0) { @@ -309,14 +309,15 @@ static ULONG STDMETHODCALLTYPE d3d12_versioned_root_signature_deserializer_Relea return refcount; }
-static enum vkd3d_root_signature_version vkd3d_root_signature_version_from_d3d12(D3D_ROOT_SIGNATURE_VERSION version) +static enum vkd3d_shader_root_signature_version vkd3d_root_signature_version_from_d3d12( + D3D_ROOT_SIGNATURE_VERSION version) { switch (version) { case D3D_ROOT_SIGNATURE_VERSION_1_0: - return VKD3D_ROOT_SIGNATURE_VERSION_1_0; + return VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0; case D3D_ROOT_SIGNATURE_VERSION_1_1: - return VKD3D_ROOT_SIGNATURE_VERSION_1_1; + return VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_1; default: WARN("Unknown root signature version %#x.\n", version); return 0; @@ -562,7 +563,7 @@ HRESULT vkd3d_serialize_root_signature(const D3D12_ROOT_SIGNATURE_DESC *desc, if (error_blob) *error_blob = NULL;
- vkd3d_desc.version = VKD3D_ROOT_SIGNATURE_VERSION_1_0; + vkd3d_desc.version = VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0; vkd3d_desc.u.v_1_0 = *(const struct vkd3d_root_signature_desc *)desc; if ((ret = vkd3d_shader_serialize_root_signature(&vkd3d_desc, &dxbc)) < 0) { diff --git a/tests/vkd3d_shader_api.c b/tests/vkd3d_shader_api.c index 6aef004..569b0e9 100644 --- a/tests/vkd3d_shader_api.c +++ b/tests/vkd3d_shader_api.c @@ -90,7 +90,7 @@ static void test_vkd3d_shader_pfns(void)
static const struct vkd3d_versioned_root_signature_desc empty_rs_desc = { - .version = VKD3D_ROOT_SIGNATURE_VERSION_1_0, + .version = VKD3D_SHADER_ROOT_SIGNATURE_VERSION_1_0, }; static const DWORD vs_code[] = {