r199280 - lit.cfg: Don't expand %ms_abi_triple to non-X86 triples
Hans Wennborg
hans at hanshq.net
Tue Jan 14 17:08:42 PST 2014
Author: hans
Date: Tue Jan 14 19:08:42 2014
New Revision: 199280
URL: http://llvm.org/viewvc/llvm-project?rev=199280&view=rev
Log:
lit.cfg: Don't expand %ms_abi_triple to non-X86 triples
This is a follow-up to r199260. On ARM hosts, we were attempting to run
tests with triples such as armv7l-unknown-win32. This expands that fix to
cover all non-x86 targets since we only support MS ABI on x86.
Modified:
cfe/trunk/test/lit.cfg
Modified: cfe/trunk/test/lit.cfg
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/lit.cfg?rev=199280&r1=199279&r2=199280&view=diff
==============================================================================
--- cfe/trunk/test/lit.cfg (original)
+++ cfe/trunk/test/lit.cfg Tue Jan 14 19:08:42 2014
@@ -241,17 +241,17 @@ def makeMSABITriple(triple):
m = re.match(r'(\w+)-(\w+)-(\w+)', triple)
if not m:
lit_config.fatal("Could not turn '%s' into MS ABI triple" % triple)
- isa = m.group(1)
- vendor = m.group(2)
- os = m.group(3)
- if os.lower() == 'win32':
+ isa = m.group(1).lower()
+ vendor = m.group(2).lower()
+ os = m.group(3).lower()
+ if os == 'win32':
# If the OS is win32, we're done.
return triple
- if "arm" in isa.lower():
- # FIXME: Fix failures with arm*-*-win32.
- isa = "i686"
- # Otherwise, replace the OS part with Win32.
- return isa + '-' + vendor + '-win32'
+ if isa.startswith('x86') or isa == 'amd64' or re.match(r'i\d86', isa):
+ # For x86 ISAs, adjust the OS.
+ return isa + '-' + vendor + '-win32'
+ # -win32 is not supported for non-x86 targets; use a default.
+ return 'i686-pc-win32'
config.substitutions.append( ('%clang_cc1', '%s -cc1 -internal-isystem %s'
% (config.clang,
More information about the cfe-commits
mailing list