Skip to content

Commit 695b298

Browse files
committed
Add static linking option
1 parent 2a0dc99 commit 695b298

File tree

4 files changed

+126
-15
lines changed

4 files changed

+126
-15
lines changed

.github/workflows/basic.yml

Lines changed: 70 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -228,6 +228,76 @@ jobs:
228228
--umf-version ${{env.UMF_VERSION}}
229229
${{ matrix.shared_library == 'ON' && '--shared-library' || ''}}
230230
231+
windows-dynamic_build_hwloc:
232+
name: "Windows dynamic UMF + static hwloc"
233+
env:
234+
BUILD_DIR : "${{github.workspace}}/build/${{matrix.build_type}}"
235+
strategy:
236+
matrix:
237+
os: ['windows-2019', 'windows-2022']
238+
build_type: [Release]
239+
240+
runs-on: ${{matrix.os}}
241+
242+
steps:
243+
- name: Checkout
244+
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
245+
246+
- name: Configure build
247+
run: >
248+
cmake
249+
-B ${{env.BUILD_DIR}}
250+
-DUMF_BUILD_SHARED_LIBRARY=ON
251+
-DUMF_FORMAT_CODE_STYLE=OFF
252+
-DUMF_DEVELOPER_MODE=ON
253+
-DUMF_BUILD_LIBUMF_POOL_DISJOINT=ON
254+
-DUMF_BUILD_LIBUMF_POOL_JEMALLOC=OFF
255+
-DUMF_BUILD_LEVEL_ZERO_PROVIDER=ON
256+
-DUMF_TESTS_FAIL_ON_SKIP=ON
257+
-DUMF_LINK_HWLOC_STATICALLY=ON
258+
259+
- name: Build UMF
260+
run: cmake --build ${{env.BUILD_DIR}} --config ${{matrix.build_type}} -j $Env:NUMBER_OF_PROCESSORS
261+
262+
- name: Run tests
263+
working-directory: ${{env.BUILD_DIR}}
264+
run: ctest -C ${{matrix.build_type}} --output-on-failure --test-dir test
265+
266+
windows-static_build_hwloc:
267+
name: "Windows static UMF + static hwloc"
268+
env:
269+
BUILD_DIR : "${{github.workspace}}/build/${{matrix.build_type}}"
270+
strategy:
271+
matrix:
272+
os: ['windows-2019', 'windows-2022']
273+
build_type: [Release]
274+
275+
runs-on: ${{matrix.os}}
276+
277+
steps:
278+
- name: Checkout
279+
uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
280+
281+
- name: Configure build
282+
run: >
283+
cmake
284+
-B ${{env.BUILD_DIR}}
285+
-DUMF_BUILD_SHARED_LIBRARY=OFF
286+
-DUMF_FORMAT_CODE_STYLE=OFF
287+
-DUMF_DEVELOPER_MODE=ON
288+
-DUMF_BUILD_LIBUMF_POOL_DISJOINT=ON
289+
-DUMF_BUILD_LIBUMF_POOL_JEMALLOC=OFF
290+
-DUMF_BUILD_LEVEL_ZERO_PROVIDER=ON
291+
-DUMF_TESTS_FAIL_ON_SKIP=ON
292+
-DUMF_LINK_HWLOC_STATICALLY=ON
293+
294+
- name: Build UMF
295+
run: cmake --build ${{env.BUILD_DIR}} --config ${{matrix.build_type}} -j $Env:NUMBER_OF_PROCESSORS
296+
297+
- name: Run tests
298+
working-directory: ${{env.BUILD_DIR}}
299+
run: ctest -C ${{matrix.build_type}} --output-on-failure --test-dir test
300+
231301
macos-build:
232302
name: MacOS
233303
strategy:

CMakeLists.txt

Lines changed: 50 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,11 @@ set(UMF_CMAKE_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR})
1414
list(APPEND CMAKE_MODULE_PATH "${UMF_CMAKE_SOURCE_DIR}/cmake")
1515
include(${UMF_CMAKE_SOURCE_DIR}/cmake/helpers.cmake)
1616

