arch/generic/crc32_braid_c.c (1)

209-211: Remove duplicate crc32_braid definition (build breaks).

There are two crc32_braid definitions, and the second one calls crc32_braid_internal, which no longer exists. This matches the CI redefinition/implicit-declaration failures.

🧹 Suggested fix (remove the stale wrapper)

-Z_INTERNAL uint32_t crc32_braid(uint32_t crc, const uint8_t *buf, size_t len) {
-    return ~crc32_braid_internal(~crc, buf, len);
-}

arch/x86/chorba_sse2.c (1)

23-846: SSE2 small chorba now returns pre-conditioned CRC.

After the inversion refactor, this path still seeds with raw crc and returns crc without a final complement, which likely makes SSE2 results inconsistent with generic paths and the updated wrapper behavior.

✅ Suggested fix to re-align with post-conditioned outputs

-    uint64_t next1 = crc;
+    uint64_t next1 = ~crc;
@@
-    return crc;
+    return ~crc;

arch/x86/chorba_sse41.c (1)

55-303: SSE4.1 32K chorba path still missing CRC complement.

With the wrapper now returning directly, this function still treats crc as pre-conditioned and returns it without the final ~, which likely breaks parity with the generic path.

✅ Suggested fix to re-align with post-conditioned outputs

-    __m128i init_crc = _mm_cvtsi64_si128(crc);
+    __m128i init_crc = _mm_cvtsi64_si128(~crc);
@@
-    return crc;
+    return ~crc;