Skip to content

Commit b13f189

Browse files
Merge pull request #49 from psychocoderHPC/fix-wrongCMakeUsage
fix wrong usage of `target_compile_definitions`
2 parents 7d65804 + bc73d33 commit b13f189

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ if(CUDA_MEMTEST_BACKEND STREQUAL "cuda")
147147
else()
148148
target_link_libraries(cuda_memtest PRIVATE hip::host)
149149
target_link_libraries(cuda_memtest PRIVATE hip::device)
150-
target_compile_definitions(cuda_memtest PRIVATE "$<$<COMPILE_LANGUAGE:CXX>:-D__HIP_PLATFORM_AMD__>")
150+
target_compile_definitions(cuda_memtest PRIVATE "$<$<COMPILE_LANGUAGE:CXX>:__HIP_PLATFORM_AMD__>")
151151
target_compile_definitions(cuda_memtest PRIVATE "ENABLE_NVML=0")
152152
endif()
153153

@@ -175,7 +175,7 @@ endif()
175175

176176
option(CUDA_MEMTEST_RELEASE "disable all runtime asserts" ON)
177177
if(CUDA_MEMTEST_RELEASE)
178-
target_compile_definitions(cuda_memtest PRIVATE -DNDEBUG)
178+
target_compile_definitions(cuda_memtest PRIVATE NDEBUG)
179179
endif(CUDA_MEMTEST_RELEASE)
180180

181181
################################################################################

0 commit comments

Comments
 (0)