[llvm] r351374 - Fix most of LLVM's tests with LLVM_ENABLE_PIC=OFF

Nico Weber via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 16 12:44:37 PST 2019


Author: nico
Date: Wed Jan 16 12:44:36 2019
New Revision: 351374

URL: http://llvm.org/viewvc/llvm-project?rev=351374&view=rev
Log:
Fix most of LLVM's tests with LLVM_ENABLE_PIC=OFF

Lots of tests rely on llvm-lto being present, but LLVM_ENABLE_PIC=OFF currently
disables building that executable.

There's no reason for not building llvm-lto with LLVM_ENABLE_PIC=OFF so just
build it. r191042 moved it into a "if (!WIN)" block at the time, and then
211852 made that "if(NOT CYGWIN AND LLVM_ENABLE_PIC)" -- but that's only needed
for LTO (the ld64 plugin), not for the llvm-lto binary.

Differential Revision: https://reviews.llvm.org/D56801

Modified:
    llvm/trunk/tools/CMakeLists.txt

Modified: llvm/trunk/tools/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/CMakeLists.txt?rev=351374&r1=351373&r2=351374&view=diff
==============================================================================
--- llvm/trunk/tools/CMakeLists.txt (original)
+++ llvm/trunk/tools/CMakeLists.txt Wed Jan 16 12:44:36 2019
@@ -27,7 +27,6 @@ endif()
 
 if(CYGWIN OR NOT LLVM_ENABLE_PIC)
   set(LLVM_TOOL_LTO_BUILD Off)
-  set(LLVM_TOOL_LLVM_LTO_BUILD Off)
 endif()
 
 # Add LTO, llvm-ar, llvm-config, and llvm-profdata before clang, ExternalProject




More information about the llvm-commits mailing list