Skip to content

Commit f252e17

Browse files
Kenovchuravy
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)
1 parent 3f53397 commit f252e17

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

clang/cmake/modules/AddClang.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ macro(add_clang_tool name)
170170

171171
install(TARGETS ${name}
172172
${export_to_clangtargets}
173-
RUNTIME DESTINATION bin
173+
RUNTIME DESTINATION ${LLVM_TOOLS_INSTALL_DIR}
174174
COMPONENT ${name})
175175

176176
if(NOT LLVM_ENABLE_IDE)

0 commit comments

Comments
 (0)