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

[Stefan Dösinger (＠stefan)]

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

--
  v2: wined3d: Tell vkd3d_shader about dual source blending.

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

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

[Stefan Dösinger]

From: 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.
---
 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..db1840d29e2 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;
     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..42be7541d67 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;
             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;

[Stefan Dösinger (＠stefan)]

fwiw we already have test_dual_source_blend() in dlls/d3d11/tests/d3d11.c. This patch also fixes this test on MVK.

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

[Zebediah Figura (＠zfigura)]

This merge request was approved by Zebediah Figura.