Patchwork [testsuite] Patch for gcc.dg/pr49994-3.c on HP-UX

login
register
mail settings
Submitter Steve Ellcey
Date Oct. 20, 2011, 4:14 p.m.
Message ID <201110201614.p9KGEic20374@adlwrk05.cce.hp.com>
Download mbox | patch
Permalink /patch/120834/
State New
Headers show

Comments

Steve Ellcey - Oct. 20, 2011, 4:14 p.m.
I am going to check this change in as obvious later today if there are
no objections,  the test gives warnings on HP-UX because it calls
__builtin_return_address with arguments of 0 through 5 but the value 0
is the only valid argument to __builtin_return_address on HP-UX.

Tested on IA64 and PA HP-UX.

Steve Ellcey
sje@cup.hp.com



2011-10-20  Steve Ellcey  <sje@cup.hp.com>

	PR testsuite/50722
	* gcc.dg/pr49994-3.c: Skip on HP-UX.
Rainer Orth - Oct. 20, 2011, 4:23 p.m.
Steve Ellcey <sje@cup.hp.com> writes:

> Index: gcc.dg/pr49994-3.c
> ===================================================================
> --- gcc.dg/pr49994-3.c	(revision 180233)
> +++ gcc.dg/pr49994-3.c	(working copy)
> @@ -2,6 +2,7 @@
>  /* { dg-options "-O2 -fsched2-use-superblocks -g" } */
>  /* { dg-options "-O2 -fsched2-use-superblocks -g -mbackchain" { target s390*-*-* } } */
>  /* { dg-require-effective-target scheduling } */
> +/* { dg-skip-if "" { *-*-hpux* } { "*" } { "" } } */

Would you please include either an explanation or a PR reference in the
dg-skip-if?  Having to search the archives for an explanation is tedious.

Btw., you should be able to omit both the "*" and "".

Thanks.
        Rainer
Steve Ellcey - Oct. 20, 2011, 9:27 p.m.
On Thu, 2011-10-20 at 18:23 +0200, Rainer Orth wrote:
> Steve Ellcey <sje@cup.hp.com> writes:
> 
> > Index: gcc.dg/pr49994-3.c
> > ===================================================================
> > --- gcc.dg/pr49994-3.c	(revision 180233)
> > +++ gcc.dg/pr49994-3.c	(working copy)
> > @@ -2,6 +2,7 @@
> >  /* { dg-options "-O2 -fsched2-use-superblocks -g" } */
> >  /* { dg-options "-O2 -fsched2-use-superblocks -g -mbackchain" { target s390*-*-* } } */
> >  /* { dg-require-effective-target scheduling } */
> > +/* { dg-skip-if "" { *-*-hpux* } { "*" } { "" } } */
> 
> Would you please include either an explanation or a PR reference in the
> dg-skip-if?  Having to search the archives for an explanation is tedious.
> 
> Btw., you should be able to omit both the "*" and "".
> 
> Thanks.
>         Rainer

I put "PR testsuite/50722" in the comment section and removed the "*"
and "" after verifying that it works and then checked in the change.

Steve Ellcey
sje@cup.hp.com

Patch

Index: gcc.dg/pr49994-3.c
===================================================================
--- gcc.dg/pr49994-3.c	(revision 180233)
+++ gcc.dg/pr49994-3.c	(working copy)
@@ -2,6 +2,7 @@ 
 /* { dg-options "-O2 -fsched2-use-superblocks -g" } */
 /* { dg-options "-O2 -fsched2-use-superblocks -g -mbackchain" { target s390*-*-* } } */
 /* { dg-require-effective-target scheduling } */
+/* { dg-skip-if "" { *-*-hpux* } { "*" } { "" } } */
 
 void *
 foo (int offset)