From e67e3d95f3ab863c8b44a6b16ce9f25d40b0d517 Mon Sep 17 00:00:00 2001 From: Yuanyuan Chen Date: Fri, 24 Oct 2025 08:13:17 +0000 Subject: [PATCH] Simplify the CUPTI CMake check for kineto (#161370) Simplify the CUPTI check because kineto has used `CUDA::cupti`. Pull Request resolved: https://github.com/pytorch/pytorch/pull/161370 Approved by: https://github.com/Skylion007 --- cmake/Dependencies.cmake | 35 ++--------------------------------- 1 file changed, 2 insertions(+), 33 deletions(-) diff --git a/cmake/Dependencies.cmake b/cmake/Dependencies.cmake index 733183ef50b..49a97225751 100644 --- a/cmake/Dependencies.cmake +++ b/cmake/Dependencies.cmake @@ -1638,38 +1638,7 @@ if(USE_KINETO) message(STATUS " KINETO_LIBRARY_TYPE = ${KINETO_LIBRARY_TYPE}") if(NOT LIBKINETO_NOCUPTI) - set(CUDA_SOURCE_DIR "${CUDA_TOOLKIT_ROOT_DIR}" CACHE STRING "") - message(STATUS " CUDA_SOURCE_DIR = ${CUDA_SOURCE_DIR}") - message(STATUS " CUDA_INCLUDE_DIRS = ${CUDA_INCLUDE_DIRS}") - - if(NOT MSVC) - if(USE_CUPTI_SO) - set(CUPTI_LIB_NAME "libcupti.so") - else() - set(CUPTI_LIB_NAME "libcupti_static.a") - endif() - else() - set(CUPTI_LIB_NAME "cupti.lib") - endif() - - find_library(CUPTI_LIBRARY_PATH ${CUPTI_LIB_NAME} PATHS - ${CUDA_SOURCE_DIR} - ${CUDA_SOURCE_DIR}/extras/CUPTI/lib64 - ${CUDA_SOURCE_DIR}/lib - ${CUDA_SOURCE_DIR}/lib64 - NO_DEFAULT_PATH) - - find_path(CUPTI_INCLUDE_DIR cupti.h PATHS - ${CUDA_SOURCE_DIR}/extras/CUPTI/include - ${CUDA_INCLUDE_DIRS} - ${CUDA_SOURCE_DIR} - ${CUDA_SOURCE_DIR}/include - NO_DEFAULT_PATH) - - if(CUPTI_LIBRARY_PATH AND CUPTI_INCLUDE_DIR) - message(STATUS " CUPTI_INCLUDE_DIR = ${CUPTI_INCLUDE_DIR}") - set(CUDA_cupti_LIBRARY ${CUPTI_LIBRARY_PATH}) - message(STATUS " CUDA_cupti_LIBRARY = ${CUDA_cupti_LIBRARY}") + if(TARGET CUDA::cupti) message(STATUS "Found CUPTI") set(LIBKINETO_NOCUPTI OFF CACHE STRING "" FORCE) @@ -1682,7 +1651,7 @@ if(USE_KINETO) if(NOT APPLE) set(CMAKE_REQUIRED_LIBRARIES ${CMAKE_REQUIRED_LIBRARIES} "dl" "pthread") endif() - set(CMAKE_REQUIRED_LINK_OPTIONS "-Wl,--whole-archive,${CUPTI_LIBRARY_PATH},--no-whole-archive") + set(CMAKE_REQUIRED_LIBRARIES ${CMAKE_REQUIRED_LIBRARIES} $) check_cxx_source_runs("#include int main() { try {