all repos — mgba @ d5445799a6c6923c9978939913c33237aadd2cf8

mGBA Game Boy Advance Emulator

DS GX: Add polygon IDs to the framebuffer
Vicki Pfau vi@endrift.com
Tue, 21 Mar 2017 15:15:06 -0700
commit

d5445799a6c6923c9978939913c33237aadd2cf8

parent

d7ce8f9f0660cfab0990cc1c1e7b1fc43dae9b3d

3 files changed, 22 insertions(+), 11 deletions(-)

jump to
M include/mgba/internal/ds/gx.h → include/mgba/internal/ds/gx.h 
@@ -55,6 +55,7 @@ DECL_BIT(DSGXPolygonAttrs, BackFace, 7);
 DECL_BIT(DSGXPolygonAttrs, UpdateDepth, 11);
 // TODO
 DECL_BITS(DSGXPolygonAttrs, Alpha, 16, 5);
+DECL_BITS(DSGXPolygonAttrs, Id, 24, 6);
 
 enum DSGXCommand {
 	DS_GX_CMD_NOP = 0,
M include/mgba/internal/ds/gx/software.h → include/mgba/internal/ds/gx/software.h 
@@ -70,6 +70,7 @@ };
 
 struct DSGXSoftwareSpan {
 	struct DSGXSoftwarePolygon* poly;
+	int polyId;
 	struct DSGXSoftwareEndpoint ep[2];
 	struct DSGXSoftwareEndpoint step;
 };

@@ -87,6 +88,7 @@ struct DSGXSoftwareSpanList activeSpans;
 	struct DSGXSoftwareSpan** bucket;
 
 	int32_t depthBuffer[DS_VIDEO_HORIZONTAL_PIXELS];
+	uint8_t polygonIdBuffer[DS_VIDEO_HORIZONTAL_PIXELS];
 	color_t* scanlineCache;
 	bool flushPending;
 	bool wSort;
M src/ds/gx/software.c → src/ds/gx/software.c 
@@ -639,6 +639,7 @@ } else if (!span) {
 			span = DSGXSoftwareSpanListAppend(&softwareRenderer->activeSpans);
 			memset(&span->ep[1], 0, sizeof(span->ep[1]));
 			span->poly = DSGXSoftwarePolygonListGetPointer(&softwareRenderer->activePolys, poly);
+			span->polyId = DSGXPolygonAttrsGetId(span->poly->poly->polyParams);
 			if (!_edgeToSpan(span, edge, 0, y << 12)) {
 				// Horizontal line
 				DSGXSoftwareSpanListShift(&softwareRenderer->activeSpans, DSGXSoftwareSpanListSize(&softwareRenderer->activeSpans) - 1, 1);

@@ -650,6 +651,7 @@ }
 
 	color_t* scanline = &softwareRenderer->scanlineCache[DS_VIDEO_HORIZONTAL_PIXELS * y];
 	memset(scanline, 0, sizeof(color_t) * DS_VIDEO_HORIZONTAL_PIXELS);
+	memset(softwareRenderer->polygonIdBuffer, 0, sizeof(softwareRenderer->polygonIdBuffer[0]) * DS_VIDEO_HORIZONTAL_PIXELS);
 	for (i = 0; i < DS_VIDEO_HORIZONTAL_PIXELS; i += 4) {
 		softwareRenderer->depthBuffer[i] = INT32_MAX;
 		softwareRenderer->depthBuffer[i + 1] = INT32_MAX;

@@ -678,11 +680,13 @@ if (a == 0x1F) {
 				if (softwareRenderer->wSort) {
 					if (span->ep[0].w < softwareRenderer->depthBuffer[x]) {
 						softwareRenderer->depthBuffer[x] = span->ep[0].w;
+						softwareRenderer->polygonIdBuffer[x] = span->polyId;
 						scanline[x] = color;
 					}
 				} else {
 					if (span->ep[0].z < softwareRenderer->depthBuffer[x]) {
 						softwareRenderer->depthBuffer[x] = span->ep[0].z;
+						softwareRenderer->polygonIdBuffer[x] = span->polyId;
 						scanline[x] = color;
 					}
 				}

@@ -692,19 +696,23 @@ if (b) {
 					color = _mix32(a, color, 0x1F - a, current);
 					color |= ab << 27;
 				}
-				if (softwareRenderer->wSort) {
-					if (span->ep[0].w < softwareRenderer->depthBuffer[x]) {
-						if (DSGXPolygonAttrsIsUpdateDepth(span->poly->poly->polyParams)) {
-							softwareRenderer->depthBuffer[x] = span->ep[0].w;
+				if (softwareRenderer->polygonIdBuffer[x] != span->polyId) {
+					if (softwareRenderer->wSort) {
+						if (span->ep[0].w < softwareRenderer->depthBuffer[x]) {
+							if (DSGXPolygonAttrsIsUpdateDepth(span->poly->poly->polyParams)) {
+								softwareRenderer->depthBuffer[x] = span->ep[0].w;
+							}
+							softwareRenderer->polygonIdBuffer[x] = span->polyId;
+							scanline[x] = color;
 						}
-						scanline[x] = color;
-					}
-				} else {
-					if (span->ep[0].z < softwareRenderer->depthBuffer[x]) {
-						if (DSGXPolygonAttrsIsUpdateDepth(span->poly->poly->polyParams)) {
-							softwareRenderer->depthBuffer[x] = span->ep[0].z;
+					} else {
+						if (span->ep[0].z < softwareRenderer->depthBuffer[x]) {
+							if (DSGXPolygonAttrsIsUpdateDepth(span->poly->poly->polyParams)) {
+								softwareRenderer->depthBuffer[x] = span->ep[0].z;
+							}
+							softwareRenderer->polygonIdBuffer[x] = span->polyId;
+							scanline[x] = color;
 						}
-						scanline[x] = color;
 					}
 				}
 			}