Walkthrough

This PR adds Windows/MSVC compatibility to quantization code by introducing a preprocessor shim in C to handle the popcount operation, and systematically restricts Neon SIMD implementations across multiple Rust metric modules to non-Windows platforms. The C change maps __builtin_popcount to __popcnt for MSVC. The Rust changes add a not(windows) condition to existing aarch64 Neon cfg attributes, preventing Neon-specific code from compiling on Windows while preserving it on other platforms.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

The changes follow a consistent, repetitive pattern across files with minimal logic density. Key areas to verify during review:

• The C preprocessor change in neon.c does not inadvertently affect non-Windows or non-MSVC builds
• The not(windows) condition is consistently applied across all intended locations in the metric modules (verify no conditional compilation paths were missed)
• Fallback implementations exist for Windows builds when Neon is excluded, ensuring no feature gaps
• The changes maintain backward compatibility for existing non-Windows builds

• 📝 Generate docstrings

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

• X
• Mastodon
• Reddit
• LinkedIn