[llvm] 716e27b - [llvm] Disable LLVM_ENABLE_PER_TARGET_RUNTIME_DIR for Arm Linux
David Spickett via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 21 04:51:57 PDT 2022
Author: David Spickett
Date: 2022-06-21T11:51:43Z
New Revision: 716e27bc9ad4699bbc82318834e938bcc9682cf8
URL: https://github.com/llvm/llvm-project/commit/716e27bc9ad4699bbc82318834e938bcc9682cf8
DIFF: https://github.com/llvm/llvm-project/commit/716e27bc9ad4699bbc82318834e938bcc9682cf8.diff
LOG: [llvm] Disable LLVM_ENABLE_PER_TARGET_RUNTIME_DIR for Arm Linux
I did this before but the other change got reverted and relanded
recently.
arm vs armhf isn't handled correctly so this setting doesn't work
for 32 bit Arm Linux. Causing failures like:
https://lab.llvm.org/buildbot/#/builders/178/builds/2293
Added:
Modified:
llvm/CMakeLists.txt
Removed:
################################################################################
diff --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt
index 14a215257b39..79b355750bf2 100644
--- a/llvm/CMakeLists.txt
+++ b/llvm/CMakeLists.txt
@@ -746,7 +746,8 @@ endif()
set(LLVM_TARGET_TRIPLE_ENV CACHE STRING "The name of environment variable to override default target. Disabled by blank.")
mark_as_advanced(LLVM_TARGET_TRIPLE_ENV)
-if(CMAKE_SYSTEM_NAME MATCHES "Linux")
+# Per target dir not yet supported on Arm 32 bit due to arm vs armhf handling
+if(CMAKE_SYSTEM_NAME MATCHES "Linux" AND NOT CMAKE_SYSTEM_PROCESSOR MATCHES "^arm")
set(LLVM_ENABLE_PER_TARGET_RUNTIME_DIR_default ON)
else()
set(LLVM_ENABLE_PER_TARGET_RUNTIME_DIR_default OFF)
More information about the llvm-commits
mailing list