On Thu, Nov 1, 2012 at 2:41 PM, H.J. Lu <hjl.to...@gmail.com> wrote:
> On Thu, Nov 1, 2012 at 6:47 AM, Jack Howarth <howa...@bromo.med.uc.edu> wrote:
>> On Fri, Aug 24, 2012 at 04:13:20PM +0200, Rainer Orth wrote:
>>> Jack Howarth <howa...@bromo.med.uc.edu> writes:
>>>
>>> > Currently the new testcase for gcc.target/i386/pr53249.c is failing
>>> > on darwin due to the absence of -mx32 support on that target. The
>>> > following
>>> > patch skips this testcase on darwin. Tested on x86_64-apple-darwin12...
>>>
>>> This also fails on Solaris/x86 (cf. PR testsuite/53365) and
>>> i686-unknown-linux-gnu. I'd strongly prefer if HJ could devise a real
>>> fix instead of just skipping the test on an explicit list of systems.
>>>
>>> Rainer
>>
>> Rainer,
>> What about using...
>>
>> Index: gcc/testsuite/gcc.target/i386/pr53249.c
>> ===================================================================
>> --- gcc/testsuite/gcc.target/i386/pr53249.c (revision 193061)
>> +++ gcc/testsuite/gcc.target/i386/pr53249.c (working copy)
>> @@ -1,4 +1,4 @@
>> -/* { dg-do compile { target { ! { ia32 } } } } */
>> +/* { dg-do compile { target { ! { ia32 || llp64 } } } } */
>> /* { dg-options "-O2 -mx32 -ftls-model=initial-exec -maddress-mode=short" }
>> */
>>
>> struct gomp_task
>>
>> This converts the failure at -m64 into an unsupported testcase on
>> x86_64-apple-darwin12.
>
> This will disable test on Linux/x86-64.
>
> --
> H.J.
We can add
# Return 1 if -mx32 can compile, 0 otherwise.
proc check_effective_target_maybe_x32 { } {
return [check_no_compiler_messages maybe_x32 object {
void foo (void) {}
} {-mx32}]
}
and check it before compiling with -mx32.
--
H.J.
