[llvm] r201283 - [CMake] llvm_add_library: Add handling of the parameter ADDITIONAL_HEADERS to pass through to process_source.
NAKAMURA Takumi
geek4civic at gmail.com
Wed Feb 12 17:00:52 PST 2014
Author: chapuni
Date: Wed Feb 12 19:00:52 2014
New Revision: 201283
URL: http://llvm.org/viewvc/llvm-project?rev=201283&view=rev
Log:
[CMake] llvm_add_library: Add handling of the parameter ADDITIONAL_HEADERS to pass through to process_source.
I was insightless then about unknown optional parameters.
(Consider that LINK_LIBS foo bar ADDITIONAL_HEADERS qux quux)
Suggested by Michael Kruse. Thanks!
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=201283&r1=201282&r2=201283&view=diff
==============================================================================
--- llvm/trunk/cmake/modules/AddLLVM.cmake (original)
+++ llvm/trunk/cmake/modules/AddLLVM.cmake Wed Feb 12 19:00:52 2014
@@ -174,17 +174,21 @@ endfunction()
# Same as the variable LLVM_LINK_COMPONENTS.
# LINK_LIBS lib_targets...
# Same semantics as target_link_libraries().
-# ADDITIONAL_HEADERS (implemented in LLVMProcessSources)
+# ADDITIONAL_HEADERS
# May specify header files for IDE generators.
# )
function(llvm_add_library name)
cmake_parse_arguments(ARG
"MODULE;SHARED;STATIC"
"OUTPUT_NAME"
- "DEPENDS;LINK_COMPONENTS;LINK_LIBS"
+ "ADDITIONAL_HEADERS;DEPENDS;LINK_COMPONENTS;LINK_LIBS"
${ARGN})
list(APPEND LLVM_COMMON_DEPENDS ${ARG_DEPENDS})
- llvm_process_sources(ALL_FILES ${ARG_UNPARSED_ARGUMENTS})
+ if(ARG_ADDITIONAL_HEADERS)
+ # Pass through ADDITIONAL_HEADERS.
+ set(ARG_ADDITIONAL_HEADERS ADDITIONAL_HEADERS ${ARG_ADDITIONAL_HEADERS})
+ endif()
+ llvm_process_sources(ALL_FILES ${ARG_UNPARSED_ARGUMENTS} ${ARG_ADDITIONAL_HEADERS})
if(ARG_MODULE)
if(ARG_SHARED OR ARG_STATIC)
More information about the llvm-commits
mailing list