From de1d8ec0706321287ad747b1d6ffa91377460bd6 Mon Sep 17 00:00:00 2001 From: JosiahWI <41302989+JosiahWI@users.noreply.github.com> Date: Sat, 20 Apr 2024 13:37:46 -0500 Subject: [PATCH] Do not link /latomic on MSVC (#14564) MSVC does not recognize /latomic, resulting in a warning upon configure. --- src/CMakeLists.txt | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 03e914a5f..68249afc8 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -321,11 +321,13 @@ endif() # Note that find_library does not reliably find it so we have to resort to this. # Also, passing -latomic is not always the same as adding atomic to the library list. include(CheckCSourceCompiles) -set(CMAKE_REQUIRED_LIBRARIES "-latomic") -check_c_source_compiles("int main(){}" HAVE_LINK_ATOMIC) -set(CMAKE_REQUIRED_LIBRARIES "") -if(HAVE_LINK_ATOMIC) - set(PLATFORM_LIBS ${PLATFORM_LIBS} "-latomic") +if(NOT CMAKE_CXX_COMPILER_ID STREQUAL "MSVC") + set(CMAKE_REQUIRED_LIBRARIES "-latomic") + check_c_source_compiles("int main(){}" HAVE_LINK_ATOMIC) + set(CMAKE_REQUIRED_LIBRARIES "") + if(HAVE_LINK_ATOMIC) + set(PLATFORM_LIBS ${PLATFORM_LIBS} "-latomic") + endif() endif() include(CheckSymbolExists)