Optimize batch_bool load in avx512f

Author: junparser

include/xsimd/arch/xsimd_avx512bw.hpp

@@ -320,11 +320,15 @@ namespace xsimd
         XSIMD_INLINE batch_bool<T, A> load(bool const* mem, batch_bool<T, A>, requires_arch<avx512bw>) noexcept
         {
             using register_type = typename batch_bool<T, A>::register_type;
-            constexpr auto size = batch_bool<T, A>::size;
-            __mmask64 mask = size >= 64 ? ~(__mmask64)0 : (1ULL << size) - 1;
-            __m512i zeros = _mm512_setzero_si512();
-            __m512i bool_val = _mm512_mask_loadu_epi8(zeros, mask, (void*)mem);
-            return (register_type)_mm512_cmpgt_epu8_mask(bool_val, zeros);
+            XSIMD_IF_CONSTEXPR(batch_bool<T, A>::size == 64)
+            {
+                __m512i bool_val = _mm512_loadu_si512((__m512i const*)mem);
+                return (register_type)_mm512_cmpgt_epu8_mask(bool_val, _mm512_setzero_si512());
+            }
+            else
+            {
+                return load(mem, batch_bool<T, A>(), avx512dq {});
+            }
         }
 
         // max

include/xsimd/arch/xsimd_avx512f.hpp

@@ -1193,16 +1193,35 @@ namespace xsimd
             return detail::compare_int_avx512f<A, T, _MM_CMPINT_LE>(self, other);
         }
 
+        namespace detail
+        {
+            // Generate a bitset from an array of boolean.
+            XSIMD_INLINE unsigned char tobitset(unsigned char unpacked[8])
+            {
+                uint64_t data;
+                memcpy(&data, unpacked, sizeof(uint64_t));
+
+                const uint64_t magic = (0x80 + 0x4000 + 0x200000 + 0x10000000 + 0x0800000000 + 0x040000000000 + 0x02000000000000 + 0x0100000000000000);
+
+                unsigned char res = ((data * magic) >> 56) & 0xFF;
+                return res;
+            }
+        }
+
         // load
         template <class A, class T>
         XSIMD_INLINE batch_bool<T, A> load(bool const* mem, batch_bool<T, A>, requires_arch<avx512f>) noexcept
         {
             using register_type = typename batch_bool<T, A>::register_type;
             constexpr auto size = batch_bool<T, A>::size;
+            constexpr auto iter = size / 8;
+            assert((size % 8 == 0) && "ucorrectly size of bool batch");
             register_type mask = 0;
-            for (std::size_t i = 0; i < size; ++i)
-                mask |= (register_type(mem[i] ? 1 : 0) << i);
-
+            for (std::size_t i = 0; i < iter; ++i)
+            {
+                unsigned char block = detail::tobitset((unsigned char*)mem + i * 8);
+                mask |= (register_type(block) << (i * 8));
+            }
             return mask;
         }