Skip to content

Commit 0253220

Browse files
authored
Merge pull request swiftlang#34250 from akyrtzi/parser-lib-build-less-build
[parser lib build] For the "parser lib only" build, avoid building unnecessary llvm libraries that clangBasic brings
2 parents a482c9c + b848bf7 commit 0253220

File tree

1 file changed

+8
-9
lines changed

1 file changed

+8
-9
lines changed

lib/AST/CMakeLists.txt

Lines changed: 8 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -126,15 +126,14 @@ target_link_libraries(swiftAST
126126
PUBLIC swiftBasic
127127
PRIVATE swiftSyntax)
128128
if(SWIFT_BUILD_ONLY_SYNTAXPARSERLIB)
129-
# Add clangBasic as a single direct dependency to avoid bringing along some
130-
# llvm libraries that we don't need.
131-
if("${SWIFT_HOST_VARIANT_SDK}" STREQUAL "WINDOWS")
132-
target_link_libraries(swiftAST PRIVATE
133-
"${LLVM_LIBRARY_OUTPUT_INTDIR}/clangBasic.lib")
134-
else()
135-
target_link_libraries(swiftAST PRIVATE
136-
"${LLVM_LIBRARY_OUTPUT_INTDIR}/libclangBasic.a")
137-
endif()
129+
# Remove dependencies from clangBasic to avoid bringing along some llvm
130+
# libraries that we don't need to be building.
131+
set_property(TARGET clangBasic PROPERTY LINK_LIBRARIES "")
132+
set_property(TARGET clangBasic PROPERTY LINK_DEPENDS "")
133+
set_property(TARGET clangBasic PROPERTY INTERFACE_LINK_LIBRARIES "")
134+
set_property(TARGET clangBasic PROPERTY INTERFACE_LINK_DEPENDS "")
135+
target_link_libraries(swiftAST
136+
PRIVATE clangBasic)
138137
target_compile_definitions(swiftAST PRIVATE
139138
SWIFT_BUILD_ONLY_SYNTAXPARSERLIB=1)
140139
else()

0 commit comments

Comments
 (0)