Skip to content

Commit 2794694

Browse files
Kenomaleadt
authored andcommitted
[clang/CMake] Respect LLVM_TOOLS_INSTALL_DIR
Otherwise clang installs all of its tools into `bin/` while LLVM installs its tools into (LLVM_TOOLS_INSTALL_DIR). I could swear this used to work (and in fact the julia build system assumes it), but I can't pin down a specific commit that would have broken this, and julia has been relying on pre-compiled binaries for a while now (that don't use this setting), so it may have been broken for quite a while. Differential Revision: https://reviews.llvm.org/D88630 (cherry picked from commit 6104e14) (cherry picked from commit f252e17) (cherry picked from commit 9039ce8)
1 parent 0debc4a commit 2794694

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

clang/cmake/modules/AddClang.cmake

+1-1
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ macro(add_clang_tool name)
169169
get_target_export_arg(${name} Clang export_to_clangtargets)
170170
install(TARGETS ${name}
171171
${export_to_clangtargets}
172-
RUNTIME DESTINATION "${CMAKE_INSTALL_BINDIR}"
172+
RUNTIME DESTINATION "${LLVM_TOOLS_INSTALL_DIR}"
173173
COMPONENT ${name})
174174

175175
if(NOT LLVM_ENABLE_IDE)

0 commit comments

Comments
 (0)