modules/drizzle: low-hanging fruit optimizations

- switch puddle_sample() to 0..1 coordinates to avoid
  some pointless extra arithmetic on every pixel

- avoid redundant ->w multiplies in puddle_sample()

- avoid multiplies in inner loops of drizzle_render_fragment()
  by accumulating coordinates w/addition instead

I noticed full-screen 'compose' was struggling to keep a full
frame rate on my laptop when testing with the new 'plato' layer.

valgrind profiles showed drizzle as the big hog, mostly the
puddle_sample() function.  These changes help but it's still not
great, getting much better will likely become invasive and
crufty.  It would be nice to cache the vertical lerp results and
reuse them across puddle_sample() calls when valid, that might be
a useful TODO.

The runner-up is spiro, prolly some low-hanging fruit there as
well, I haven't looked yet.

1 files changed, 8 insertions, 5 deletions

diff --git a/src/modules/drizzle/drizzle.c b/src/modules/drizzle/drizzle.c
index bc04456..f546cea 100644
--- a/src/modules/drizzle/drizzle.c
+++ b/src/modules/drizzle/drizzle.c
@@ -127,24 +127,27 @@ static void drizzle_prepare_frame(void *context, unsigned ticks, unsigned n_cpus
 static void drizzle_render_fragment(void *context, unsigned ticks, unsigned cpu, fb_fragment_t *fragment)
 {
 	drizzle_context_t	*ctxt = context;
-	float			xf = 2.f / (float)fragment->frame_width;
-	float			yf = 2.f / (float)fragment->frame_height;
+	float			xf = 1.f / (float)fragment->frame_width;
+	float			yf = 1.f / (float)fragment->frame_height;
 	v2f_t			coord;
 
+	coord.y = yf * (float)fragment->y;
 	for (int y = fragment->y; y < fragment->y + fragment->height; y++) {
-		coord.y = yf * (float)y - 1.f;
 
+		coord.x = xf * (float)fragment->x;
 		for (int x = fragment->x; x < fragment->x + fragment->width; x++) {
 			v3f_t		color = {};
 			uint32_t	pixel;
 
-			coord.x = xf * (float)x - 1.f;
-
 			color.z = puddle_sample(ctxt->puddle, &coord);
 
 			pixel = color_to_uint32(color);
 			fb_fragment_put_pixel_unchecked(fragment, x, y, pixel);
+
+			coord.x += xf;
 		}
+
+		coord.y += yf;
 	}
 }