Re: [committed] Fix failure of gcc.dg/tree-prof/pr59003.c on hppa*-*-hpux*
John David Anglin dave.ang...@bell.net writes: Index: gcc.dg/tree-prof/pr59003.c === --- gcc.dg/tree-prof/pr59003.c(revision 208769) +++ gcc.dg/tree-prof/pr59003.c(working copy) @@ -1,6 +1,7 @@ /* PR target/59003 */ /* { dg-options -O2 } */ /* { dg-options -O2 -mtune=amdfam10 { target i?86-*-* x86_64-*-* } } */ +/* { dg-options -O2 -fno-common { target hppa*-*-hpux* } } */ Better handled with dg-additional-options rather than duplicating dg-options. No need to change though, since the x86 entry already used the less preferred style. Rainer -- - Rainer Orth, Center for Biotechnology, Bielefeld University
Re: [committed] Fix failure of gcc.dg/tree-prof/pr59003.c on hppa*-*-hpux*
On 3/24/2014 2:48 AM, Rainer Orth wrote: John David Anglin dave.ang...@bell.net writes: Index: gcc.dg/tree-prof/pr59003.c === --- gcc.dg/tree-prof/pr59003.c (revision 208769) +++ gcc.dg/tree-prof/pr59003.c (working copy) @@ -1,6 +1,7 @@ /* PR target/59003 */ /* { dg-options -O2 } */ /* { dg-options -O2 -mtune=amdfam10 { target i?86-*-* x86_64-*-* } } */ +/* { dg-options -O2 -fno-common { target hppa*-*-hpux* } } */ Better handled with dg-additional-options rather than duplicating dg-options. No need to change though, since the x86 entry already used the less preferred style. I already tried it. dg-additional-options isn't supported in tree-prof. Dave -- John David Anglindave.ang...@bell.net
Re: [committed] Fix failure of gcc.dg/tree-prof/pr59003.c on hppa*-*-hpux*
John David Anglin dave.ang...@bell.net writes: On 3/24/2014 2:48 AM, Rainer Orth wrote: John David Anglin dave.ang...@bell.net writes: Index: gcc.dg/tree-prof/pr59003.c === --- gcc.dg/tree-prof/pr59003.c (revision 208769) +++ gcc.dg/tree-prof/pr59003.c (working copy) @@ -1,6 +1,7 @@ /* PR target/59003 */ /* { dg-options -O2 } */ /* { dg-options -O2 -mtune=amdfam10 { target i?86-*-* x86_64-*-* } } */ +/* { dg-options -O2 -fno-common { target hppa*-*-hpux* } } */ Better handled with dg-additional-options rather than duplicating dg-options. No need to change though, since the x86 entry already used the less preferred style. I already tried it. dg-additional-options isn't supported in tree-prof. It may well be that just accepting it in profopt.exp (and compat.exp and lto.exp and plugin-support.exp that all have copies of that code ;-) works. I'll give it a try. Rainer -- - Rainer Orth, Center for Biotechnology, Bielefeld University
Re: [committed] Fix failure of gcc.dg/tree-prof/pr59003.c on hppa*-*-hpux*
John David Anglin dave.ang...@bell.net writes: I already tried it. dg-additional-options isn't supported in tree-prof. Shouldn't be hard to fix. Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7 And now for something completely different.
[committed] Fix failure of gcc.dg/tree-prof/pr59003.c on hppa*-*-hpux*
gcc.dg/tree-prof/pr59003.c fails on hppa*-*-hpux* because of limited variable alignment when -fno-common is not specified in the options. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2014-03-23 John David Anglin dang...@gcc.gnu.org * gcc.dg/tree-prof/pr59003.c: Add -fno-common to dg-options on hppa*-*-hpux*. Index: gcc.dg/tree-prof/pr59003.c === --- gcc.dg/tree-prof/pr59003.c (revision 208769) +++ gcc.dg/tree-prof/pr59003.c (working copy) @@ -1,6 +1,7 @@ /* PR target/59003 */ /* { dg-options -O2 } */ /* { dg-options -O2 -mtune=amdfam10 { target i?86-*-* x86_64-*-* } } */ +/* { dg-options -O2 -fno-common { target hppa*-*-hpux* } } */ __attribute__((noinline, noclone)) void * foo (void *p, unsigned int q)