[llvm] Escape LLVM_TARGETS_TO_BUILD while checking against LLVM_ALL_TARGETS and LLVM_EXPERIMENTAL_TARGETS_TO_BUILD (PR #70885)
Shoaib Meenai via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 31 23:32:27 PDT 2023
https://github.com/smeenai updated https://github.com/llvm/llvm-project/pull/70885
>From 31a06749b6f4312ac91b4bfea99abf010a052d27 Mon Sep 17 00:00:00 2001
From: Natschz <>
Date: Wed, 1 Nov 2023 02:56:53 +0100
Subject: [PATCH 1/2] Escape LLVM_TARGETS_TO_BUILD while checking against
LLVM_ALL_TARGETS and LLVM_EXPERIMENTAL_TARGETS_TO_BUILD
---
llvm/CMakeLists.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt
index 82d4beea91e346e..f96704beddc7721 100644
--- a/llvm/CMakeLists.txt
+++ b/llvm/CMakeLists.txt
@@ -942,7 +942,7 @@ foreach(t ${LLVM_TARGETS_TO_BUILD})
# LLVM_EXPERIMENTAL_TARGETS_TO_BUILD, not LLVM_TARGETS_TO_BUILD.
# We allow experimental targets that are not in LLVM_ALL_EXPERIMENTAL_TARGETS,
# as long as they are passed via LLVM_EXPERIMENTAL_TARGETS_TO_BUILD.
- if ( NOT ${t} IN_LIST LLVM_ALL_TARGETS AND NOT ${t} IN_LIST LLVM_EXPERIMENTAL_TARGETS_TO_BUILD )
+ if ( NOT "${t}" IN_LIST LLVM_ALL_TARGETS AND NOT "${t}" IN_LIST LLVM_EXPERIMENTAL_TARGETS_TO_BUILD )
if( ${t} IN_LIST LLVM_ALL_EXPERIMENTAL_TARGETS )
message(FATAL_ERROR "The target `${t}' is experimental and must be passed "
"via LLVM_EXPERIMENTAL_TARGETS_TO_BUILD.")
>From 4b055a6816b160bedaec81bfe3c664a13d15a2dc Mon Sep 17 00:00:00 2001
From: Shoaib Meenai <shoaib.meenai at gmail.com>
Date: Tue, 31 Oct 2023 23:32:21 -0700
Subject: [PATCH 2/2] Update llvm/CMakeLists.txt
---
llvm/CMakeLists.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/llvm/CMakeLists.txt b/llvm/CMakeLists.txt
index f96704beddc7721..292efa3316df748 100644
--- a/llvm/CMakeLists.txt
+++ b/llvm/CMakeLists.txt
@@ -943,7 +943,7 @@ foreach(t ${LLVM_TARGETS_TO_BUILD})
# We allow experimental targets that are not in LLVM_ALL_EXPERIMENTAL_TARGETS,
# as long as they are passed via LLVM_EXPERIMENTAL_TARGETS_TO_BUILD.
if ( NOT "${t}" IN_LIST LLVM_ALL_TARGETS AND NOT "${t}" IN_LIST LLVM_EXPERIMENTAL_TARGETS_TO_BUILD )
- if( ${t} IN_LIST LLVM_ALL_EXPERIMENTAL_TARGETS )
+ if( "${t}" IN_LIST LLVM_ALL_EXPERIMENTAL_TARGETS )
message(FATAL_ERROR "The target `${t}' is experimental and must be passed "
"via LLVM_EXPERIMENTAL_TARGETS_TO_BUILD.")
else()
More information about the llvm-commits
mailing list