[PATCH 0/1] MR784: wined3d: Tell vkd3d_shader about dual source blending.

[Stefan Dösinger]

Dual source blending implies an additional shader output, which needs to
be declared. This improves WoT on high graphics settings on MoltenVK.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/784

[PATCH 1/1] wined3d: Tell vkd3d_shader about dual source blending.

[Stefan Dösinger]

From: Stefan D��singer <stefan(a)codeweavers.com>

Dual source blending implies an additional shader output, which needs to
be declared. This improves WoT on high graphics settings on MoltenVK.
---
 dlls/wined3d/context_vk.c   | 8 +++++++-
 dlls/wined3d/shader_spirv.c | 4 ++++
 2 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/dlls/wined3d/context_vk.c b/dlls/wined3d/context_vk.c
index b738e97b6cf..755bd036bff 100644
--- a/dlls/wined3d/context_vk.c
+++ b/dlls/wined3d/context_vk.c
@@ -3422,6 +3422,8 @@ VkCommandBuffer wined3d_context_vk_apply_draw_state(struct wined3d_context_vk *c
 {
     struct wined3d_device_vk *device_vk = wined3d_device_vk(context_vk->c.device);
     const struct wined3d_vk_info *vk_info = context_vk->vk_info;
+    const struct wined3d_blend_state *b = state->blend_state;
+    bool dual_source_blend = b ? b->dual_source : false;
     struct wined3d_rendertarget_view *dsv;
     struct wined3d_rendertarget_view *rtv;
     struct wined3d_buffer_vk *buffer_vk;
@@ -3433,8 +3435,12 @@ VkCommandBuffer wined3d_context_vk_apply_draw_state(struct wined3d_context_vk *c
     bool invalidate_ds = false;
 
     if (wined3d_context_is_graphics_state_dirty(&context_vk->c, STATE_SHADER(WINED3D_SHADER_TYPE_PIXEL))
-            || wined3d_context_is_graphics_state_dirty(&context_vk->c, STATE_FRAMEBUFFER))
+            || wined3d_context_is_graphics_state_dirty(&context_vk->c, STATE_FRAMEBUFFER)
+            || dual_source_blend != context_vk->c.last_was_dual_source_blend)
+    {
         context_vk->c.shader_update_mask |= (1u << WINED3D_SHADER_TYPE_PIXEL);
+        context_vk->c.last_was_dual_source_blend = dual_source_blend;
+    }
     if (wined3d_context_is_graphics_state_dirty(&context_vk->c, STATE_SHADER(WINED3D_SHADER_TYPE_VERTEX)))
         context_vk->c.shader_update_mask |= (1u << WINED3D_SHADER_TYPE_VERTEX);
     if (wined3d_context_is_graphics_state_dirty(&context_vk->c, STATE_SHADER(WINED3D_SHADER_TYPE_GEOMETRY)))
diff --git a/dlls/wined3d/shader_spirv.c b/dlls/wined3d/shader_spirv.c
index 1f89e3d3ef5..10f03539538 100644
--- a/dlls/wined3d/shader_spirv.c
+++ b/dlls/wined3d/shader_spirv.c
@@ -57,6 +57,7 @@ struct shader_spirv_compile_arguments
         {
             uint32_t alpha_swizzle;
             unsigned int sample_count;
+            bool dual_source_blending;
         } fs;
     } u;
 };
@@ -147,6 +148,7 @@ static void shader_spirv_compile_arguments_init(struct shader_spirv_compile_argu
                     args->u.fs.alpha_swizzle |= 1u << i;
             }
             args->u.fs.sample_count = sample_count;
+            args->u.fs.dual_source_blending = state->blend_state ? state->blend_state->dual_source : false;
             break;
 
         default:
@@ -177,6 +179,8 @@ static void shader_spirv_init_compile_args(struct wined3d_shader_spirv_compile_a
         shader_parameter->data_type = VKD3D_SHADER_PARAMETER_DATA_TYPE_UINT32;
         shader_parameter->u.immediate_constant.u.u32 = compile_args->u.fs.sample_count;
 
+        args->spirv_target.dual_source_blending = compile_args->u.fs.dual_source_blending;
+
         args->spirv_target.parameter_count = 1;
         args->spirv_target.parameters = shader_parameter;
 
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/784

[Stefan Dösinger]

If there is an intended nicer way to handle the state dependency blend->pixel shader in the Vulkan code let me know...

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/784#note_7666