vkd3d-shader: Rename the vkd3d_shader_target enumeration to vkd3d_shader_spirv_environment.

Signed-off-by: Henri Verbeet <hverbeet@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Henri Verbeet 2020-06-18 18:52:50 +04:30 committed by Alexandre Julliard
parent 2ee8429c03
commit 425dab80b8
4 changed files with 39 additions and 35 deletions

View File

@ -218,13 +218,13 @@ struct vkd3d_shader_transform_feedback_info
unsigned int buffer_stride_count; unsigned int buffer_stride_count;
}; };
enum vkd3d_shader_target enum vkd3d_shader_spirv_environment
{ {
VKD3D_SHADER_TARGET_NONE, VKD3D_SHADER_SPIRV_ENVIRONMENT_NONE,
VKD3D_SHADER_TARGET_SPIRV_OPENGL_4_5, VKD3D_SHADER_SPIRV_ENVIRONMENT_OPENGL_4_5,
VKD3D_SHADER_TARGET_SPIRV_VULKAN_1_0, /* default target */ VKD3D_SHADER_SPIRV_ENVIRONMENT_VULKAN_1_0, /* default target */
VKD3D_FORCE_32_BIT_ENUM(VKD3D_SHADER_TARGET), VKD3D_FORCE_32_BIT_ENUM(VKD3D_SHADER_SPIRV_ENVIRONMENT),
}; };
enum vkd3d_shader_target_extension enum vkd3d_shader_target_extension
@ -239,7 +239,7 @@ struct vkd3d_shader_spirv_target_info
enum vkd3d_shader_structure_type type; enum vkd3d_shader_structure_type type;
const void *next; const void *next;
enum vkd3d_shader_target target; enum vkd3d_shader_spirv_environment environment;
unsigned int target_extension_count; unsigned int target_extension_count;
const enum vkd3d_shader_target_extension *target_extensions; const enum vkd3d_shader_target_extension *target_extensions;

View File

