On Wed, Apr 02, 2025 at 02:10:40PM +0700, John Naylor wrote: > Thanks for looking, I plan to commit this over the weekend unless > there are objections.
I noticed that autoconf is defining USE_AVX512_CRC_WITH_RUNTIME_CHECK, but everywhere else expects USE_AVX512_CRC32C_WITH_RUNTIME_CHECK (with the "32C" included). I tested the v16 patches (with the macro fixed and assertions enabled) on a machine with AVX-512 (verified with some extra debug logging), and everything passed. -- nathan