Fixed fmin, fmax and fabs for integer batches (avx512)

Author: JohanMabille

include/xsimd/types/xsimd_avx512_int16.hpp

@@ -207,6 +207,7 @@ namespace xsimd
     {
         template <class T>
         struct avx512_int16_batch_kernel
+            : avx512_int_kernel_base<batch<T, 32>>
         {
             using batch_type = batch<T, 32>;
             using value_type = T;

include/xsimd/types/xsimd_avx512_int32.hpp

@@ -155,6 +155,7 @@ namespace xsimd
     {
         template <class T>
         struct avx512_int32_batch_kernel
+            : avx512_int_kernel_base<batch<T, 16>>
         {
             using batch_type = batch<T, 16>;
             using value_type = T;

include/xsimd/types/xsimd_avx512_int64.hpp

@@ -213,6 +213,7 @@ namespace xsimd
     {
         template <class T>
         struct avx512_int64_batch_kernel
+            : avx512_int_kernel_base<batch<T, 8>>
         {
             using batch_type = batch<T, 8>;
             using value_type = T;

include/xsimd/types/xsimd_avx512_int8.hpp

@@ -207,6 +207,7 @@ namespace xsimd
     {
         template <class T>
         struct avx512_int8_batch_kernel
+            : avx512_int_kernel_base<batch<T, 64>>
         {
             using batch_type = batch<T, 64>;
             using value_type = T;

include/xsimd/types/xsimd_avx512_int_base.hpp

@@ -316,6 +316,30 @@ namespace xsimd
         _mm512_storeu_si512(dst, this->m_value);
     }
 
+    namespace detail
+    {
+        template <class B>
+        struct avx512_int_kernel_base
+        {
+            using batch_type = B;
+
+            static batch_type fmin(const batch_type& lhs, const batch_type& rhs)
+            {
+                return min(lhs, rhs);
+            }
+
+            static batch_type fmax(const batch_type& lhs, const batch_type& rhs)
+            {
+                return max(lhs, rhs);
+            }
+
+            static batch_type fabs(const batch_type& rhs)
+            {
+                return abs(rhs);
+            }
+        };
+    }
+
     namespace avx512_detail
     {
         template <class F, class T, std::size_t N>

test/xsimd_basic_math_test.cpp

@@ -79,6 +79,13 @@ TEST(xsimd, avx_double_basic_math)
 #endif
 
 #if XSIMD_X86_INSTR_SET >= XSIMD_X86_AVX512_VERSION
+TEST(xsimd, avx512_int32_basic_math)
+{
+    std::ofstream out("log/avx512_int32_basic_math.log", std::ios_base::out);
+    bool res = xsimd::test_basic_math<int32_t, 16, 64>(out, "avx512 int32_t");
+    EXPECT_TRUE(res);
+}
+
 TEST(xsimd, avx512_float_basic_math)
 {
     std::ofstream out("log/avx512_float_basic_math.log", std::ios_base::out);