Author: Joseph Huber Date: 2023-03-13T12:41:16-05:00 New Revision: 1b40df1de7a4b3664a55becd776c250f20db0041
URL: https://github.com/llvm/llvm-project/commit/1b40df1de7a4b3664a55becd776c250f20db0041 DIFF: https://github.com/llvm/llvm-project/commit/1b40df1de7a4b3664a55becd776c250f20db0041.diff LOG: [Clang] Fix not including clangBasic in the dynamic build Summary: The previous patch only included the needed library if you had HSA and CUDA installed. Fix that. Added: Modified: clang/tools/amdgpu-arch/CMakeLists.txt clang/tools/nvptx-arch/CMakeLists.txt Removed: ################################################################################ diff --git a/clang/tools/amdgpu-arch/CMakeLists.txt b/clang/tools/amdgpu-arch/CMakeLists.txt index cc65217c7ee2a..4349fb9108a71 100644 --- a/clang/tools/amdgpu-arch/CMakeLists.txt +++ b/clang/tools/amdgpu-arch/CMakeLists.txt @@ -14,8 +14,8 @@ add_clang_tool(amdgpu-arch AMDGPUArch.cpp) find_package(hsa-runtime64 QUIET 1.2.0 HINTS ${CMAKE_INSTALL_PREFIX} PATHS /opt/rocm) if(hsa-runtime64_FOUND AND NOT (LLVM_BUILD_32_BITS OR CMAKE_SIZEOF_VOID_P EQUAL 4)) set_target_properties(amdgpu-arch PROPERTIES INSTALL_RPATH_USE_LINK_PATH ON) - target_link_libraries(amdgpu-arch PRIVATE hsa-runtime64::hsa-runtime64) - clang_target_link_libraries(amdgpu-arch PRIVATE clangBasic) + target_link_libraries(amdgpu-arch PRIVATE hsa-runtime64::hsa-runtime64 clangBasic) else() target_compile_definitions(amdgpu-arch PRIVATE "DYNAMIC_HSA") + target_link_libraries(amdgpu-arch PRIVATE clangBasic) endif() diff --git a/clang/tools/nvptx-arch/CMakeLists.txt b/clang/tools/nvptx-arch/CMakeLists.txt index 1bbff79e89673..3444cf5b70dfd 100644 --- a/clang/tools/nvptx-arch/CMakeLists.txt +++ b/clang/tools/nvptx-arch/CMakeLists.txt @@ -13,8 +13,8 @@ find_package(CUDAToolkit QUIET) # If we found the CUDA library directly we just dynamically link against it. if(CUDAToolkit_FOUND AND NOT (LLVM_BUILD_32_BITS OR CMAKE_SIZEOF_VOID_P EQUAL 4)) - target_link_libraries(nvptx-arch PRIVATE CUDA::cuda_driver) - clang_target_link_libraries(nvptx-arch PRIVATE clangBasic) + target_link_libraries(nvptx-arch PRIVATE CUDA::cuda_driver clangBasic) else() target_compile_definitions(nvptx-arch PRIVATE "DYNAMIC_CUDA") + target_link_libraries(nvptx-arch PRIVATE clangBasic) endif() _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits