commit adbc354d23af714a557730cc0e15fc38909e4ef0
author Evgenii Kliuchnikov <eustas@google.com> Tue Jan 16 04:01:35 2024 -0800
committer Copybara-Service <copybara-worker@google.com> Tue Jan 16 04:02:00 2024 -0800
tree 638190febed115afa75fe03640f888a927534786
parent 02458f34431b9850725aafdb8beb1f31efa885fb

simplify log2 check; currently we rely more on compiler than build system

PiperOrigin-RevId: 598794971

CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 70d0f6c..7389560 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -78,27 +78,12 @@
   set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} -fsanitize=${ENABLE_SANITIZER}")
 endif ()
 
-include(CheckFunctionExists)
+include(CheckLibraryExists)
 set(LIBM_LIBRARY)
-CHECK_FUNCTION_EXISTS(log2 LOG2_RES)
-if(NOT LOG2_RES)
-  set(_ORIG_REQ_LIBS "${CMAKE_REQUIRED_LIBRARIES}")
-  set(CMAKE_REQUIRED_LIBRARIES "${CMAKE_REQUIRED_LIBRARIES};m")
-  CHECK_FUNCTION_EXISTS(log2 LOG2_LIBM_RES)
-  if(LOG2_LIBM_RES)
-    set(LIBM_LIBRARY "m")
-    add_definitions(-DBROTLI_HAVE_LOG2=1)
-  else()
-    add_definitions(-DBROTLI_HAVE_LOG2=0)
-  endif()
-
-  set(CMAKE_REQUIRED_LIBRARIES "${_ORIG_REQ_LIBS}")
-  unset(LOG2_LIBM_RES)
-  unset(_ORIG_REQ_LIBS)
-else()
-  add_definitions(-DBROTLI_HAVE_LOG2=1)
+CHECK_LIBRARY_EXISTS(m log2 "" HAVE_LIB_M)
+if(HAVE_LIB_M)
+  set(LIBM_LIBRARY "m")
 endif()
-unset(LOG2_RES)
 
 set(BROTLI_INCLUDE_DIRS "${CMAKE_CURRENT_SOURCE_DIR}/c/include")
 mark_as_advanced(BROTLI_INCLUDE_DIRS)