Skip to content

Commit

Permalink
[clang-tools-extra] Fix linking dylib for LLVMFrontendOpenMP
Browse files Browse the repository at this point in the history
Use LLVM_LINK_COMPONENTS to link the FrontendOpenMP library
instead of passing it explicitly to LINK_LIBS.  This fixes duplicating
the library when clang-tidy is linked to LLVM dylib.

Differential Revision: https://reviews.llvm.org/D71674
  • Loading branch information
mgorny committed Dec 18, 2019
1 parent e3fa460 commit 4121399
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions clang-tools-extra/clang-tidy/openmp/CMakeLists.txt
@@ -1,4 +1,6 @@
set(LLVM_LINK_COMPONENTS support)
set(LLVM_LINK_COMPONENTS
FrontendOpenMP
Support)

add_clang_library(clangTidyOpenMPModule
ExceptionEscapeCheck.cpp
Expand All @@ -11,5 +13,4 @@ add_clang_library(clangTidyOpenMPModule
clangBasic
clangTidy
clangTidyUtils
LLVMFrontendOpenMP
)

0 comments on commit 4121399

Please sign in to comment.