fixing shuffle

Author: DiamonDinoia

include/xsimd/arch/common/xsimd_common_swizzle.hpp

@@ -232,12 +232,8 @@ namespace xsimd
             template <uint32_t... Values>
             XSIMD_INLINE constexpr std::uint32_t shuffle() noexcept
             {
-                return shuffle_impl<0,
-                                    sizeof...(Values),
-                                    log2_c<sizeof...(Values)>::value,
-                                    Values...>::value;
+                return shuffle_impl<0, sizeof...(Values), log2_c<sizeof...(Values)>::value, Values...>::value;
             }
-
             template <uint32_t... Values>
             XSIMD_INLINE constexpr std::uint32_t mod_shuffle() noexcept
             {

include/xsimd/arch/xsimd_avx2.hpp

@@ -964,14 +964,15 @@ namespace xsimd
                 constexpr auto imm = ((V0 & 1) << 0) | ((V1 & 1) << 1) | ((V2 & 1) << 2) | ((V3 & 1) << 3);
                 return _mm256_permute_pd(self, imm);
             }
+            constexpr auto imm = detail::mod_shuffle<V0, V1, V2, V3>();
             // fallback to full 4-element permute
-            return _mm256_permute4x64_pd(self, detail::shuffle<V0, V1, V2, V3>());
+            return _mm256_permute4x64_pd(self, imm);
         }
 
         template <class A, uint64_t V0, uint64_t V1, uint64_t V2, uint64_t V3>
         XSIMD_INLINE batch<uint64_t, A> swizzle(batch<uint64_t, A> const& self, batch_constant<uint64_t, A, V0, V1, V2, V3>, requires_arch<avx2>) noexcept
         {
-            constexpr auto mask = detail::shuffle<V0, V1, V2, V3>();
+            constexpr auto mask = detail::mod_shuffle<V0, V1, V2, V3>();
             return _mm256_permute4x64_epi64(self, mask);
         }
         template <class A, uint64_t V0, uint64_t V1, uint64_t V2, uint64_t V3>