From: Nathan Moinvaziri <nathan@nathanm.com>
Date: Tue, 17 Mar 2026 02:03:25 +0000 (-0700)
Subject: Call adler32_c directly in adler32_copy_c scalar fallback
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7fe648e89172a45a09b807a16e6c157e5e2cf0f3;p=thirdparty%2Fzlib-ng.git

Call adler32_c directly in adler32_copy_c scalar fallback

The generic copy function was calling through the function
table, which dispatched to the best SIMD implementation
instead of the scalar path. This led to incorrect benchmarks
for adler32_copy/c since it measured the SIMD path rather
than the scalar fallback. Call adler32_c directly so the
scalar copy variant actually exercises the scalar checksum.
---

diff --git a/arch/generic/adler32_c.c b/arch/generic/adler32_c.c
index 755d60a62..5161406bc 100644
--- a/arch/generic/adler32_c.c
+++ b/arch/generic/adler32_c.c
@@ -54,7 +54,7 @@ Z_INTERNAL uint32_t adler32_c(uint32_t adler, const uint8_t *buf, size_t len) {
 }
 
 Z_INTERNAL uint32_t adler32_copy_c(uint32_t adler, uint8_t *dst, const uint8_t *src, size_t len) {
-    adler = FUNCTABLE_CALL(adler32)(adler, src, len);
+    adler = adler32_c(adler, src, len);
     memcpy(dst, src, len);
     return adler;
 }