Fix portability issue with avx512f upper_half of batch of float

serge-sans-paille · serge-sans-paille · commit f3ad5f11b99e · 2025-10-31T19:00:46.000Z
It was relying on an intrinsic only available with avx512dq
diff --git a/include/xsimd/arch/xsimd_avx512f.hpp b/include/xsimd/arch/xsimd_avx512f.hpp
@@ -55,7 +55,7 @@ namespace xsimd
             }
             XSIMD_INLINE __m256 upper_half(__m512 self) noexcept
             {
-                return _mm512_extractf32x8_ps(self, 1);
+                return _mm256_castsi256_ps(_mm512_extracti64x4_epi64(_mm512_castps_si512(self), 1));
             }
             XSIMD_INLINE __m256d upper_half(__m512d self) noexcept
             {

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ namespace xsimd`
`55`	`55`	`}`
`56`	`56`	`XSIMD_INLINE __m256 upper_half(__m512 self) noexcept`
`57`	`57`	`{`
`58`		`- return _mm512_extractf32x8_ps(self, 1);`
	`58`	`+ return _mm256_castsi256_ps(_mm512_extracti64x4_epi64(_mm512_castps_si512(self), 1));`
`59`	`59`	`}`
`60`	`60`	`XSIMD_INLINE __m256d upper_half(__m512d self) noexcept`
`61`	`61`	`{`