[PATCH] D39988: [CMake][runtimes] Don't passthrough prefixes for non-default targets
Petr Hosek via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 13 16:39:14 PST 2017
phosek created this revision.
Herald added a subscriber: mgorny.
The passthrough is useful for setting up the options for the default
build, but we already have a different mechanism to pass CMake flags
to builds for builtins and runtimes targets so this is not really
needed there. Furthermore, when the flags are set for the default
build, with the prefix passthrough set we have to explicitly override
all options in other targets which can be cumbersome.
Repository:
rL LLVM
https://reviews.llvm.org/D39988
Files:
runtimes/CMakeLists.txt
Index: runtimes/CMakeLists.txt
===================================================================
--- runtimes/CMakeLists.txt
+++ runtimes/CMakeLists.txt
@@ -249,7 +249,6 @@
-DCOMPILER_RT_DEFAULT_TARGET_ONLY=On
${${target}_extra_args}
TOOLCHAIN_TOOLS clang lld llvm-ar llvm-ranlib
- PASSTHROUGH_PREFIXES COMPILER_RT
USE_TOOLCHAIN
${EXTRA_ARGS})
endfunction()
@@ -336,7 +335,7 @@
# runtime_register_target(target)
# Utility function to register external runtime target.
- function(runtime_register_target name target deps prefixes)
+ function(runtime_register_target name target deps)
include(${LLVM_BINARY_DIR}/runtimes/${name}/Components.cmake OPTIONAL)
set_property(DIRECTORY APPEND PROPERTY CMAKE_CONFIGURE_DEPENDS ${LLVM_BINARY_DIR}/runtimes/${name}/Components.cmake)
@@ -397,7 +396,6 @@
-DLLVM_RUNTIMES_TARGET=${name}
${${name}_extra_args}
TOOLCHAIN_TOOLS clang lld llvm-ar llvm-ranlib
- PASSTHROUGH_PREFIXES ${prefixes}
EXTRA_TARGETS ${${name}_extra_targets}
${${name}_test_targets}
USE_TOOLCHAIN
@@ -434,7 +432,7 @@
list(GET target_list 1 target)
endif()
- runtime_register_target(${name} ${target} ${deps} ${prefixes})
+ runtime_register_target(${name} ${target} ${deps})
add_dependencies(runtimes runtimes-${name})
add_dependencies(runtimes-configure runtimes-${name}-configure)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D39988.122746.patch
Type: text/x-patch
Size: 1816 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171114/8b0f3657/attachment.bin>
More information about the llvm-commits
mailing list