diff options
| author | Don Hinton <hintonda@gmail.com> | 2017-12-18 19:15:15 +0000 |
|---|---|---|
| committer | Don Hinton <hintonda@gmail.com> | 2017-12-18 19:15:15 +0000 |
| commit | d33f76cc205bea1c577bed741b6b857f38a14eb6 (patch) | |
| tree | 236540d1a63e576f6d3174ed49cf72dbe21a8bb2 /CMakeLists.txt | |
| parent | a6d921ec5bff51da7c7f95340777b4ce75d7f840 (diff) | |
[cmake] Update experimental target error message
Summary:
Update this error message indicate this test only ensures experimental
targets were passed via LLVM_EXPERIMENTAL_TARGETS_TO_BUILD.
Originally, this test validated all targets, but in r184923, it was moved
after the LLVMBUILDTOOL test, which also validates all targets, making
that part of the test redundant.
Differential Revision: https://reviews.llvm.org/D41273
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@321012 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'CMakeLists.txt')
| -rw-r--r-- | CMakeLists.txt | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt index 84dca6bf598..c100adef258 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -678,9 +678,13 @@ foreach(t ${LLVM_TARGETS_TO_BUILD}) list(FIND LLVM_ALL_TARGETS ${t} idx) list(FIND LLVM_EXPERIMENTAL_TARGETS_TO_BUILD ${t} idy) + # At this point, LLVMBUILDTOOL already checked all the targets passed in + # LLVM_TARGETS_TO_BUILD and LLVM_EXPERIMENTAL_TARGETS_TO_BUILD, so + # this test just makes sure that any experimental targets were passed via + # LLVM_EXPERIMENTAL_TARGETS_TO_BUILD, not LLVM_TARGETS_TO_BUILD. if( idx LESS 0 AND idy LESS 0 ) - message(FATAL_ERROR "The target `${t}' does not exist. - It should be one of\n${LLVM_ALL_TARGETS}") + message(FATAL_ERROR "The target `${t}' is experimental and must be passed " + "via LLVM_EXPERIMENTAL_TARGETS_TO_BUILD.") else() set(LLVM_ENUM_TARGETS "${LLVM_ENUM_TARGETS}LLVM_TARGET(${t})\n") endif() |