-
-
Notifications
You must be signed in to change notification settings - Fork 56.5k
Closed
Labels
category: build/installcategory: gpu/cuda (contrib)OpenCV 4.0+: moved to opencv_contribOpenCV 4.0+: moved to opencv_contribwontfix
Description
System Information
Latest OpenCV 4.x main as of today
Detailed description
The main CMakeLists.txt defines USER_DEFINED_CMAKE_CUDA_ARCHITECTURES here:
Line 687 in 6eaaaa4
| if(CMAKE_CUDA_ARCHITECTURES) |
However this variable isn't used anywhere else in any CMake script, it seems:
opencv$ grep -R --include='*[cC][mM][aA][kK][eE]*' --exclude-dir={build,install} "USER_DEFINED_CMAKE_CUDA_ARCHITECTURES" .
./CMakeLists.txt: set(USER_DEFINED_CMAKE_CUDA_ARCHITECTURES TRUE)
./CMakeLists.txt: if(NOT USER_DEFINED_CMAKE_CUDA_ARCHITECTURES)
Perhaps this is an artefact from the past?
It seems this code could be simplified to the following, since this is all it seems to be doing:
if(NOT CMAKE_CUDA_ARCHITECTURES)
set(CMAKE_CUDA_ARCHITECTURES "")
endif()
enable_language(CUDA)
And that feels a bit pointless?
Steps to reproduce
See above.
Issue submission checklist
- I report the issue, it's not a question
- I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
- I updated to the latest OpenCV version and the issue is still there
- There is reproducer code and related data files (videos, images, onnx, etc)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
category: build/installcategory: gpu/cuda (contrib)OpenCV 4.0+: moved to opencv_contribOpenCV 4.0+: moved to opencv_contribwontfix