Fix FindNVTX, thrust::identity<T>, and cuFFT removed error in cuda 13 #1926
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes #1925, which mentioned building issue in cuda 13.
Three issues:
CUDAToolkit_INCLUDE_DIRS
might contain more than one.${list}/suffix
only adds suffix to the last one, which leads thefind_path
failed.thrust::identity<T>
: it is removed in cuda 13. They provide::cuda::std::identity
now. Note. it does not contain conversion likethrust::identity<T>
reference: https://nvidia.github.io/cccl/cccl/3.0_migration_guide.htmlCUFFT_INCOMPLETE_PARAMETER_LIST
,CUFFT_PARSE_ERROR
,CUFFT_LICENSE_ERROR
) reference: https://docs.nvidia.com/cuda/cuda-toolkit-release-notes/index.html#cufft-release-13-0close #1925