From 26529a31ab97f2743fe1a7c52969079d7764bc6d Mon Sep 17 00:00:00 2001 From: TellowKrinkle Date: Thu, 16 Jun 2022 17:26:30 -0500 Subject: VideoCommon: Fix SSBO layout and remove associated "bug" --- Source/Core/VideoCommon/PixelShaderGen.cpp | 19 +++---------------- 1 file changed, 3 insertions(+), 16 deletions(-) (limited to 'Source/Core/VideoCommon/PixelShaderGen.cpp') diff --git a/Source/Core/VideoCommon/PixelShaderGen.cpp b/Source/Core/VideoCommon/PixelShaderGen.cpp index 4a9e9105a8..950763f5d8 100644 --- a/Source/Core/VideoCommon/PixelShaderGen.cpp +++ b/Source/Core/VideoCommon/PixelShaderGen.cpp @@ -464,22 +464,9 @@ void WritePixelShaderCommonHeader(ShaderCode& out, APIType api_type, } else { - out.Write("SSBO_BINDING(0) buffer BBox {{\n"); - - if (DriverDetails::HasBug(DriverDetails::BUG_BROKEN_SSBO_FIELD_ATOMICS)) - { - // AMD drivers on Windows seemingly ignore atomic writes to fields or array elements of an - // SSBO other than the first one, but using an int4 seems to work fine - out.Write(" int4 bbox_data;\n"); - } - else - { - // The Metal shader compiler fails to compile the atomic instructions when operating on - // individual components of a vector - out.Write(" int bbox_data[4];\n"); - } - - out.Write("}};"); + out.Write("SSBO_BINDING(0) buffer BBox {{\n" + " int bbox_data[4];\n" + "}};"); } out.Write(R"( -- cgit v1.2.3