@ -36,21 +36,21 @@
#ifdef HAVE_SPIRV_TOOLS #ifdef HAVE_SPIRV_TOOLS
# include "spirv-tools/libspirv.h" # include "spirv-tools/libspirv.h"
static spv_target_env spv_target_env_from_vkd3d(enum vkd3d_shader_target target) static spv_target_env spv_target_env_from_vkd3d(enum vkd3d_shader_spirv_environment environment)
{ {
switch (target) switch (environment)
{ {
case VKD3D_SHADER_TARGET_SPIRV_OPENGL_4_5: case VKD3D_SHADER_SPIRV_ENVIRONMENT_OPENGL_4_5:
return SPV_ENV_OPENGL_4_5; return SPV_ENV_OPENGL_4_5;
case VKD3D_SHADER_TARGET_SPIRV_VULKAN_1_0: case VKD3D_SHADER_SPIRV_ENVIRONMENT_VULKAN_1_0:
return SPV_ENV_VULKAN_1_0; return SPV_ENV_VULKAN_1_0;
default: default:
ERR("Invalid shader target %#x.\n", target); ERR("Invalid environment %#x.\n", environment);
return SPV_ENV_VULKAN_1_0; return SPV_ENV_VULKAN_1_0;
} }
} }
static void vkd3d_spirv_dump(const struct vkd3d_shader_code *spirv, enum vkd3d_shader_target target) static void vkd3d_spirv_dump(const struct vkd3d_shader_code *spirv, enum vkd3d_shader_spirv_environment environment)
{ {
const static uint32_t options const static uint32_t options
= SPV_BINARY_TO_TEXT_OPTION_FRIENDLY_NAMES | SPV_BINARY_TO_TEXT_OPTION_INDENT; = SPV_BINARY_TO_TEXT_OPTION_FRIENDLY_NAMES | SPV_BINARY_TO_TEXT_OPTION_INDENT;
@ -59,7 +59,7 @@ static void vkd3d_spirv_dump(const struct vkd3d_shader_code *spirv, enum vkd3d_s
spv_context context; spv_context context;
spv_result_t ret; spv_result_t ret;
context = spvContextCreate(spv_target_env_from_vkd3d(target)); context = spvContextCreate(spv_target_env_from_vkd3d(environment));
if (!(ret = spvBinaryToText(context, spirv->code, spirv->size / sizeof(uint32_t), if (!(ret = spvBinaryToText(context, spirv->code, spirv->size / sizeof(uint32_t),
options, &text, &diagnostic))) options, &text, &diagnostic)))
@ -82,13 +82,14 @@ static void vkd3d_spirv_dump(const struct vkd3d_shader_code *spirv, enum vkd3d_s
spvContextDestroy(context); spvContextDestroy(context);
} }
static void vkd3d_spirv_validate(const struct vkd3d_shader_code *spirv, enum vkd3d_shader_target target) static void vkd3d_spirv_validate(const struct vkd3d_shader_code *spirv,
enum vkd3d_shader_spirv_environment environment)
{ {
spv_diagnostic diagnostic = NULL; spv_diagnostic diagnostic = NULL;
spv_context context; spv_context context;
spv_result_t ret; spv_result_t ret;
context = spvContextCreate(spv_target_env_from_vkd3d(target)); context = spvContextCreate(spv_target_env_from_vkd3d(environment));
if ((ret = spvValidateBinary(context, spirv->code, spirv->size / sizeof(uint32_t), if ((ret = spvValidateBinary(context, spirv->code, spirv->size / sizeof(uint32_t),
&diagnostic))) &diagnostic)))
@ -103,8 +104,10 @@ static void vkd3d_spirv_validate(const struct vkd3d_shader_code *spirv, enum vkd
#else #else
static void vkd3d_spirv_dump(const struct vkd3d_shader_code *spirv, enum vkd3d_shader_target target) {} static void vkd3d_spirv_dump(const struct vkd3d_shader_code *spirv,
static void vkd3d_spirv_validate(const struct vkd3d_shader_code *spirv, enum vkd3d_shader_target target) {} enum vkd3d_shader_spirv_environment environment) {}
static void vkd3d_spirv_validate(const struct vkd3d_shader_code *spirv,
enum vkd3d_shader_spirv_environment environment) {}
#endif /* HAVE_SPIRV_TOOLS */ #endif /* HAVE_SPIRV_TOOLS */
@ -2180,16 +2183,17 @@ struct vkd3d_dxbc_compiler *vkd3d_dxbc_compiler_create(const struct vkd3d_shader
return compiler; return compiler;
} }
static enum vkd3d_shader_target vkd3d_dxbc_compiler_get_target(const struct vkd3d_dxbc_compiler *compiler) static enum vkd3d_shader_spirv_environment vkd3d_dxbc_compiler_get_target_environment(
const struct vkd3d_dxbc_compiler *compiler)
{ {
const struct vkd3d_shader_spirv_target_info *info = compiler->spirv_target_info; const struct vkd3d_shader_spirv_target_info *info = compiler->spirv_target_info;
return info ? info->target : VKD3D_SHADER_TARGET_SPIRV_VULKAN_1_0; return info ? info->environment : VKD3D_SHADER_SPIRV_ENVIRONMENT_VULKAN_1_0;
} }
static bool vkd3d_dxbc_compiler_is_opengl_target(const struct vkd3d_dxbc_compiler *compiler) static bool vkd3d_dxbc_compiler_is_opengl_target(const struct vkd3d_dxbc_compiler *compiler)
{ {
return vkd3d_dxbc_compiler_get_target(compiler) == VKD3D_SHADER_TARGET_SPIRV_OPENGL_4_5; return vkd3d_dxbc_compiler_get_target_environment(compiler) == VKD3D_SHADER_SPIRV_ENVIRONMENT_OPENGL_4_5;
} }
static bool vkd3d_dxbc_compiler_is_target_extension_supported(const struct vkd3d_dxbc_compiler *compiler, static bool vkd3d_dxbc_compiler_is_target_extension_supported(const struct vkd3d_dxbc_compiler *compiler,
@ -3544,12 +3548,12 @@ static const struct
{ {
enum vkd3d_shader_input_sysval_semantic sysval; enum vkd3d_shader_input_sysval_semantic sysval;
struct vkd3d_spirv_builtin builtin; struct vkd3d_spirv_builtin builtin;
enum vkd3d_shader_target target; enum vkd3d_shader_spirv_environment environment;
} }
vkd3d_system_value_builtins[] = vkd3d_system_value_builtins[] =
{ {
{VKD3D_SIV_VERTEX_ID, {VKD3D_TYPE_INT, 1, SpvBuiltInVertexId}, VKD3D_SHADER_TARGET_SPIRV_OPENGL_4_5}, {VKD3D_SIV_VERTEX_ID, {VKD3D_TYPE_INT, 1, SpvBuiltInVertexId}, VKD3D_SHADER_SPIRV_ENVIRONMENT_OPENGL_4_5},
{VKD3D_SIV_INSTANCE_ID, {VKD3D_TYPE_INT, 1, SpvBuiltInInstanceId}, VKD3D_SHADER_TARGET_SPIRV_OPENGL_4_5}, {VKD3D_SIV_INSTANCE_ID, {VKD3D_TYPE_INT, 1, SpvBuiltInInstanceId}, VKD3D_SHADER_SPIRV_ENVIRONMENT_OPENGL_4_5},
{VKD3D_SIV_POSITION, {VKD3D_TYPE_FLOAT, 4, SpvBuiltInPosition}}, {VKD3D_SIV_POSITION, {VKD3D_TYPE_FLOAT, 4, SpvBuiltInPosition}},
{VKD3D_SIV_VERTEX_ID, {VKD3D_TYPE_INT, 1, SpvBuiltInVertexIndex, sv_vertex_id_fixup}}, {VKD3D_SIV_VERTEX_ID, {VKD3D_TYPE_INT, 1, SpvBuiltInVertexIndex, sv_vertex_id_fixup}},
@ -3632,7 +3636,7 @@ static void vkd3d_dxbc_compiler_emit_register_execution_mode(struct vkd3d_dxbc_c
static const struct vkd3d_spirv_builtin *get_spirv_builtin_for_sysval( static const struct vkd3d_spirv_builtin *get_spirv_builtin_for_sysval(
const struct vkd3d_dxbc_compiler *compiler, enum vkd3d_shader_input_sysval_semantic sysval) const struct vkd3d_dxbc_compiler *compiler, enum vkd3d_shader_input_sysval_semantic sysval)
{ {
enum vkd3d_shader_target target; enum vkd3d_shader_spirv_environment environment;
unsigned int i; unsigned int i;
if (!sysval) if (!sysval)
@ -3642,12 +3646,12 @@ static const struct vkd3d_spirv_builtin *get_spirv_builtin_for_sysval(
if (sysval == VKD3D_SIV_POSITION && compiler->shader_type == VKD3D_SHADER_TYPE_PIXEL) if (sysval == VKD3D_SIV_POSITION && compiler->shader_type == VKD3D_SHADER_TYPE_PIXEL)
return &vkd3d_pixel_shader_position_builtin; return &vkd3d_pixel_shader_position_builtin;
target = vkd3d_dxbc_compiler_get_target(compiler); environment = vkd3d_dxbc_compiler_get_target_environment(compiler);
for (i = 0; i < ARRAY_SIZE(vkd3d_system_value_builtins); ++i) for (i = 0; i < ARRAY_SIZE(vkd3d_system_value_builtins); ++i)
{ {
if (vkd3d_system_value_builtins[i].sysval == sysval if (vkd3d_system_value_builtins[i].sysval == sysval
&& (!vkd3d_system_value_builtins[i].target && (!vkd3d_system_value_builtins[i].environment
|| vkd3d_system_value_builtins[i].target == target)) || vkd3d_system_value_builtins[i].environment == environment))
return &vkd3d_system_value_builtins[i].builtin; return &vkd3d_system_value_builtins[i].builtin;
} }
@ -8711,9 +8715,9 @@ int vkd3d_dxbc_compiler_generate_spirv(struct vkd3d_dxbc_compiler *compiler,
if (TRACE_ON()) if (TRACE_ON())
{ {
enum vkd3d_shader_target target = vkd3d_dxbc_compiler_get_target(compiler); enum vkd3d_shader_spirv_environment environment = vkd3d_dxbc_compiler_get_target_environment(compiler);
vkd3d_spirv_dump(spirv, target); vkd3d_spirv_dump(spirv, environment);
vkd3d_spirv_validate(spirv, target); vkd3d_spirv_validate(spirv, environment);
} }
return VKD3D_OK; return VKD3D_OK;

View File

@ -114,13 +114,13 @@ static int vkd3d_shader_validate_spirv_target_info(const struct vkd3d_shader_spi
return VKD3D_ERROR_INVALID_ARGUMENT; return VKD3D_ERROR_INVALID_ARGUMENT;
} }
switch (info->target) switch (info->environment)
{ {
case VKD3D_SHADER_TARGET_SPIRV_OPENGL_4_5: case VKD3D_SHADER_SPIRV_ENVIRONMENT_OPENGL_4_5:
case VKD3D_SHADER_TARGET_SPIRV_VULKAN_1_0: case VKD3D_SHADER_SPIRV_ENVIRONMENT_VULKAN_1_0:
break; break;
default: default:
WARN("Invalid shader target %#x.\n", info->target); WARN("Invalid target environment %#x.\n", info->environment);
return VKD3D_ERROR_INVALID_ARGUMENT; return VKD3D_ERROR_INVALID_ARGUMENT;
} }

View File

@ -2166,7 +2166,7 @@ static HRESULT d3d12_pipeline_state_init_graphics(struct d3d12_pipeline_state *s
ps_target_info.type = VKD3D_SHADER_STRUCTURE_TYPE_SPIRV_TARGET_INFO; ps_target_info.type = VKD3D_SHADER_STRUCTURE_TYPE_SPIRV_TARGET_INFO;
ps_target_info.next = NULL; ps_target_info.next = NULL;
ps_target_info.target = VKD3D_SHADER_TARGET_SPIRV_VULKAN_1_0; ps_target_info.environment = VKD3D_SHADER_SPIRV_ENVIRONMENT_VULKAN_1_0;
ps_target_info.target_extensions = vk_info->shader_extensions; ps_target_info.target_extensions = vk_info->shader_extensions;
ps_target_info.target_extension_count = vk_info->shader_extension_count; ps_target_info.target_extension_count = vk_info->shader_extension_count;
ps_target_info.parameters = ps_shader_parameters; ps_target_info.parameters = ps_shader_parameters;