Re: [wwwdocs] document -Wshadow change in 4.8

2013-02-09 Thread Gerald Pfeifer
On Sat, 9 Feb 2013, Manuel López-Ibáñez wrote: > OK to commit? > > * htdocs/gcc-4.8/changes.html: Document -Wshadow change. Thanks! The only question I have is about "unless the variable is a function". How can a variable be a function? Gerald

Go patch committed: Let test import package that imports tested pkg

2013-02-09 Thread Ian Lance Taylor
This patch to gccgo lets a test of package P1 import a package P2 where P2 itself imports P1. gccgo was giving an error for this kind of import. That can be useful in detecting a failure to rebuild a package that depends on some other package, but it prevented this useful behaviour. So I removed

Re: [PATCH] Fix PR56231

2013-02-09 Thread Andi Kleen
Richard Biener writes: > This fixes broken include information printed on diagnostics from the > LTO frontend. We fail to properly leave the file we're still in > after processing one data_in. Instead of figuring out where exactly > we miss a clear_line_info call the following simply caches the

Re: [PATCH] Fix bootstrap with -O3

2013-02-09 Thread Marc Glisse
On Thu, 7 Feb 2013, Marek Polacek wrote: When bootstrapping with BOOT_FLAGS='-O3 -g', I get: /home/polacek/src/gcc/gcc/c/c-parser.c: In function ‘c_expr c_parser_postfix_expression_after_primary(c_parser*, location_t, c_expr)’: /home/polacek/src/gcc/gcc/c/c-parser.c:6908:16: error: ‘origtypes’

libgo patch committed: If using DejaGNU, don't frob log file

2013-02-09 Thread Ian Lance Taylor
If the libgo testsuite is using DejaGNU, there is no need to frob the log file to make it look like DejaGNU output. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu (without DejaGNU). Committed to mainline. Ian

libgo patch committed: Include timeout.exp in libgo.exp

2013-02-09 Thread Ian Lance Taylor
Andreas pointed out a problem in the libgo testsuite: it needs to explicitly include timeout.exp before including go.exp. Fixed like so. Committed to mainline. Ian diff -r 45e8ffa934c1 libgo/testsuite/lib/libgo.exp --- a/libgo/testsuite/lib/libgo.exp Fri Feb 08 11:22:34 2013 -0800 +++ b/libgo/te

Re: [PATCH, PR] Crash of Bessel functions at x==0!

2013-02-09 Thread François Dumont
Attached patch applied then. 2013-02-09 François Dumont * include/tr1/bessel_function.tcc (__cyl_bessel_ij_series): Code simplification. On 02/08/2013 08:46 PM, Paolo Carlini wrote: On 02/08/2013 07:08 PM, François Dumont wrote: Just a small remark, in bessel_function.tcc, the fol

C++ PATCH for c++/56247 (ICE with PMF in two template member functions)

2013-02-09 Thread Jason Merrill
Here, the internal representation of &Base::method in the templates involves an OFFSET_REF around 'this', and we were treating the two instances of Wrapper as equivalent. But they involve different 'this' parameters, which makes a difference when we go to look up the local specialization. So

C++ PATCH to decls_match to improve efficiency of version checking

2013-02-09 Thread Jason Merrill
While looking at 56247 I noticed that the new multiversioning code was unnecessarily duplicating the calls to compare parameter and return types of a function. Fixed by moving the version code a little later. Tested x86_64-pc-linux-gnu, applying to trunk. commit 3c47969eacbc63b68740e48c0e9dae0

C++ PATCH for c++/56238 (ice-checking with static_cast)

2013-02-09 Thread Jason Merrill
The code in build_non_dependent_expr has caught a number of issues in the compiler; this is another case that fold_non_dependent_expr wasn't properly recognizing as dependent. I think we want to use instantiation_dependent there going forward, but for 4.8 I think let's just check it in the ENA

Re: [PATCH] Improve debugging info (PR debug/53948 P1 regression)

2013-02-09 Thread Jeff Law
On 02/09/2013 06:49 AM, Eric Botcazou wrote: Do we have any dwarf targets that don't support .loc? Solaris targets with the Solaris assembler, I think. Thanks. I found out that ia64 only conditionally uses .loc as well. Regardless, I used Jakub's regexp, so we should be good for targets wit

