[llvm-commits] [test-suite] r91925 - /test-suite/trunk/External/SPEC/CINT95/Makefile

[Dale Johannesen]

On Dec 22, 2009, at 2:23 PMPST, Bob Wilson wrote:

>
> On Dec 22, 2009, at 2:04 PM, Nick Lewycky wrote:
>
>> Daniel Dunbar wrote:
>>> Author: ddunbar
>>> Date: Tue Dec 22 16:02:47 2009
>>> New Revision: 91925
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=91925&view=rev
>>> Log:
>>> 147.vortex doesn't work on x86_64, skip it.
>>>
>> What? We can't handle SPEC and that's okay now?
>>
>> Could you explain why this doesn't work?
>
> The source code is not 64-bit clean.  It doesn't work with gcc,  
> either.

I'm pretty sure I got all of SPEC95 to run in a 64-bit environment,  
although not recently, but there is no clean way to do it from the top- 
level makefile as there is in later SPECs.  Here, try
-D__RISC_64__ (yes, it's a namespace violation).  If it doesn't work  
with that there's probably something really wrong.

Vortex has some violations of strict aliasing, like *((type1*)ptr-to- 
object-of-type2) .  gcc routinely generates unexpected code for this  
(permitted by standards, it's undefined behavior), which is one reason  
vortex doesn't work there.  It doesn't work with gcc in many 32-bit  
environments either.

> Daniel, I ran SPEC last week for x86_64 and I thought I remembered  
> seeing more than just vortex failing.  Are there more of them to  
> skip or am I just misremembering?
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits