Re: [committed] Fix failure of gcc.dg/tree-prof/pr59003.c on hppa*-*-hpux*

2014-03-24 Thread Rainer Orth
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*

2014-03-24 Thread John David Anglin

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*

2014-03-24 Thread Rainer Orth
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*

2014-03-24 Thread Andreas Schwab
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*

2014-03-23 Thread John David Anglin
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)