[llvm] [LLVM] Fix runtimes builds not triggering without LLVM_ENABLE_RUNTIMES (PR #94225)
Joseph Huber via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 3 07:09:33 PDT 2024
https://github.com/jhuber6 created https://github.com/llvm/llvm-project/pull/94225
Summary:
The runtimes builds create separate projects. Normally these use
`LLVM_EANBLE_RUNTIMES`. However, we can also use `LLVM_RUNTIME_TARGETS`
and `RUNTIMES_<target>_LLVM_ENABLE_RUNTIMES` to enable them. Currently,
if `LLVM_RUNTIME_TARGETS` isn't specified this is completely ignored.
This patch fixes the behavior so we can only specify the cross-compiling
versions.
>From b8717a9b9bd22c7ca82f3f1efae23494052684de Mon Sep 17 00:00:00 2001
From: Joseph Huber <huberjn at outlook.com>
Date: Mon, 3 Jun 2024 09:06:22 -0500
Subject: [PATCH] [LLVM] Fix runtimes builds not triggering without
LLVM_ENABLE_RUNTIMES
Summary:
The runtimes builds create separate projects. Normally these use
`LLVM_EANBLE_RUNTIMES`. However, we can also use `LLVM_RUNTIME_TARGETS`
and `RUNTIMES_<target>_LLVM_ENABLE_RUNTIMES` to enable them. Currently,
if `LLVM_RUNTIME_TARGETS` isn't specified this is completely ignored.
This patch fixes the behavior so we can only specify the cross-compiling
versions.
---
llvm/runtimes/CMakeLists.txt | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/llvm/runtimes/CMakeLists.txt b/llvm/runtimes/CMakeLists.txt
index 7fd4e61cc46e6..e60c9dd6041b4 100644
--- a/llvm/runtimes/CMakeLists.txt
+++ b/llvm/runtimes/CMakeLists.txt
@@ -430,7 +430,17 @@ function(runtime_register_target name)
endforeach()
endfunction()
+# Check if we have any runtimes to build.
if(runtimes)
+ set(build_runtimes TRUE)
+endif()
+foreach(name ${LLVM_RUNTIME_TARGETS})
+ if(RUNTIMES_${name}_LLVM_ENABLE_RUNTIMES)
+ set(build_runtimes TRUE)
+ endif()
+endforeach()
+
+if(build_runtimes)
# Create a runtimes target that uses this file as its top-level CMake file.
# The runtimes target is a configuration of all the runtime libraries
# together in a single CMake invocation.
More information about the llvm-commits
mailing list