Skip to content

Commit 24b844c

Browse files
committed
Make Thrust a private dependency of the CUDA library
This was previously a public dependency due to a lingering include of the Thrust pair code in a header, but this dependency is no longer necessary. Thus, we can remove the include from the header file and thereby turn Thrust into a private dependency of the CUDA library. This should also fix the install interface, which is currently broken due to the lack of Thrust setup code.
1 parent 7e7f17c commit 24b844c

File tree

2 files changed

+1
-4
lines changed

2 files changed

+1
-4
lines changed

device/cuda/CMakeLists.txt

+1-1
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ thrust_create_target( traccc::cuda_thrust
113113
HOST CPP
114114
DEVICE CUDA )
115115
target_link_libraries( traccc::cuda_thrust INTERFACE CUDA::cudart )
116-
target_link_libraries( traccc_cuda PUBLIC traccc::cuda_thrust )
116+
target_link_libraries( traccc_cuda PRIVATE traccc::cuda_thrust )
117117

118118
# For CUDA 11 turn on separable compilation. This is necessary for using
119119
# Thrust 2.1.0.

device/cuda/include/traccc/cuda/finding/finding_algorithm.hpp

-3
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,6 @@
3030
#include <vecmem/utils/copy.hpp>
3131
#include <vecmem/utils/cuda/copy.hpp>
3232

33-
// Thrust Library
34-
#include <thrust/pair.h>
35-
3633
namespace traccc::cuda {
3734

3835
/// Track Finding algorithm for a set of tracks

0 commit comments

Comments
 (0)