From e57730a97d574791a6603a32e093aba790e6ff65 Mon Sep 17 00:00:00 2001
From: "Unknown W. Brackets" <checkins@unknownbrackets.org>
Date: Sun, 9 Jan 2022 21:33:45 -0800
Subject: [PATCH 1/2] softgpu: Output normals to GE debugger.

---
 GPU/Software/TransformUnit.cpp | 19 +++++++++----------
 1 file changed, 9 insertions(+), 10 deletions(-)

diff --git a/GPU/Software/TransformUnit.cpp b/GPU/Software/TransformUnit.cpp
index 021e0427f5..3e8a6a9ab0 100644
--- a/GPU/Software/TransformUnit.cpp
+++ b/GPU/Software/TransformUnit.cpp
@@ -694,11 +694,6 @@ bool TransformUnit::GetCurrentSimpleVertices(int count, std::vector<GPUDebugVert
 			vertices[i].x = vert.pos.x;
 			vertices[i].y = vert.pos.y;
 			vertices[i].z = vert.pos.z;
-			if (gstate.vertType & GE_VTYPE_COL_MASK) {
-				memcpy(vertices[i].c, vert.color, sizeof(vertices[i].c));
-			} else {
-				memset(vertices[i].c, 0, sizeof(vertices[i].c));
-			}
 		} else {
 			float clipPos[4];
 			Vec3ByMatrix44(clipPos, vert.pos.AsArray(), worldviewproj);
@@ -715,12 +710,16 @@ bool TransformUnit::GetCurrentSimpleVertices(int count, std::vector<GPUDebugVert
 			vertices[i].x = drawPos.x;
 			vertices[i].y = drawPos.y;
 			vertices[i].z = drawPos.z;
-			if (gstate.vertType & GE_VTYPE_COL_MASK) {
-				memcpy(vertices[i].c, vert.color, sizeof(vertices[i].c));
-			} else {
-				memset(vertices[i].c, 0, sizeof(vertices[i].c));
-			}
 		}
+
+		if (gstate.vertType & GE_VTYPE_COL_MASK) {
+			memcpy(vertices[i].c, vert.color, sizeof(vertices[i].c));
+		} else {
+			memset(vertices[i].c, 0, sizeof(vertices[i].c));
+		}
+		vertices[i].nx = vert.nrm.x;
+		vertices[i].ny = vert.nrm.y;
+		vertices[i].nz = vert.nrm.z;
 	}
 
 	// The GE debugger expects these to be set.

From a9a07b0935b34bdbed7356feaa7eabca5401463c Mon Sep 17 00:00:00 2001
From: "Unknown W. Brackets" <checkins@unknownbrackets.org>
Date: Sun, 9 Jan 2022 21:34:05 -0800
Subject: [PATCH 2/2] Vulkan: Avoid crash on bad shader.

---
 GPU/Vulkan/DrawEngineVulkan.cpp | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/GPU/Vulkan/DrawEngineVulkan.cpp b/GPU/Vulkan/DrawEngineVulkan.cpp
index 5cd4ef69fe..c9331aa79b 100644
--- a/GPU/Vulkan/DrawEngineVulkan.cpp
+++ b/GPU/Vulkan/DrawEngineVulkan.cpp
@@ -907,6 +907,9 @@ void DrawEngineVulkan::DoFlush() {
 				VulkanPipeline *pipeline = pipelineManager_->GetOrCreatePipeline(renderManager, pipelineLayout_, renderPass, pipelineKey_, &dec_->decFmt, vshader, fshader, false);
 				if (!pipeline || !pipeline->pipeline) {
 					// Already logged, let's bail out.
+					decodedVerts_ = 0;
+					numDrawCalls = 0;
+					decodeCounter_ = 0;
 					return;
 				}
 				BindShaderBlendTex();  // This might cause copies so super important to do before BindPipeline.