Re: [patch, Fortran] Fix PR 56224

2013-02-09 Thread Steve Kargl
On Sat, Feb 09, 2013 at 02:14:53PM +0100, Thomas Koenig wrote: > Am 09.02.2013 11:22, schrieb Tobias Burnus: > > > Why did you put a FIXME there? What's wrong with adding the directory here? > > I think module files are different enough from include files that I > would like to have them in diffe

Re: Go testsuite patch committed: Load timeout.exp

2013-02-09 Thread Ian Lance Taylor
On Sat, Feb 9, 2013 at 3:18 AM, Andreas Schwab wrote: > Ian Lance Taylor writes: > >> 2013-02-08 Ian Lance Taylor >> >> * lib/go.exp: Load timeout.exp. > > ERROR: Couldn't find library file timeout.exp. Sorry, I was told on IRC that the patch worked. As I mentioned on the bug, can you

Re: [PATCH] Fix libiberty/regex on 64-bit hosts (PR other/56245)

2013-02-09 Thread Ian Lance Taylor
On Sat, Feb 9, 2013 at 12:24 AM, Jakub Jelinek wrote: > > 2013-02-09 Jakub Jelinek > > PR other/56245 > * regex.c (PTR_INT_TYPE): Define. > (EXTEND_BUFFER): Change incr type from int to PTR_INT_TYPE. This is OK. Thanks. Ian

[wwwdocs] document -Wshadow change in 4.8

2013-02-09 Thread Manuel López-Ibáñez
OK to commit? * htdocs/gcc-4.8/changes.html: Document -Wshadow change. wshadow.diff Description: Binary data

[PATCH] Allow implicit Altivec and VSX on AIX 6.1 and above

2013-02-09 Thread David Edelsohn
AIX 6.1 assembler allows enough alignment for Altivec to work with GCC, so the following patch allows -mcpu= to enable Altivec and VSX by default. Bootstrapped on powerpc-ibm-aix7.1.0.0. - David * config/rs6000/aix61.h (OS_MISSING_ALTIVEC): Undefine. Index: aix61.h =

Re: [PATCH, asan]: Compile g++.dg/asan tests with -D__NO_INLINE__ for *-*-linux* targets

2013-02-09 Thread Uros Bizjak
On Sat, Feb 9, 2013 at 2:47 PM, Jakub Jelinek wrote: > On Sat, Feb 09, 2013 at 12:46:23PM +0100, Uros Bizjak wrote: >> 2013-02-09 Uros Bizjak >> >> * g++.dg/asan/asan_test.C: Compile with -D__NO_INLINE__ >> for *-*-linux* targets. >> * g++.dg/asan/interception-test-1.c: Ditto.

Re: [PATCH] Improve debugging info (PR debug/53948 P1 regression)

2013-02-09 Thread Eric Botcazou
> Do we have any dwarf targets that don't support .loc? Solaris targets with the Solaris assembler, I think. -- Eric Botcazou

Re: [PATCH, asan]: Compile g++.dg/asan tests with -D__NO_INLINE__ for *-*-linux* targets

2013-02-09 Thread Jakub Jelinek
On Sat, Feb 09, 2013 at 12:46:23PM +0100, Uros Bizjak wrote: > 2013-02-09 Uros Bizjak > > * g++.dg/asan/asan_test.C: Compile with -D__NO_INLINE__ > for *-*-linux* targets. > * g++.dg/asan/interception-test-1.c: Ditto. > * g++.dg/asan/interception-failure-test-1.C: Ditto.

Re: [patch, Fortran] Fix PR 56224

2013-02-09 Thread Thomas Koenig
Am 09.02.2013 11:22, schrieb Tobias Burnus: Why did you put a FIXME there? What's wrong with adding the directory here? I think module files are different enough from include files that I would like to have them in different directories. > "Seems to break testing, all testcases emit > Warning

[PATCH, asan]: Compile g++.dg/asan tests with -D__NO_INLINE__ for *-*-linux* targets

2013-02-09 Thread Uros Bizjak
Hello! Older glibcs (CentOS 2.5) have specialized extern inline functions defined in stdlib.h that interfere with ASANs testcases in g++.dg/asan: #ifdef __USE_EXTERN_INLINES /* Define inline functions which call the internal entry points. */ ... These cause many strtol, strtoll and atoi runtime

