This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [testsuite] Fix gcc.dg/torture/pr47917.c on IRIX 6.5
- From: Steve Ellcey <sje at cup dot hp dot com>
- To: ro at cebitec dot uni-bielefeld dot de
- Cc: richard dot guenther at gmail dot com, gcc-patches at gcc dot gnu dot org
- Date: Tue, 29 Mar 2011 10:28:38 -0700 (PDT)
- Subject: Re: [testsuite] Fix gcc.dg/torture/pr47917.c on IRIX 6.5
On Fri, Mar 25, 2011 at 4:27 PM, Rainer Orth wrote:
> The new gcc.dg/torture/pr47917.c test currently fails the execution test
> on IRIX 6.5. To get a C99-conformant snprintf, one needs to include
> <stdio.h> with _XOPEN_SOURCE defined as 500. The following patch does
> this.
Rainer,
This patch re-broke pr47917.c on HP-UX because it doesn't see the
snprintf prototype in stdio.h when using -std=c99. I do see it if I use
-std=gnu99 instead and that is what I probably should have used in the
first place. I was wondering if using -std=gnu99 would also work for
mips-sgi-irix6.5 in place of "-std=c99 -D_XOPEN_SOURCE=500".
If so I would propose this patch to just use -std=gnu99 for all
platforms:
$ svn diff gcc.dg/torture/pr47917.c
Index: gcc.dg/torture/pr47917.c
===================================================================
--- gcc.dg/torture/pr47917.c (revision 171649)
+++ gcc.dg/torture/pr47917.c (working copy)
@@ -1,6 +1,5 @@
/* { dg-do run } */
-/* { dg-options "-std=c99" } */
-/* { dg-options "-std=c99 -D_XOPEN_SOURCE=500" { target mips-sgi-irix6.5 } } */
+/* { dg-options "-std=gnu99" } */
/* { dg-xfail-if "no C99 snprintf function" { *-*-hpux10* } } */
/* { dg-xfail-run-if "non-conforming C99 snprintf" { *-*-hpux11.[012]* } } */
Steve Ellcey
sje@cup.hp.com