[cfe-commits] r166679 - in /cfe/trunk/lib: Basic/Targets.cpp CodeGen/TargetInfo.cpp

[Joey Gouly]

Hi David, 

Those two tests need a x86-64 target to run, some ARM builders only target
ARM, so these fail.

The attached patch fixes it, please review/build/commit!

Thanks,
Joey

-----Original Message-----
From: cfe-commits-bounces at cs.uiuc.edu
[mailto:cfe-commits-bounces at cs.uiuc.edu] On Behalf Of David Tweed
Sent: 26 October 2012 09:40
To: 'Dmitri Gribenko'
Cc: cfe-commits at cs.uiuc.edu
Subject: Re: [cfe-commits] r166679 - in /cfe/trunk/lib: Basic/Targets.cpp
CodeGen/TargetInfo.cpp

Hi David,

| Seems like this was the last missing bit for clang tests to pass on
| Debian armhf!

Ah, so are you getting a clean pass on the clang tests on an armhf platform?
Wei-Ren Chen is also getting a clean pass. I'm not :-( . On my ARM
Pandaboard test machine I'm seeing two intermittent failures that have
started to crop up in the last week or so:

CodeGen/2010-06-17-asmcrash.c
Sema/ms-inline-asm.c

(I've never seen these on x86.) The apparent cause is

error: unable to create target: 'No available targets are compatible with
this triple, see -version for the available targets.'

What's really annoying is that I start to try to investigate and without
changing anything they suddenly start to pass (and there are other tests
that reliably pass with the same target triples as specified here). Then
after a while they start to fail again. I'm starting to wonder if there's
some uninitialized variable somewhere that sometimes takes an OK value and
sometimes doesn't.

Has anyone else been seeing anything like this happening, particularly with
those tests?

Cheers,
Dave




_______________________________________________
cfe-commits mailing list
cfe-commits at cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add_requires_x86_64.diff
Type: application/octet-stream
Size: 725 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20121026/4ec2e4ce/attachment.obj>