Re: Go testsuite patch committed: Load timeout.exp

2013-02-09 Thread Andreas Schwab
Ian Lance Taylor writes: > 2013-02-08 Ian Lance Taylor > > * lib/go.exp: Load timeout.exp. ERROR: Couldn't find library file timeout.exp. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something

Re: [patch, Fortran] Fix PR 56224

2013-02-09 Thread Jakub Jelinek
On Sat, Feb 09, 2013 at 11:22:18AM +0100, Tobias Burnus wrote: > Are you sure that the test case in gfortran.dg/gomp/omp_lib_1.F > works? The OpenMP Fortran headers (and modules) are generated in > libgomp/. Thus, I am not sure they are available in > gcc/testsuite/gfortran.dg/gomp (in particular w

Re: [patch, Fortran] Fix PR 56224

2013-02-09 Thread Tobias Burnus
Paul Richard Thomas wrote: It looks OK to me - is Jakub OK with it? On 8 February 2013 21:48, Thomas Koenig wrote: the attached patch fixes the PR by (re-)adding a search path for the path used by intrinsic modules. Regression-tested. OK for trunk? I am still recovering from a cold, but I

Re: [patch, Fortran] Fix PR 56224

2013-02-09 Thread Paul Richard Thomas
It looks OK to me - is Jakub OK with it? Cheers Paul On 8 February 2013 21:48, Thomas Koenig wrote: > Hello world, > > the attached patch fixes the PR by (re-)adding a search path > for the path used by intrinsic modules. > > Regression-tested. OK for trunk? > > Thomas > > 2013-02-08

[Patch, fortran] PR55362 - [4.6/4.7/4.8 Regression] ICE with size() on character pointer

2013-02-09 Thread Paul Richard Thomas
Dear All, Committed as obvious revision 195915. The patch is a generalisation of Tobias' fix. 2013-02-09 Paul Thomas PR fortran/55362 * check.c (array_check): It is an error if a procedure is passed. 2013-02-09 Paul Thomas PR fortran/55362 * gfortran.dg/intrinsic_siz

Re: Fix inline-1.C and inline-3.C testcases

2013-02-09 Thread Jan Hubicka
> > > > > > gcc/testsuite/g++.dg/ipa/inline-3.C > > > has > > > /* { dg-final { cleanup-tree-dump "inline" } } */ > > > while supposedly it should have: > > > /* { dg-final { cleanup-ipa-dump "inline" } } */ > > > > > > Also, > > > gcc/testsuite/gcc.dg/tree-ssa/inline-3.c > > > is now failing, su

[PATCH, 4.7]: Include output.h in lto.c

2013-02-09 Thread Uros Bizjak
Hello! We need to #include "output.h" due to the use of %wx in a diagnostic string. For 4.8 Steven moved "typedef HOST_WIDE_INT __gcc_host_wide_int__" to hwint.h, but for 4.7 branch we need output.h for the typedef. 2013-02-09 Uros Bizjak * lto.c: Include output.h. Bootstrapped on x8

Re: [PATCH] Restore backwards compatibility of ppc inline asm with asm dialects (PR target/56256)

2013-02-09 Thread David Edelsohn
On Sat, Feb 9, 2013 at 3:21 AM, Jakub Jelinek wrote: > Hi! > > When old mnemonics support has been dropped, the ASSEMBLER_DIALECT > macro has been removed also. This has unfortunate effect that > projects that used {|} in inline asm for ppc no longer compile, as the > {, | and } characters are no

[PATCH] Fix libiberty/regex on 64-bit hosts (PR other/56245)

2013-02-09 Thread Jakub Jelinek
Hi! gengtype when built with -fsanitize=address fails on x86_64-linux. The problem is that realloc often returns addresses which are more than 2GB appart from the old value, and regex.c was using int variable for the pointer difference to adjust the pointers in the realloced buffer. glibc uses a

[PATCH] Restore backwards compatibility of ppc inline asm with asm dialects (PR target/56256)

2013-02-09 Thread Jakub Jelinek
Hi! When old mnemonics support has been dropped, the ASSEMBLER_DIALECT macro has been removed also. This has unfortunate effect that projects that used {|} in inline asm for ppc no longer compile, as the {, | and } characters are now passed through to the assembler. This patch defines ASSEMBLER_