17+
include(CTest)
18+
include(CMakePackageConfigHelpers)
19+
include(GNUInstallDirs)
20+
find_package(PkgConfig)
21+
1722
# CMAKE_PROJECT_VERSION[_MAJOR|_MINOR|_PATCH] variables are set via 'project'
1823
# command. They cannot contain any "pre-release" part, though. We use custom
1924
# "UMF_SRC_VERSION" to store more accurate (source) version - this var should be
@@ -37,6 +42,8 @@ option(UMF_BUILD_BENCHMARKS "Build UMF benchmarks" OFF)
3742
option(UMF_BUILD_BENCHMARKS_MT "Build UMF multithreaded benchmarks" OFF)
3843
option(UMF_BUILD_EXAMPLES "Build UMF examples" ON)
3944
option(UMF_BUILD_GPU_EXAMPLES "Build UMF GPU examples" OFF)
45+
option(UMF_LINK_HWLOC_STATICALLY
46+
"Link UMF with HWLOC library statically (Windows+Release only)" OFF)
4047
option(UMF_DEVELOPER_MODE "Enable developer checks, treats warnings as errors"
4148
OFF)
4249
option(UMF_FORMAT_CODE_STYLE
@@ -83,6 +90,47 @@ else()
8390
message(FATAL_ERROR "Unknown OS type")
8491
endif()
8592

93+
if(NOT UMF_LINK_HWLOC_STATICALLY)
94+
pkg_check_modules(LIBHWLOC hwloc>=2.3.0)
95+
if(NOT LIBHWLOC_FOUND)
96+
find_package(LIBHWLOC 2.3.0 REQUIRED hwloc)
97+
endif()
98+
# add PATH to DLL on Windows
99+
set(DLL_PATH_LIST
100+
"${DLL_PATH_LIST};PATH=path_list_append:${LIBHWLOC_LIBRARY_DIRS}/../bin"
101+
)
102+
else()
103+
if(NOT WINDOWS)
104+
message(FATAL_ERROR "hwloc can be statically linked only on Windows")
105+
endif()
106+
include(FetchContent)
107+
FetchContent_Declare(
108+
hwloc_targ
109+
GIT_REPOSITORY "https://github.com/open-mpi/hwloc.git"
110+
GIT_TAG hwloc-2.10.0
111+
FIND_PACKAGE_ARGS)
112+
113+
FetchContent_GetProperties(hwloc_targ)
114+
if(NOT hwloc_targ_POPULATED)
115+
FetchContent_MakeAvailable(hwloc_targ)
116+
endif()
117+
118+
set(HWLOC_ENABLE_TESTING OFF)
119+
set(HWLOC_SKIP_LSTOPO ON)
120+
set(HWLOC_SKIP_TOOLS ON)
121+
set(hwloc_cmake ${hwloc_targ_SOURCE_DIR}/contrib/windows-cmake)
122+
123+
add_subdirectory(${hwloc_cmake} ${hwloc_targ_SOURCE_DIR})
124+
125+
set(LIBHWLOC_INCLUDE_DIRS ${hwloc_targ_SOURCE_DIR}/include)
126+
set(LIBHWLOC_LIBRARY_DIRS ${hwloc_targ_SOURCE_DIR}/Release)
127+
set(LIBHWLOC_LIBRARIES ${LIBHWLOC_LIBRARY_DIRS}/hwloc.lib)
128+
129+
message(STATUS " LIBHWLOC_LIBRARIES = ${LIBHWLOC_LIBRARIES}")
130+
message(STATUS " LIBHWLOC_INCLUDE_DIRS = ${LIBHWLOC_INCLUDE_DIRS}")
131+
message(STATUS " LIBHWLOC_LIBRARY_DIRS = ${LIBHWLOC_LIBRARY_DIRS}")
132+
endif()
133+
86134
# This build type check is not possible on Windows when CMAKE_BUILD_TYPE is not
87135
# set, because in this case the build type is determined after a CMake
88136
# configuration is done (at the build time)
@@ -130,11 +178,6 @@ foreach(option_name ${OPTIONS_REQUIRING_CXX})
130178
endif()
131179
endforeach()
132180

133-
include(CTest)
134-
include(CMakePackageConfigHelpers)
135-
include(GNUInstallDirs)
136-
find_package(PkgConfig)
137-
138181
set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib)
139182
set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/lib)
140183
set(CMAKE_UMF_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR}/bin)
@@ -179,7 +222,8 @@ if(WINDOWS)
179222
add_compile_definitions(_CRT_SECURE_NO_WARNINGS)
180223
# set PATH to DLLs on Windows
181224
set(DLL_PATH_LIST
182-
"PATH=path_list_append:${PROJECT_BINARY_DIR}/bin/$<CONFIG>")
225+
"${DLL_PATH_LIST};PATH=path_list_append:${PROJECT_BINARY_DIR}/bin/$<CONFIG>"
226+
)
183227
# add path to the proxy lib DLL
184228
set(DLL_PATH_LIST
185229
"${DLL_PATH_LIST};PATH=path_list_append:${PROJECT_BINARY_DIR}/src/proxy_lib"
@@ -190,14 +234,6 @@ if(WINDOWS)
190234
)
191235
endif()
192236

193-
pkg_check_modules(LIBHWLOC hwloc>=2.3.0)
194-
if(NOT LIBHWLOC_FOUND)
195-
find_package(LIBHWLOC 2.3.0 REQUIRED hwloc)
196-
endif()
197-
# add PATH to DLL on Windows
198-
set(DLL_PATH_LIST
199-
"${DLL_PATH_LIST};PATH=path_list_append:${LIBHWLOC_LIBRARY_DIRS}/../bin")
200-
201237
pkg_check_modules(TBB tbb)
202238
if(NOT TBB_FOUND)
203239
find_package(TBB OPTIONAL_COMPONENTS tbb)

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,7 @@ List of options provided by CMake:
126126
| USE_MSAN | Enable MemorySanitizer checks | ON/OFF | OFF |
127127
| USE_VALGRIND | Enable Valgrind instrumentation | ON/OFF | OFF |
128128
| USE_GCOV | Enable gcov support (Linux only) | ON/OFF | OFF |
129+
| UMF_LINK_HWLOC_STATICALLY | Link UMF with HWLOC library statically (Windows+Release only) | ON/OFF | OFF |
129130

130131
## Architecture: memory pools and providers
131132

src/CMakeLists.txt

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,10 @@ else()
131131
LIBS ${UMF_LIBS})
132132
endif()
133133

134+
if(UMF_LINK_HWLOC_STATICALLY)
135+
add_dependencies(umf hwloc)
136+
endif()
137+
134138
target_link_directories(umf PRIVATE ${UMF_PRIVATE_LIBRARY_DIRS})
135139

136140
target_compile_definitions(umf PRIVATE ${UMF_PRIVATE_COMPILE_DEFINITIONS})
@@ -174,6 +178,6 @@ install(TARGETS umf EXPORT ${PROJECT_NAME}-targets)
174178

175179
add_subdirectory(pool)
176180

177-
if(UMF_PROXY_LIB_ENABLED)
181+
if(UMF_PROXY_LIB_ENABLED AND NOT UMF_LINK_HWLOC_STATICALLY)
178182
add_subdirectory(proxy_lib)
179183
endif()

0 commit comments

Comments
 (0)