Skip to content

Commit d00e0f7

Browse files
committed
MDEV-33817/MDEV-37170 fixup: Remove evex512
It turns out that there is no need to use the evex512 target attribute that had been introduced in GCC 14 and clang 18, and will be removed in GCC 16. At the time when the evex512 attribute was introduced, Intel had plans to release products with AVX10.1-256, supporting the AVX512 instruction set but limited to 256-bit registers. The evex512 attribute would have been needed to "opt in" to the 512-bit instruction variants (using the EVEX instruction prefix). Later on, Intel revised its plans to make AVX10.1 always 512-bit. Reviewed by: Kristian Nielsen
1 parent 5487c05 commit d00e0f7

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

mysys/crc32/crc32c_x86.cc

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,6 @@
2525
#else
2626
# include <cpuid.h>
2727
# ifdef __APPLE__ /* AVX512 states are not enabled in XCR0 */
28-
# elif __GNUC__ >= 15
29-
# define TARGET "pclmul,avx10.1,vpclmulqdq"
30-
# define USE_VPCLMULQDQ __attribute__((target(TARGET)))
31-
# elif __GNUC__ >= 14 || (defined __clang_major__ && __clang_major__ >= 18)
32-
# define TARGET "pclmul,evex512,avx512f,avx512dq,avx512bw,avx512vl,vpclmulqdq"
33-
# define USE_VPCLMULQDQ __attribute__((target(TARGET)))
3428
# elif __GNUC__ >= 11 || (defined __clang_major__ && __clang_major__ >= 9)
3529
/* clang 8 does not support _xgetbv(), which we also need */
3630
# define TARGET "pclmul,avx512f,avx512dq,avx512bw,avx512vl,vpclmulqdq"

0 commit comments

Comments
 (0)