Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Richard Biener
Gerald Pfeifer wrote: >On Fri, 21 Dec 2012, Dominique Dhumieres wrote: >> I think revision 194665 breaks bootstrap on at least >x86_64-apple-darwin10: > >Same on everything FreeBSD. > >PR 55784 - [4.8 regression] declaration of C function 'const char* >strsignal(int)' conflicts with /usr/include

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Andreas Schwab
Richard Biener writes: > (I suppose somehow gmp.h is pulled in configure tests?) AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ #include "ansidecl.h" #include "system.h" #ifdef HAVE_SYS_RESOURCE_H #include #endif ]], [[rlim_t l = 0;]])],[],[AC_DEFINE([rlim_t],[long], [Define to `long' if doesn't define

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Richard Biener
On Sat, Dec 22, 2012 at 10:49 AM, Andreas Schwab wrote: > Richard Biener writes: > >> (I suppose somehow gmp.h is pulled in configure tests?) > > AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ > #include "ansidecl.h" > #include "system.h" > #ifdef HAVE_SYS_RESOURCE_H > #include > #endif > ]], [[rlim_t l

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Dominique Dhumieres
The following patch allowed me to proceed for c,c++,lto,fortran,ada,objc,obj-c++ up to libada which has the same problem: --- ../_clean/gcc/configure 2012-12-20 17:19:54.0 +0100 +++ ../p_work/gcc/configure 2012-12-21 23:44:46.0 +0100 @@ -10321,9 +10321,9 @@ $as_echo "#defin

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Andreas Schwab
Richard Biener writes: > On Sat, Dec 22, 2012 at 10:49 AM, Andreas Schwab > wrote: >> Richard Biener writes: >> >>> (I suppose somehow gmp.h is pulled in configure tests?) >> >> AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[ >> #include "ansidecl.h" >> #include "system.h" >> #ifdef HAVE_SYS_RESOURCE_H

Re: [PATCH] Fix PR54659, include gmp.h from system.h

2012-12-22 Thread Richard Sandiford
domi...@lps.ens.fr (Dominique Dhumieres) writes: > The following patch allowed me to proceed for > c,c++,lto,fortran,ada,objc,obj-c++ > up to libada which has the same problem: > > --- ../_clean/gcc/configure 2012-12-20 17:19:54.0 +0100 > +++ ../p_work/gcc/configure 2012-12-21 23:44:46

Re: [PATCH] PR c++/52343 - error with alias template as template template argument

2012-12-22 Thread Dodji Seketeli
Gabriel Dos Reis writes: > Thank you very much for the explanation; your previous message > makes sense to me now. You are welcome. > The question I have is why are we using TREE_TYPE of a TEMPLATE_DECL > to represent the current instantiation of a template alias? My understanding is that in t

[PATCH] PR c++/55311 - Cannot specialize alias template with arg of type array of char

2012-12-22 Thread Dodji Seketeli
Hello, Consider this test case: 1 template 2 struct A 3 {}; 4 5 struct B {}; 6 7 extern constexpr char HELLO_WORLD[] = "hello world"; 8 9 A g; // <-- This works fine 10 11 template 12 using PartiallySpecialized = A; // <-- This f

Re: [PATCH] PR c++/52343 - error with alias template as template template argument

2012-12-22 Thread Gabriel Dos Reis
On Sat, Dec 22, 2012 at 9:53 AM, Dodji Seketeli wrote: > Gabriel Dos Reis writes: > >> Thank you very much for the explanation; your previous message >> makes sense to me now. > > You are welcome. > >> The question I have is why are we using TREE_TYPE of a TEMPLATE_DECL >> to represent the curren

Re: [Patch, Fortran] PR55763 fix .mod reading plus CALL with CLASS(*)

2012-12-22 Thread Paul Richard Thomas
Dear Tobias, I did this one myself on the plane to the UK last night! We came to the same patch, so yes it's OK for trunk. Thanks Paul On 21 December 2012 22:41, Tobias Burnus wrote: > Another two fixes for CLASS(*). (We really should audit all calls to > gfc_find_derived_vtab for possible is

Merge from trunk to gccgo branch

2012-12-22 Thread Ian Lance Taylor
I've merged trunk revision 194692 to the gccgo branch. Ian

Re: C++ PATCH for c++/54325 (wrong error initializing abstract base class)

2012-12-22 Thread Jason Merrill
On 12/21/2012 06:38 AM, Paolo Carlini wrote: I was looking a bit more into this Bug, and something seems still weird about the testcase in Comment #1 of the audit trail, which we also didn't reject with 4.6.x: What's weird about it? Jason

Re: [Patch, libfortran] PR 55539 Regression with -fno-sign-zero

2012-12-22 Thread Janne Blomqvist
Ping! On Sat, Dec 15, 2012 at 6:32 PM, Janne Blomqvist wrote: > Hello, > > the attached patch fixes PR 55539 (4.8 regression). Regtested on > x86_64-unknown-linux-gnu, Ok for trunk? > > > 2012-12-15 Janne Blomqvist > > PR fortran/55539 > * io/write_float.def (output_float): Tak

Re: C++ PATCH for c++/54325 (wrong error initializing abstract base class)

2012-12-22 Thread Paolo Carlini
Hi, Jason Merrill ha scritto: >On 12/21/2012 06:38 AM, Paolo Carlini wrote: >> I was looking a bit more into this Bug, and something seems still >weird about the testcase in Comment #1 of the audit trail, which we >also didn't reject with 4.6.x: > >What's weird about it? Well, we still reject i

PR lto/54728 (streamer ICE)

2012-12-22 Thread Jan Hubicka
Hi, in the testcase I don't really know how to put into testsuite since it needs DSO there is issue with LTO not merging external declaration with body. This is because logic in symtab_real_symbol_p that really applies only to LTO symbol table output not to the symbol merging. Bootstrapped/regte