From a203d91b9fa237c6ae27b7fddc8351dde8f56214 Mon Sep 17 00:00:00 2001 From: Yikun Jiang Date: Mon, 15 Jul 2019 11:16:55 -0700 Subject: [PATCH] Cleanup the Arm64 CRC32 unused warning (#5565) Summary: When 'HAVE_ARM64_CRC' is set, the blew methods: - bool rocksdb::crc32c::isSSE42() - bool rocksdb::crc32c::isPCLMULQDQ() are defined but not used, the unused-function is raised when do rocksdb build. This patch try to cleanup these warnings by add ifndef, if it build under the HAVE_ARM64_CRC, we will not define `isSSE42` and `isPCLMULQDQ`. Pull Request resolved: https://github.com/facebook/rocksdb/pull/5565 Differential Revision: D16233654 fbshipit-source-id: c32a9dda7465dbf65f9ccafef159124db92cdffd --- util/crc32c.cc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/util/crc32c.cc b/util/crc32c.cc index 93142dbc0e6..af650192a65 100644 --- a/util/crc32c.cc +++ b/util/crc32c.cc @@ -406,6 +406,8 @@ template __attribute__((target("sse4.2"))) uint32_t ExtendImpl(uint32_t, const char*, size_t); #endif +// Detect if ARM64 CRC or not. +#ifndef HAVE_ARM64_CRC // Detect if SS42 or not. #ifndef HAVE_POWER8 @@ -442,6 +444,7 @@ static bool isPCLMULQDQ() { } #endif // HAVE_POWER8 +#endif // HAVE_ARM64_CRC typedef uint32_t (*Function)(uint32_t, const char*, size_t);