[llvm] 6facfe1 - Revert "[compiler-rt][CMake] Properly set COMPILER_RT_HAS_LLD"

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Wed May 31 11:27:37 PDT 2023


Author: Arthur Eubanks
Date: 2023-05-31T11:27:24-07:00
New Revision: 6facfe10258b475713440953ef46fca6734c0e5e

URL: https://github.com/llvm/llvm-project/commit/6facfe10258b475713440953ef46fca6734c0e5e
DIFF: https://github.com/llvm/llvm-project/commit/6facfe10258b475713440953ef46fca6734c0e5e.diff

LOG: Revert "[compiler-rt][CMake] Properly set COMPILER_RT_HAS_LLD"

This reverts commit 395a614d2cb69a431bd11e266021d91503c1d709.

Causes some bots to break, e.g. https://ci.chromium.org/ui/p/fuchsia/builders/toolchain.ci/clang-linux-x64/b8779560688633165361/overview

Added: 
    

Modified: 
    compiler-rt/CMakeLists.txt
    llvm/runtimes/CMakeLists.txt

Removed: 
    


################################################################################
diff  --git a/compiler-rt/CMakeLists.txt b/compiler-rt/CMakeLists.txt
index 80954df33c216..6489aa17c2292 100644
--- a/compiler-rt/CMakeLists.txt
+++ b/compiler-rt/CMakeLists.txt
@@ -738,8 +738,14 @@ if(COMPILER_RT_USE_LIBCXX)
   endif()
 endif()
 
-if(COMPILER_RT_HAS_TRUNK_LLD AND LLVM_USE_LINKER STREQUAL "lld")
+set(COMPILER_RT_LLD_PATH ${LLVM_MAIN_SRC_DIR}/tools/lld)
+if(EXISTS ${COMPILER_RT_LLD_PATH}/ AND LLVM_TOOL_LLD_BUILD)
   set(COMPILER_RT_HAS_LLD TRUE)
+else()
+  set(COMPILER_RT_LLD_PATH ${LLVM_MAIN_SRC_DIR}/../lld)
+  if(EXISTS ${COMPILER_RT_LLD_PATH}/ AND LLVM_TOOL_LLD_BUILD)
+    set(COMPILER_RT_HAS_LLD TRUE)
+  endif()
 endif()
 
 if(ANDROID)

diff  --git a/llvm/runtimes/CMakeLists.txt b/llvm/runtimes/CMakeLists.txt
index 4f2c1a1fe32cf..75256c1326be9 100644
--- a/llvm/runtimes/CMakeLists.txt
+++ b/llvm/runtimes/CMakeLists.txt
@@ -226,10 +226,6 @@ function(runtime_default_target)
 
   set_enable_per_target_runtime_dir()
 
-  if ("lld" IN_LIST LLVM_ENABLE_PROJECTS)
-    list(APPEND extra_args -DCOMPILER_RT_HAS_TRUNK_LLD=TRUE)
-  endif()
-
   llvm_ExternalProject_Add(runtimes
                            ${CMAKE_CURRENT_SOURCE_DIR}/../../runtimes
                            DEPENDS ${ARG_DEPENDS}
@@ -246,7 +242,6 @@ function(runtime_default_target)
                                       ${COMMON_CMAKE_ARGS}
                                       ${RUNTIMES_CMAKE_ARGS}
                                       ${ARG_CMAKE_ARGS}
-                                      ${extra_args}
                            PASSTHROUGH_PREFIXES LLVM_ENABLE_RUNTIMES
                                                 LLVM_USE_LINKER
                                                 ${ARG_PREFIXES}
@@ -339,10 +334,6 @@ function(runtime_register_target name)
 
   set_enable_per_target_runtime_dir()
 
-  if ("lld" IN_LIST LLVM_ENABLE_PROJECTS)
-    list(APPEND ${name}_extra_args -DCOMPILER_RT_HAS_TRUNK_LLD=TRUE)
-  endif()
-
   llvm_ExternalProject_Add(runtimes-${name}
                            ${CMAKE_CURRENT_SOURCE_DIR}/../../runtimes
                            DEPENDS ${ARG_DEPENDS}


        


More information about the llvm-commits mailing list