[cmake] Change lli-child-target to use add_llvm_utility instead of add_llvm_executable.

We are currently using add_llvm_utility for executable targets that:

1. Are built by default.
2. Used for testing.
3. Are not installed by default.

Originally, lli-child-target used add_llvm_tool instead of add_llvm_executable
directly. This was changed so that lli-child-target would not be installed. This
was good since this is only used for testing and should never be installed for
users. This also had the unfortunate side effect that one can never turn off the
building of lli-child-target by default, a regression for projects that by
default do not want to compile any LLVM tools beyond tablegen/llvm-config.

This patch changes lli-child-target to use add_llvm_utility. This makes sense
since:

1. lli-child-target matches the semantics of executables created with
add_llvm_utility.
2. We fix the regression since now one can use the flag LLVM_BUILD_UTILS to
eliminate default compilation.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@275008 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Michael Gottesman 2016-07-10 02:43:50 +00:00
parent 76b8ab5181
commit ff802309ff
2 changed files with 4 additions and 3 deletions

View File

@ -1,4 +1,6 @@
add_subdirectory(ChildTarget)
if ( LLVM_INCLUDE_UTILS )
add_subdirectory(ChildTarget)
endif()
set(LLVM_LINK_COMPONENTS
CodeGen

View File

@ -4,8 +4,7 @@ set(LLVM_LINK_COMPONENTS
Support
)
add_llvm_executable(lli-child-target
add_llvm_utility(lli-child-target
ChildTarget.cpp
)
set_target_properties(lli-child-target PROPERTIES FOLDER "Misc")