Skip to content

Commit 245b0fa

Browse files
stellaraccidenttaylding-amd
authored andcommitted
Extend the hack to propagate HIP usage requirements a bit further.
1 parent f3ea8b1 commit 245b0fa

File tree

3 files changed

+12
-1
lines changed

3 files changed

+12
-1
lines changed

cmake/rocprofiler_config_interfaces.cmake

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,14 @@ find_package(
127127
${rocm_version_DIR}
128128
${ROCM_PATH})
129129
target_link_libraries(rocprofiler-sdk-hip INTERFACE hip::host)
130+
# TODO: As of 2024/2/12, the hip::host target does not advertise its
131+
# include directory but amdhip64 does. This ordinarily wouldn't be an issue
132+
# because most folks just get it transitively, but here this is doing direct
133+
# property copying to get usage requirements.
134+
# The proper fix is for hip to export a hip::headers target with only usage
135+
# requirements and depend on that.
130136
rocprofiler_config_nolink_target(rocprofiler-sdk-hip-nolink hip::host)
137+
rocprofiler_config_nolink_target(rocprofiler-sdk-hip-nolink hip::amdhip64)
131138

132139
# ----------------------------------------------------------------------------------------#
133140
#
@@ -218,7 +225,9 @@ find_library(
218225
HINTS ${rocm_version_DIR} ${ROCM_PATH}
219226
PATHS ${rocm_version_DIR} ${ROCM_PATH})
220227

221-
target_link_libraries(rocprofiler-sdk-hsa-aql INTERFACE ${hsa-amd-aqlprofile64_library})
228+
if(hsa-amd-aqlprofile64_library)
229+
target_link_libraries(rocprofiler-sdk-hsa-aql INTERFACE ${hsa-amd-aqlprofile64_library})
230+
endif()
222231

223232
# ----------------------------------------------------------------------------------------#
224233
#

source/lib/common/container/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,3 +9,4 @@ set(containers_sources ring_buffer.cpp record_header_buffer.cpp ring_buffer.cpp
99

1010
target_sources(rocprofiler-sdk-common-library PRIVATE ${containers_sources}
1111
${containers_headers})
12+
target_link_libraries(rocprofiler-sdk-common-library PRIVATE rocprofiler-sdk-hip-nolink)

source/lib/output/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ target_link_libraries(
6262
rocprofiler-sdk::rocprofiler-sdk-cereal
6363
rocprofiler-sdk::rocprofiler-sdk-perfetto
6464
rocprofiler-sdk::rocprofiler-sdk-otf2
65+
rocprofiler-sdk-hip-nolink
6566
rocprofiler-sdk::rocprofiler-sdk-amd-comgr
6667
rocprofiler-sdk::rocprofiler-sdk-dw
6768
rocprofiler-sdk::rocprofiler-sdk-elf)

0 commit comments

Comments
 (0)