Merge branch 'feature/dsb-ub' into 'master'

Workaround undefined behavior in bad dual source blend states

See merge request ryubing/ryujinx!4

assets/locales.json

@@ -19914,7 +19914,7 @@
         "pl_PL": "Skróty Klawiszowe Klawiatury",
         "pt_BR": "Atalhos do Teclado",
         "ru_RU": "Горячие клавиши",
-        "sv_SE": "Snabbtangenter för tangentbord",
+        "sv_SE": "Snabbtangenter",
         "th_TH": "ปุ่มลัดของคีย์บอร์ด",
         "tr_TR": "Klavye Kısayolları",
         "uk_UA": "Гарячі клавіші",

src/Ryujinx.Graphics.Shader/CodeGen/Glsl/Declarations.cs

@@ -512,13 +512,9 @@ namespace Ryujinx.Graphics.Shader.CodeGen.Glsl
                 if (context.Definitions.Stage == ShaderStage.Fragment && context.Definitions.DualSourceBlend)
                 {
                     IoDefinition firstOutput = outputs.ElementAtOrDefault(0);
-                    IoDefinition secondOutput = outputs.ElementAtOrDefault(1);
 
-                    if (firstOutput.Location + 1 == secondOutput.Location)
+                    DeclareOutputDualSourceBlendAttribute(context, firstOutput.Location);
-                    {
+                    outputs = outputs.Skip(2);
-                        DeclareOutputDualSourceBlendAttribute(context, firstOutput.Location);
-                        outputs = outputs.Skip(2);
-                    }
                 }
 
                 foreach (IoDefinition ioDefinition in outputs)

src/Ryujinx.Graphics.Shader/Translation/Translator.cs

@@ -181,13 +181,29 @@ namespace Ryujinx.Graphics.Shader.Translation
 
         private static void EmitOutputsInitialization(EmitterContext context, AttributeUsage attributeUsage, IGpuAccessor gpuAccessor, ShaderStage stage)
         {
-            // Compute has no output attributes, and fragment is the last stage, so we
+            // Compute has no output attributes, so we
-            // don't need to initialize outputs on those stages.
+            // don't need to initialize outputs on that stage.
-            if (stage == ShaderStage.Compute || stage == ShaderStage.Fragment)
+            if (stage == ShaderStage.Compute)
             {
                 return;
             }
 
+            if (stage == ShaderStage.Fragment)
+            {
+                // Fragment is the last stage, so we don't need to
+                // initialize outputs unless we're using DSB, in which
+                // we need to make sure the ouput has a valid value.
+                if (gpuAccessor.QueryGraphicsState().DualSourceBlendEnable)
+                {
+                    for (int i = 0; i < 4; i++)
+                    {
+                        context.Store(StorageKind.Output, IoVariable.FragmentOutputColor, null, Const(1), Const(i), ConstF(0));
+                    }
+                }
+
+                return;
+            }
+
             if (stage == ShaderStage.Vertex)
             {
                 InitializeVertexOutputs(context);