lemire/Code-used-on-Daniel-Lemire-s-blog

AVX-512 light and heavy operations reversed

travisdowns opened this issue · 2 comments

travisdowns commented

In mandelbrot.c we have the following code:

#ifdef USEHEAVYAVX512
    b = _mm512_add_epi32(b, t);
#else 
    b = _mm512_mul_epi32(b, t);
#endif

Unless I am mistaken, these lines of code should be reversed: the mul should be the heavy operation and the add should be the light operation.

lemire commented

I am going to fix this.

lemire commented

Fixed with the last commit. Thanks.