[llvm] r271530 - [cmake] Fix builds with LLVM_ENABLE_PIC=0

Pavel Labath via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 2 09:29:07 PDT 2016


Author: labath
Date: Thu Jun  2 11:29:07 2016
New Revision: 271530

URL: http://llvm.org/viewvc/llvm-project?rev=271530&view=rev
Log:
[cmake] Fix builds with LLVM_ENABLE_PIC=0

Summary:
When this flag is specified, the target llvm-lto is not built, but is still
used as a dependency of the test targets. cmake 2.8 silently ignored this
situation, but with cmake_minimum_required(3.4) it becomes an error. Fix this
by avoiding the inclusion of the target as a dependency.

Reviewers: beanz

Subscribers: llvm-commits

Differential Revision: http://reviews.llvm.org/D20882

Modified:
    llvm/trunk/test/CMakeLists.txt

Modified: llvm/trunk/test/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CMakeLists.txt?rev=271530&r1=271529&r2=271530&view=diff
==============================================================================
--- llvm/trunk/test/CMakeLists.txt (original)
+++ llvm/trunk/test/CMakeLists.txt Thu Jun  2 11:29:07 2016
@@ -43,7 +43,6 @@ set(LLVM_TEST_DEPENDS
           llvm-extract
           llvm-lib
           llvm-link
-          llvm-lto
           llvm-mc
           llvm-mcmarkup
           llvm-nm
@@ -67,6 +66,10 @@ set(LLVM_TEST_DEPENDS
           yaml2obj
         )
 
+if(TARGET llvm-lto)
+  set(LLVM_TEST_DEPENDS ${LLVM_TEST_DEPENDS} llvm-lto)
+endif()
+
 # If Intel JIT events are supported, depend on a tool that tests the listener.
 if( LLVM_USE_INTEL_JITEVENTS )
   set(LLVM_TEST_DEPENDS ${LLVM_TEST_DEPENDS} llvm-jitlistener)
@@ -80,7 +83,7 @@ if(TARGET llvm-go)
   set(LLVM_TEST_DEPENDS ${LLVM_TEST_DEPENDS} llvm-go)
 endif()
 
-if(APPLE)
+if(TARGET LTO)
   set(LLVM_TEST_DEPENDS ${LLVM_TEST_DEPENDS} LTO)
 endif()
 




More information about the llvm-commits mailing list