Skip to content

Commit f9d6de5

Browse files
author
Jacob Mizraji
committed
Make the llvm-include-tests flag actually do something
Before this patch, it would not actually do anything unless compute_cmake_llvm_tool_disable_flags() was called, which is currently dead code, in the state of build-script-impl currently.
1 parent e37d048 commit f9d6de5

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

utils/build-script-impl

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1973,6 +1973,13 @@ for host in "${ALL_HOSTS[@]}"; do
19731973
-DCLANG_TOOL_CLANG_FORMAT_BUILD=NO
19741974
)
19751975
fi
1976+
1977+
if [[ $(true_false "${LLVM_INCLUDE_TESTS}") == "FALSE" ]]; then
1978+
cmake_options+=(
1979+
-DLLVM_INCLUDE_TESTS=NO
1980+
-DCLANG_INCLUDE_TESTS=NO
1981+
)
1982+
fi
19761983

19771984
if [[ $(is_cross_tools_host ${host}) ]] ; then
19781985
cmake_options=(

0 commit comments

Comments
 (0)