Skip to content

Commit ca18b4a

Browse files
committed
[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
1 parent 2a859b2 commit ca18b4a

File tree

1 file changed

+2
-6
lines changed

1 file changed

+2
-6
lines changed

llvm/test/ExecutionEngine/MCJIT/lit.local.cfg

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,13 @@
11
root = config.root
22
targets = root.targets
3-
if (
3+
if not (
44
("X86" in targets)
55
| ("AArch64" in targets)
66
| ("ARM" in targets)
77
| ("Mips" in targets)
88
| ("PowerPC" in targets)
99
| ("SystemZ" in targets)
1010
):
11-
config.unsupported = False
12-
else:
1311
config.unsupported = True
1412

1513
# FIXME: autoconf and cmake produce different arch names. We should normalize
@@ -21,6 +19,7 @@ if root.host_arch not in [
2119
"AMD64",
2220
"AArch64",
2321
"ARM",
22+
"armv7",
2423
"Mips",
2524
"PowerPC",
2625
"ppc64",
@@ -29,9 +28,6 @@ if root.host_arch not in [
2928
]:
3029
config.unsupported = True
3130

32-
if "armv7" in root.host_arch:
33-
config.unsupported = False
34-
3531
if "i386-apple-darwin" in root.target_triple:
3632
config.unsupported = True
3733

0 commit comments

Comments
 (0)