Skip to content

Commit d0a90e8

Browse files
Collate all ROCm component include dirs into one
1 parent 2b138f4 commit d0a90e8

File tree

2 files changed

+9
-10
lines changed

2 files changed

+9
-10
lines changed

caffe2/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1706,7 +1706,7 @@ if(USE_ROCM)
17061706
target_link_libraries(torch_hip PRIVATE ${Caffe2_HIP_DEPENDENCY_LIBS})
17071707

17081708
# Since PyTorch files contain HIP headers, this is also needed to capture the includes.
1709-
target_include_directories(torch_hip PRIVATE ${Caffe2_HIP_INCLUDE} ${ROCM_INCLUDE})
1709+
target_include_directories(torch_hip PRIVATE ${Caffe2_HIP_INCLUDE} ${ROCM_INCLUDE_DIRS})
17101710
target_include_directories(torch_hip INTERFACE $<INSTALL_INTERFACE:include>)
17111711
endif()
17121712

cmake/public/LoadHIP.cmake

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@ else()
2626
endif()
2727
endif()
2828

29-
if(NOT DEFINED ENV{ROCM_INCLUDE_DIRS})
30-
set(ROCM_INCLUDE_DIRS ${ROCM_PATH}/include)
29+
if(NOT DEFINED ENV{ROCM_INCLUDE_DIR})
30+
set(ROCM_INCLUDE_DIR ${ROCM_PATH}/include)
3131
else()
32-
set(ROCM_INCLUDE_DIRS $ENV{ROCM_INCLUDE_DIRS})
32+
set(ROCM_INCLUDE_DIR $ENV{ROCM_INCLUDE_DIR})
3333
endif()
3434

3535
# MAGMA_HOME
@@ -72,6 +72,7 @@ list(APPEND CMAKE_PREFIX_PATH ${ROCM_PATH})
7272
macro(find_package_and_print_version PACKAGE_NAME)
7373
find_package("${PACKAGE_NAME}" ${ARGN})
7474
message("${PACKAGE_NAME} VERSION: ${${PACKAGE_NAME}_VERSION}")
75+
list(APPEND ROCM_INCLUDE_DIRS ${${PACKAGE_NAME}_INCLUDE_DIR})
7576
endmacro()
7677

7778
# Find the HIP Package
@@ -165,16 +166,14 @@ if(HIP_FOUND)
165166
endif()
166167
find_package_and_print_version(hipblaslt REQUIRED)
167168

168-
list(APPEND ROCM_INCLUDE ${rocthrust_INCLUDE_DIR})
169-
list(APPEND ROCM_INCLUDE ${rocprim_INCLUDE_DIR})
170-
list(APPEND ROCM_INCLUDE ${hipcub_INCLUDE_DIR})
171-
list(APPEND ROCM_INCLUDE ${rocRAND_INCLUDE_DIR})
172-
list(APPEND ROCM_INCLUDE ${INTERFACE_INCLUDE_DIRECTORIES})
173-
174169
if(UNIX)
175170
find_package_and_print_version(rccl)
176171
find_package_and_print_version(hsa-runtime64 REQUIRED)
172+
endif()
177173

174+
list(REMOVE_DUPLICATES ROCM_INCLUDE_DIRS)
175+
176+
if(UNIX)
178177
# roctx is part of roctracer
179178
find_library(ROCM_ROCTX_LIB roctx64 HINTS ${ROCM_PATH}/lib)
180179

0 commit comments

Comments
 (0)