[llvm] r251001 - Fix add_llvm_external_project.
Manuel Klimek via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 22 01:31:46 PDT 2015
Author: klimek
Date: Thu Oct 22 03:31:46 2015
New Revision: 251001
URL: http://llvm.org/viewvc/llvm-project?rev=251001&view=rev
Log:
Fix add_llvm_external_project.
r250835 unintentionally discarded the optional parameter to the
add_llvm_external_project() macro that may point to a path when the said
path is different from ${name}. This should fix it by passing ${ARGN} on
to add_llvm_subdirectory(). The problem manifests itself with e.g.
add_llvm_external_project(clang-tools-extra extra) from
clang/tools/CMakeLists.txt
Patch by Luchesar V. Iliev.
Modified:
llvm/trunk/cmake/modules/AddLLVM.cmake
Modified: llvm/trunk/cmake/modules/AddLLVM.cmake
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/cmake/modules/AddLLVM.cmake?rev=251001&r1=251000&r2=251001&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/AddLLVM.cmake (original)
+++ llvm/trunk/cmake/modules/AddLLVM.cmake Thu Oct 22 03:31:46 2015
@@ -781,7 +781,7 @@ endmacro()
# enable or disable building it with everything else.
# Additional parameter can be specified as the name of directory.
macro(add_llvm_external_project name)
- add_llvm_subdirectory(LLVM TOOL ${name})
+ add_llvm_subdirectory(LLVM TOOL ${name} ${ARGN})
endmacro()
macro(add_llvm_tool_subdirectory name)
More information about the llvm-commits
mailing list