Simplify the logic in MakePixelTexture a bit

hrydgard · Nov 12, 2023 · 49f5da3 · 49f5da3
1 parent cc6f9a7
commit 49f5da3
Showing 1 changed file with 21 additions and 20 deletions.
diff --git a/GPU/Common/FramebufferManagerCommon.cpp b/GPU/Common/FramebufferManagerCommon.cpp
@@ -1310,18 +1310,6 @@ void FramebufferManagerCommon::CopyFramebufferForColorTexture(VirtualFramebuffer
 
 Draw::Texture *FramebufferManagerCommon::MakePixelTexture(const u8 *srcPixels, GEBufferFormat srcPixelFormat, int srcStride, int width, int height) {
 	Draw::DataFormat depthFormat = Draw::DataFormat::UNDEFINED;
-	if (srcPixelFormat == GE_FORMAT_DEPTH16) {
-		if ((draw_->GetDataFormatSupport(Draw::DataFormat::R16_UNORM) & Draw::FMT_TEXTURE) != 0) {
-			depthFormat = Draw::DataFormat::R16_UNORM;
-		} else if ((draw_->GetDataFormatSupport(Draw::DataFormat::R8_UNORM) & Draw::FMT_TEXTURE) != 0) {
-			// This could be improved by using specific draw shaders to pack full precision in two channels.
-			// However, not really worth the trouble until we find a game that requires it.
-			depthFormat = Draw::DataFormat::R8_UNORM;
-		} else {
-			// No usable single channel format. Can't be bothered.
-			return nullptr;
-		}
-	}
 
 	int bpp = BufferFormatBytesPerPixel(srcPixelFormat);
 	int srcStrideInBytes = srcStride * bpp;
@@ -1341,6 +1329,21 @@ Draw::Texture *FramebufferManagerCommon::MakePixelTexture(const u8 *srcPixels, G
 		XXH3_freeState(hashState);
 	}
 
+	Draw::DataFormat texFormat = preferredPixelsFormat_;
+
+	if (srcPixelFormat == GE_FORMAT_DEPTH16) {
+		if ((draw_->GetDataFormatSupport(Draw::DataFormat::R16_UNORM) & Draw::FMT_TEXTURE) != 0) {
+			texFormat = Draw::DataFormat::R16_UNORM;
+		} else if ((draw_->GetDataFormatSupport(Draw::DataFormat::R8_UNORM) & Draw::FMT_TEXTURE) != 0) {
+			// This could be improved by using specific draw shaders to pack full precision in two channels.
+			// However, not really worth the trouble until we find a game that requires it.
+			texFormat = Draw::DataFormat::R8_UNORM;
+		} else {
+			// No usable single channel format. Can't be bothered.
+			return nullptr;
+		}
+	}
+
 	// TODO: We can just change the texture format and flip some bits around instead of this.
 	// Could share code with the texture cache perhaps.
 	auto generateTexture = [&](uint8_t *data, const uint8_t *initData, uint32_t w, uint32_t h, uint32_t d, uint32_t byteStride, uint32_t sliceByteStride) {
@@ -1352,28 +1355,28 @@ Draw::Texture *FramebufferManagerCommon::MakePixelTexture(const u8 *srcPixels, G
 			u8 *dst8 = (u8 *)(data + byteStride * y);
 			switch (srcPixelFormat) {
 			case GE_FORMAT_565:
-				if (preferredPixelsFormat_ == Draw::DataFormat::B8G8R8A8_UNORM)
+				if (texFormat == Draw::DataFormat::B8G8R8A8_UNORM)
 					ConvertRGB565ToBGRA8888(dst, src16, width);
 				else
 					ConvertRGB565ToRGBA8888(dst, src16, width);
 				break;
 
 			case GE_FORMAT_5551:
-				if (preferredPixelsFormat_ == Draw::DataFormat::B8G8R8A8_UNORM)
+				if (texFormat == Draw::DataFormat::B8G8R8A8_UNORM)
 					ConvertRGBA5551ToBGRA8888(dst, src16, width);
 				else
 					ConvertRGBA5551ToRGBA8888(dst, src16, width);
 				break;
 
 			case GE_FORMAT_4444:
-				if (preferredPixelsFormat_ == Draw::DataFormat::B8G8R8A8_UNORM)
+				if (texFormat == Draw::DataFormat::B8G8R8A8_UNORM)
 					ConvertRGBA4444ToBGRA8888(dst, src16, width);
 				else
 					ConvertRGBA4444ToRGBA8888(dst, src16, width);
 				break;
 
 			case GE_FORMAT_8888:
-				if (preferredPixelsFormat_ == Draw::DataFormat::B8G8R8A8_UNORM)
+				if (texFormat == Draw::DataFormat::B8G8R8A8_UNORM)
 					ConvertRGBA8888ToBGRA8888(dst, src32, width);
 				// This means use original pointer as-is.  May avoid or optimize a copy.
 				else if (srcStride == width)
@@ -1388,10 +1391,10 @@ Draw::Texture *FramebufferManagerCommon::MakePixelTexture(const u8 *srcPixels, G
 				// to do one of two different swizzle operations. However, for the only use of this so far,
 				// the Burnout lens flare trickery, swizzle doesn't matter since it's just a 0, 7fff, 0, 7fff pattern
 				// which comes out the same.
-				if (depthFormat == Draw::DataFormat::R16_UNORM) {
+				if (texFormat == Draw::DataFormat::R16_UNORM) {
 					// We just use this format straight.
 					memcpy(dst16, src16, w * 2);
-				} else if (depthFormat == Draw::DataFormat::R8_UNORM) {
+				} else if (texFormat == Draw::DataFormat::R8_UNORM) {
 					// We fall back to R8_UNORM. Precision is enough for most cases of depth clearing and initialization we've seen,
 					// but hardly ideal.
 					for (int i = 0; i < width; i++) {
@@ -1409,8 +1412,6 @@ Draw::Texture *FramebufferManagerCommon::MakePixelTexture(const u8 *srcPixels, G
 		return true;
 	};
 
-	Draw::DataFormat texFormat = srcPixelFormat == GE_FORMAT_DEPTH16 ? depthFormat : preferredPixelsFormat_;
-
 	int frameNumber = draw_->GetFrameCount();
 
 	// First look for an exact match (including contents hash) that we can re-use.