Skip to content
This repository was archived by the owner on May 28, 2025. It is now read-only.

Commit b22bf7e

Browse files
committed
[CMake] Add version to libLLVM also on non-UNIX
As discussed in https://reviews.llvm.org/D87521 llvm-config expects versioned library regardless of platform. Reviewed By: mstorsjo Differential Revision: https://reviews.llvm.org/D89009
1 parent 0a1683f commit b22bf7e

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

llvm/cmake/modules/AddLLVM.cmake

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -606,7 +606,7 @@ function(llvm_add_library name)
606606
endif()
607607
endif()
608608

609-
if(ARG_SHARED AND UNIX)
609+
if(ARG_SHARED)
610610
if(NOT APPLE AND ARG_SONAME)
611611
get_target_property(output_name ${name} OUTPUT_NAME)
612612
if(${output_name} STREQUAL "output_name-NOTFOUND")
@@ -615,10 +615,12 @@ function(llvm_add_library name)
615615
set(library_name ${output_name}-${LLVM_VERSION_MAJOR}${LLVM_VERSION_SUFFIX})
616616
set(api_name ${output_name}-${LLVM_VERSION_MAJOR}.${LLVM_VERSION_MINOR}.${LLVM_VERSION_PATCH}${LLVM_VERSION_SUFFIX})
617617
set_target_properties(${name} PROPERTIES OUTPUT_NAME ${library_name})
618-
llvm_install_library_symlink(${api_name} ${library_name} SHARED
619-
COMPONENT ${name})
620-
llvm_install_library_symlink(${output_name} ${library_name} SHARED
621-
COMPONENT ${name})
618+
if(UNIX)
619+
llvm_install_library_symlink(${api_name} ${library_name} SHARED
620+
COMPONENT ${name})
621+
llvm_install_library_symlink(${output_name} ${library_name} SHARED
622+
COMPONENT ${name})
623+
endif()
622624
endif()
623625
endif()
624626

0 commit comments

Comments
 (0)