This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 5974e452bc [enhancement] CRC32 instructions compatible arm arch
(#10261)
5974e452bc is described below
commit 5974e452bc2614118e2d65188055c64fe0e73688
Author: chenlinzhong <[email protected]>
AuthorDate: Mon Jun 20 17:49:06 2022 +0800
[enhancement] CRC32 instructions compatible arm arch (#10261)
The performance of some CPUs that do not implement CRC instructions is
particularly poor
---
be/src/vec/common/hash_table/hash.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/be/src/vec/common/hash_table/hash.h
b/be/src/vec/common/hash_table/hash.h
index 49ba1611c1..3bb1d94269 100644
--- a/be/src/vec/common/hash_table/hash.h
+++ b/be/src/vec/common/hash_table/hash.h
@@ -64,7 +64,7 @@ inline doris::vectorized::UInt64
int_hash64(doris::vectorized::UInt64 x) {
#endif
inline doris::vectorized::UInt64 int_hash_crc32(doris::vectorized::UInt64 x) {
-#if defined(__SSE4_2__) || defined(__aarch64__)
+#if defined(__SSE4_2__) || (defined(__aarch64__) &&
defined(__ARM_FEATURE_CRC32))
return _mm_crc32_u64(-1ULL, x);
#else
/// On other platforms we do not have CRC32. NOTE This can be confusing.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
