Skip to content

Commit

Permalink
Workaround bug on logic op with float framebuffer (#6858)
Browse files Browse the repository at this point in the history
* 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
  • Loading branch information
piplup55 authored May 24, 2024
1 parent e65effc commit c98b7fc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Ryujinx.Graphics.Vulkan/PipelineState.cs
Original file line number Diff line number Diff line change
Expand Up @@ -560,9 +560,9 @@ public unsafe Auto<DisposablePipeline> CreateGraphicsPipeline(
}
}

// 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
{
Expand Down

0 comments on commit c98b7fc

Please sign in to comment.