Skip to content

Commit

Permalink
Meta: Don't pass LLVM_LLVM_BUILD_LLVM_DYLIB to cmake
Browse files Browse the repository at this point in the history
`LLVM_LLVM_BUILD_LLVM_DYLIB` does not exist, so passing this does
nothing but make CMake warn.

However, since we pass `LLVM_LINK_LLVM_DYLIB`, `LLVM_BUILD_LLVM_DYLIB`
(the correct spelling) defaults to true anyways.  So let's pass fewer
flags.

No behavior change, but fixes a CMake warning.
  • Loading branch information
nico authored and linusg committed Sep 7, 2021
1 parent 1ed1a57 commit eb7b8a7
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion Toolchain/BuildClang.sh
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,6 @@ pushd "$DIR/Build/clang/$ARCH"
-DLLVM_ENABLE_PROJECTS="clang;lld" \
-DLLVM_INCLUDE_BENCHMARKS=OFF \
-DLLVM_INCLUDE_TESTS=OFF \
-DLLVM_LLVM_BUILD_LLVM_DYLIB=ON \
-DLLVM_LINK_LLVM_DYLIB=ON \
-DLLVM_INSTALL_UTILS=OFF \
${dev:+"-DLLVM_CCACHE_BUILD=ON"} || exit 1
Expand Down

0 comments on commit eb7b8a7

Please sign in to comment.