Frustum cull: Eliminate two divisions

hrydgard · Nov 13, 2023 · 1746c35 · 1746c35
1 parent c485df7
commit 1746c35
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/GPU/Common/DrawEngineCommon.cpp b/GPU/Common/DrawEngineCommon.cpp
@@ -219,15 +219,17 @@ void DrawEngineCommon::UpdatePlanes() {
 	Vec2f minViewport = (minOffset_ - Vec2f(gstate.getViewportXCenter(), gstate.getViewportYCenter())) * inverseViewportScale;
 	Vec2f maxViewport = (maxOffset_ - Vec2f(gstate.getViewportXCenter(), gstate.getViewportYCenter())) * inverseViewportScale;
 
+	Vec2f viewportInvSize = Vec2f(1.0f / (maxViewport.x - minViewport.x), 1.0f / (maxViewport.y - minViewport.y));
+
 	Lin::Matrix4x4 applyViewport{};
 	// Scale to the viewport's size.
-	applyViewport.xx = 2.0f / (maxViewport.x - minViewport.x);
-	applyViewport.yy = 2.0f / (maxViewport.y - minViewport.y);
+	applyViewport.xx = 2.0f * viewportInvSize.x;
+	applyViewport.yy = 2.0f * viewportInvSize.y;
 	applyViewport.zz = 1.0f;
 	applyViewport.ww = 1.0f;
 	// And offset to the viewport's centers.
-	applyViewport.wx = -(maxViewport.x + minViewport.x) / (maxViewport.x - minViewport.x);
-	applyViewport.wy = -(maxViewport.y + minViewport.y) / (maxViewport.y - minViewport.y);
+	applyViewport.wx = -(maxViewport.x + minViewport.x) * viewportInvSize.x;
+	applyViewport.wy = -(maxViewport.y + minViewport.y) * viewportInvSize.y;
 
 	float mtx[16];
 	Matrix4ByMatrix4(mtx, worldviewproj, applyViewport.m);