[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