Skip to content

Commit

Permalink
Merge branch 'DOR-606_cmake_minimum_version' into 'master'
Browse files Browse the repository at this point in the history
CMake Minimum Version to 3.23

Closes DOR-606

See merge request machine-learning/dorado!883
  • Loading branch information
malton-ont committed Apr 4, 2024
2 parents 39f3c65 + 7934879 commit c88e9f7
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 14 deletions.
8 changes: 4 additions & 4 deletions .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -290,7 +290,7 @@ build:linux:arm64:bionic:unittests:
- build:linux:arm64:bionic

build:linux:arm64:focal:
image: nvcr.io/nvidia/l4t-pytorch:r35.1.0-pth1.13-py3
image: ${DORADO_DOCKER_ROOT}/dorado-l4t-pytorch-r35.1.0-pth1.13-py3:1.0
variables:
BUILD_OPTIONS: "-DONT_MM2_EXE=ON"
extends:
Expand Down Expand Up @@ -433,7 +433,7 @@ build_archive:linux:arm64:bionic:
- .on_nightly_or_manual

build_archive:linux:arm64:focal:
image: nvcr.io/nvidia/l4t-pytorch:r35.1.0-pth1.13-py3
image: ${DORADO_DOCKER_ROOT}/dorado-l4t-pytorch-r35.1.0-pth1.13-py3:1.0
extends:
- .linux_dependencies
- .linux_arm64_runner
Expand Down Expand Up @@ -550,7 +550,7 @@ test:linux:arm64:bionic:
- build:linux:arm64:bionic

test:linux:arm64:focal:
image: nvcr.io/nvidia/l4t-pytorch:r35.1.0-pth1.13-py3
image: ${DORADO_DOCKER_ROOT}/dorado-l4t-pytorch-r35.1.0-pth1.13-py3:1.0
extends:
- .linux_dependencies
- .linux_orin_runner
Expand Down Expand Up @@ -619,7 +619,7 @@ test_archive:linux:arm64:18.04:
- build_archive:linux:arm64:bionic

test_archive:linux:arm64:20.04:
image: nvcr.io/nvidia/l4t-pytorch:r35.1.0-pth1.13-py3
image: ${DORADO_DOCKER_ROOT}/dorado-l4t-pytorch-r35.1.0-pth1.13-py3:1.0
extends:
- .linux_dependencies
- .linux_orin_runner
Expand Down
2 changes: 1 addition & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
cmake_minimum_required(VERSION 3.20)
cmake_minimum_required(VERSION 3.23)
set(CMAKE_POLICY_DEFAULT_CMP0063 NEW)
set(CMAKE_POLICY_DEFAULT_CMP0077 NEW)
set(CMAKE_POLICY_DEFAULT_CMP0091 NEW)
Expand Down
10 changes: 1 addition & 9 deletions cmake/Torch.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -283,14 +283,6 @@ if (USING_STATIC_TORCH_LIB)
)

elseif(CMAKE_SYSTEM_NAME STREQUAL "Linux")
# Older versions of cmake don't have the static_nocallback target for cuFFT that Torch needs, so we
# make it ourselves.
if (TARGET CUDA::cufft_static_nocallback)
set(ont_cufft_static_libs CUDA::cufft_static_nocallback)
else()
set(ont_cufft_static_libs ${CUDAToolkit_TARGET_DIR}/lib64/libcufft_static_nocallback.a)
endif()

# Some CUDA lib symbols have internal linkage, so they must be part of the helper lib too
set(ont_cuda_internal_linkage_libs CUDA::culibos CUDA::cudart_static)
if (TARGET CUDA::cupti_static)
Expand Down Expand Up @@ -387,7 +379,7 @@ if (USING_STATIC_TORCH_LIB)
CUDA::cudart_static
CUDA::cublas_static
CUDA::cublasLt_static
${ont_cufft_static_libs}
CUDA::cufft_static_nocallback
CUDA::cusolver_static
CUDA::cusparse_static
${ont_cuda_internal_linkage_libs}
Expand Down

0 comments on commit c88e9f7

Please sign in to comment.