[llvm] ca18b4a - [test] [MCJIT] Don't overwrite config.unsupported to False
Martin Storsjö via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 28 13:38:32 PDT 2023
Author: Martin Storsjö
Date: 2023-07-28T23:37:09+03:00
New Revision: ca18b4a0c1324ae679490add3ad3bbd2077b804f
URL: https://github.com/llvm/llvm-project/commit/ca18b4a0c1324ae679490add3ad3bbd2077b804f
DIFF: https://github.com/llvm/llvm-project/commit/ca18b4a0c1324ae679490add3ad3bbd2077b804f.diff
LOG: [test] [MCJIT] Don't overwrite config.unsupported to False
An outer directory may have set config.unsupported to True for
a different reason, separate from architecture concerns - we
shouldn't force it back to False just because one criterion is
fulfilled.
Differential Revision: https://reviews.llvm.org/D149995
Added:
Modified:
llvm/test/ExecutionEngine/MCJIT/lit.local.cfg
Removed:
################################################################################
diff --git a/llvm/test/ExecutionEngine/MCJIT/lit.local.cfg b/llvm/test/ExecutionEngine/MCJIT/lit.local.cfg
index b6874dd86eadb0..46a3ff9d32deab 100644
--- a/llvm/test/ExecutionEngine/MCJIT/lit.local.cfg
+++ b/llvm/test/ExecutionEngine/MCJIT/lit.local.cfg
@@ -1,6 +1,6 @@
root = config.root
targets = root.targets
-if (
+if not (
("X86" in targets)
| ("AArch64" in targets)
| ("ARM" in targets)
@@ -8,8 +8,6 @@ if (
| ("PowerPC" in targets)
| ("SystemZ" in targets)
):
- config.unsupported = False
-else:
config.unsupported = True
# FIXME: autoconf and cmake produce
diff erent arch names. We should normalize
@@ -21,6 +19,7 @@ if root.host_arch not in [
"AMD64",
"AArch64",
"ARM",
+ "armv7",
"Mips",
"PowerPC",
"ppc64",
@@ -29,9 +28,6 @@ if root.host_arch not in [
]:
config.unsupported = True
-if "armv7" in root.host_arch:
- config.unsupported = False
-
if "i386-apple-darwin" in root.target_triple:
config.unsupported = True
More information about the llvm-commits
mailing list