reugenio/zcatgui
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

perf(framebuffer): Optimize fillRect with row-wise @memset

Browse source 
For solid colors (alpha=255), use @memset per row instead of
pixel-by-pixel loop. @memset is SIMD-optimized by the compiler
(uses SSE/AVX on x86-64).

Result: Render time 1.4ms → 1.0ms (28% faster in Release build)

Also cleaner code separation between solid color fast path
and alpha blending slow path.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
reugenio 2025-12-10 12:18:47 +01:00
parent e5ba9b178c
commit f7af2ba833
1 changed files with 22 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
src/render/framebuffer.zig
View file

@ -112,6 +112,7 @@ pub const Framebuffer = struct {
}
/// Draw a filled rectangle
/// Optimized with SIMD-friendly @memset for solid colors (alpha=255)
pub fn fillRect(self: *Self, x: i32, y: i32, w: u32, h: u32, color: Color) void {
const x_start = @max(0, x);
const y_start = @max(0, y);
@ -121,16 +122,28 @@ pub const Framebuffer = struct {
if (x_start >= x_end or y_start >= y_end) return;
const c = color.toABGR();
const row_width = @as(u32, @intCast(x_end - x_start));
const ux_start = @as(u32, @intCast(x_start));
// FAST PATH: Solid colors (alpha=255) use @memset which is SIMD-optimized
if (color.a == 255) {
var py: u32 = @intCast(y_start);
const uy_end: u32 = @intCast(y_end);
while (py < uy_end) : (py += 1) {
const row_start = py * self.width + ux_start;
@memset(self.pixels[row_start..][0..row_width], c);
}
return;
}
// SLOW PATH: Alpha blending (pixel by pixel)
if (color.a > 0) {
var py = y_start;
while (py < y_end) : (py += 1) {
const row_start = @as(u32, @intCast(py)) * self.width;
var px = x_start;
while (px < x_end) : (px += 1) {
const idx = row_start + @as(u32, @intCast(px));
if (color.a == 255) {
self.pixels[idx] = c;
} else if (color.a > 0) {
const existing = self.pixels[idx];
const bg = Color{
.r = @truncate(existing),