diff options
author | Marko Mäkelä <marko.makela@mariadb.com> | 2020-08-27 09:34:53 +0300 |
---|---|---|
committer | Marko Mäkelä <marko.makela@mariadb.com> | 2020-08-27 09:34:53 +0300 |
commit | b47d61d04fe59368a4d2c2de1bd04d9a8b5a20e5 (patch) | |
tree | 829506a641a2ba9e584d3675c83a3acf47f92f76 /config.h.cmake | |
parent | bb284e3fdbfb6fc20f703762a5437c57c5c9d597 (diff) | |
download | mariadb-git-b47d61d04fe59368a4d2c2de1bd04d9a8b5a20e5.tar.gz |
MDEV-23585: Fix HAVE_CLMUL_INSTRUCTION
MDEV-22641 in commit dec3f8ca69e5eb19a4be7a175d3834874c4d880b
refactored a SIMD implementation of CRC-32 for the ISO 3309 polynomial
that uses the IA-32/AMD64 carry-less multiplication (pclmul)
instructions. The code was previously only available in Mariabackup;
it was changed to be a general replacement of the zlib crc32().
There exist AMD64 systems where CMAKE_SYSTEM_PROCESSOR matches
the pattern i[36]86 but not x86_64 or amd64. This would cause a
link failure, because mysys/checksum.c would basically assume that
the compiler support for instruction is always available on GCC-compatible
compilers on AMD64.
Furthermore, we were unnecessarily disabling the SIMD acceleration
for 32-bit executables.
Note: Until MDEV-22749 has been implemented, the PCLMUL instruction
will not be used on Microsoft Windows.
Closes: #1660
Diffstat (limited to 'config.h.cmake')
-rw-r--r-- | config.h.cmake | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/config.h.cmake b/config.h.cmake index ff0a039b3ac..0de5c2be3e0 100644 --- a/config.h.cmake +++ b/config.h.cmake @@ -102,6 +102,8 @@ /* Libraries */ #cmakedefine HAVE_LIBWRAP 1 #cmakedefine HAVE_SYSTEMD 1 + +#cmakedefine HAVE_CLMUL_INSTRUCTION 1 #cmakedefine HAVE_CRC32_VPMSUM 1 /* Support ARMv8 crc + crypto */ |