From c98b7fc702deb8d3e93f24d23dfddef375df15ff Mon Sep 17 00:00:00 2001 From: Piplup <100526773+piplup55@users.noreply.github.com> Date: Fri, 24 May 2024 02:57:26 +0100 Subject: [PATCH] Workaround bug on logic op with float framebuffer (#6858) * intel workaround built on top of the amd workaround * forgot to update the note * Logic Change Enabled workaround for all vendors that aren't nvidia * Applied Suggestions --- src/Ryujinx.Graphics.Vulkan/PipelineState.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Ryujinx.Graphics.Vulkan/PipelineState.cs b/src/Ryujinx.Graphics.Vulkan/PipelineState.cs index 211608584..c38748936 100644 --- a/src/Ryujinx.Graphics.Vulkan/PipelineState.cs +++ b/src/Ryujinx.Graphics.Vulkan/PipelineState.cs @@ -560,9 +560,9 @@ namespace Ryujinx.Graphics.Vulkan } } - // AMD has a bug where it enables logical operations even for float formats, + // Vendors other than NVIDIA have a bug where it enables logical operations even for float formats, // so we need to force disable them here. - bool logicOpEnable = LogicOpEnable && (gd.Vendor != Vendor.Amd || Internal.LogicOpsAllowed); + bool logicOpEnable = LogicOpEnable && (gd.Vendor == Vendor.Nvidia || Internal.LogicOpsAllowed); var colorBlendState = new PipelineColorBlendStateCreateInfo {