Re: [google/gcc-4_7, trunk] Fix problem with -fdebug-types-section and template instantiations, take 2

2012-08-29 Thread Hans-Peter Nilsson
On Tue, 28 Aug 2012, Cary Coutant wrote: 2012-08-28 Cary Coutant ccout...@google.com * gcc/dwarf2out.c (clone_tree_partial): Remove. (copy_decls_walk): Don't copy children of a declaration into a type unit. Can't say anything about the patch but watch out with the

RE: Ping: [PATCH] Enable bbro for -Os

2012-08-29 Thread Zhenqiang Chen
-Original Message- From: Steven Bosscher [mailto:stevenb@gmail.com] Sent: Friday, August 24, 2012 8:17 PM To: Zhenqiang Chen Cc: gcc-patches@gcc.gnu.org Subject: Re: Ping: [PATCH] Enable bbro for -Os On Wed, Aug 22, 2012 at 8:49 AM, Zhenqiang Chen zhenqiang.c...@arm.com

Re: [PATCH] MIPS16 TLS support for GCC

2012-08-29 Thread Chung-Lin Tang
On 2012/7/6 02:23 PM, Richard Sandiford wrote: Richard Sandiford rdsandif...@googlemail.com writes: (3) Also related to libraries, I edited CRT_CALL_STATIC_FUNCTION to emit a 32-bit code sequence under both MIPS/MIPS16 mode (under O32). As you can see in the original Feb. patch, I had changes

[AArch64] Merge from upstream trunk r190706

2012-08-29 Thread Sofiane Naci
Hi, I've just merged upstream trunk on the aarch64-branch up to r190706. Thanks Sofiane

remove dependency on cp/parser.h from cp/lang.c

2012-08-29 Thread Aaron Gray
Just got my copyright assignment through, so here's my first GCC patch, This is a one liner removing the unneeded dependency of cp-lang.c on cp/parser.h. This has been tested on Linux. Aaron diff --git a/gcc/cp/cp-lang.c b/gcc/cp/cp-lang.c index da7f1e1..5ca0b0a 100644 --- a/gcc/cp/cp-lang.c

Re: Inline hints

2012-08-29 Thread Martin Jambor
Hi, On Tue, Aug 28, 2012 at 06:05:27PM +0200, Jan Hubicka wrote: On Sun, Aug 19, 2012 at 07:43:45AM +0200, Jan Hubicka wrote: * gcc.dg/ipa/iinline-1.c: Update testcase to test inline hints. * ipa-inline.c (want_inline_small_function_p): Bypass inline limits for hinted

Re: [PATCH, libstdc++] Make empty std::string storage readonly

2012-08-29 Thread Michael Haubenwallner
On 08/28/2012 08:12 PM, Jonathan Wakely wrote: On 28 August 2012 18:27, Michael Haubenwallner wrote: Does it actually produce a segfault? I suppose it might on some platforms, but not all, so I'm not sure it's worth changing. It does segfault here on (32bit each): i686-pc-linux-gnu

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-08-29 Thread Jan Hubicka
Index: libgcc/libgcov.c === --- libgcc/libgcov.c (revision 190736) +++ libgcc/libgcov.c (working copy) @@ -276,6 +276,78 @@ gcov_version (struct gcov_info *ptr, gcov_unsigned return 1; } +/* Insert counter VALUE into

Re: out-of-line and arch-specific random_device

2012-08-29 Thread Paolo Carlini
Hi, On 8/28/12 1:41 PM, Ulrich Drepper wrote: On Tue, Aug 28, 2012 at 4:44 AM, Paolo Carlinipaolo.carl...@oracle.com wrote: Again, without context, I think this is not the point: random_device is meant to be just a simple high level wrapper around things like dev/random, inspired by

[PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Tulio Magno Quites Machado Filho
Add __builtin_ppc_get_timebase to read the time base register on PowerPC. This is required for applications that measure time at high frequencies with high precision that can't afford a syscall. [gcc] 2012-08-29 Tulio Magno Quites Machado Filho tul...@linux.vnet.ibm.com *

Re: [PATCH,PING]] gcc/config/freebsd-spec.h: Fix building PIE

2012-08-29 Thread Alexis Ballier
Hi Gerald, On Sun, 26 Aug 2012 23:28:49 +0200 (CEST) Gerald Pfeifer ger...@pfeifer.com wrote: I have tested this patch on i386-unknown-freebsd10.0 and volunteer to create a ChangeLog and apply if approved. Thanks for taking care of this, I thought this patch had been completely forgotten :)

Re: faster random number engine

2012-08-29 Thread Paolo Carlini
On 8/29/12 4:19 PM, Ulrich Drepper wrote: Therandom header so far contains the random number engines documented in the header. None of these are well suited for modern CPUs. There is a variant of the Mersenne twister engines which is explicitly designed to perform well on CPUs with SIMD

Re: [PATCH 2/3] Incorporate aggregate jump functions into inlining analysis

2012-08-29 Thread H.J. Lu
On Thu, Aug 2, 2012 at 12:28 PM, Martin Jambor mjam...@suse.cz wrote: Hi, this patch uses the aggregate jump functions created by the previous patch in the series to determine benefits of inlining a particular call graph edge. It has not changed much since the last time I posted it, except

Re: out-of-line and arch-specific random_device

2012-08-29 Thread Ulrich Drepper
On Wed, Aug 29, 2012 at 9:48 AM, Paolo Carlini paolo.carl...@oracle.com wrote: Minor nit: are you sure we need to open a new minor version for the new symbol? Because it seemed to me that 4.7.x was behind by one. I have 4.7 installed and that version already defines the symbols defined in

Re: out-of-line and arch-specific random_device

2012-08-29 Thread Paolo Carlini
On 8/29/12 4:49 PM, Ulrich Drepper wrote: On Wed, Aug 29, 2012 at 9:48 AM, Paolo Carlinipaolo.carl...@oracle.com wrote: Minor nit: are you sure we need to open a new minor version for the new symbol? Because it seemed to me that 4.7.x was behind by one. I have 4.7 installed and that version

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Segher Boessenkool
Hi Tulio, Add __builtin_ppc_get_timebase to read the time base register on PowerPC. This is required for applications that measure time at high frequencies with high precision that can't afford a syscall. For things that do mftb with high frequency, maybe you should also add a builtin

[PATCH] PR other/54411: libiberty: objalloc_alloc integer overflows (CVE-2012-3509)

2012-08-29 Thread Florian Weimer
This patches fixes an integer overflow in libiberty, which leads to crashes in binutils. The long version of the objalloc_alloc macro would have needed another conditional, so I removed that and replaced it with a call to the actual implementation. This has been compiled-tested only. We do not

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Andrew Pinski
On Wed, Aug 29, 2012 at 6:56 AM, Tulio Magno Quites Machado Filho tul...@linux.vnet.ibm.com wrote: Add __builtin_ppc_get_timebase to read the time base register on PowerPC. This is required for applications that measure time at high frequencies with high precision that can't afford a syscall.

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Hans-Peter Nilsson
On Wed, 29 Aug 2012, Segher Boessenkool wrote: +++ b/gcc/testsuite/gcc.target/powerpc/ppc-get-timebase.c @@ -0,0 +1,22 @@ +/* { dg-do run { target { powerpc*-*-* } } } */ + +/* Test if __builtin_ppc_get_timebase() is compatible with the current + processor and if it's changing

Re: Loop iterations inline hint

2012-08-29 Thread Martin Jambor
Hi, On Tue, Aug 21, 2012 at 08:55:02AM +0200, Jan Hubicka wrote: Hi, this patch adds a hint that if inlining makes bounds on loop iterations known, it is probably good idea. This is primarely targetting Fortran's array descriptors, but should be generally useful. Fortran will still need

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Michael Meissner
On Wed, Aug 29, 2012 at 01:56:05PM -0400, Hans-Peter Nilsson wrote: On Wed, 29 Aug 2012, Segher Boessenkool wrote: On some systems the timebase runs at a rather low frequency, say 20MHz. This test will spuriously fail there. Waste a million CPU cycles before reading TB the second time?

[PATCH, PR 54394] Compute loops when generating inline summaries

2012-08-29 Thread Martin Jambor
Hi, the patch below fixes PR 54394. The problem is that since revision 190346 we depend on bb-loop_father being non-NULL to get loop_depth. However, with loops not computed, the loop_father is NULL, loop_depth is thus considered zero and call graph edges out of such BB can be considered much

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Tulio Magno Quites Machado Filho
Hi Segher, Segher Boessenkool seg...@kernel.crashing.org writes: Add __builtin_ppc_get_timebase to read the time base register on PowerPC. This is required for applications that measure time at high frequencies with high precision that can't afford a syscall. For things that do mftb with

Re: faster random number engine

2012-08-29 Thread Ulrich Drepper
On Wed, Aug 29, 2012 at 11:43 AM, Paolo Carlini paolo.carl...@oracle.com wro The substance isn't of course. But normally we don't have __gnu_cxx things in the same std header. Can't we have a new ext/random and put it in there? If we can separate the new code to it, I think people would not

Re: [PATCH] MIPS16 TLS support for GCC

2012-08-29 Thread Richard Sandiford
Chung-Lin Tang clt...@codesourcery.com writes: On 2012/7/6 02:23 PM, Richard Sandiford wrote: Richard Sandiford rdsandif...@googlemail.com writes: (3) Also related to libraries, I edited CRT_CALL_STATIC_FUNCTION to emit a 32-bit code sequence under both MIPS/MIPS16 mode (under O32). As you

Re: remove dependency on cp/parser.h from cp/lang.c

2012-08-29 Thread Marc Glisse
On Wed, 29 Aug 2012, Aaron Gray wrote: Just got my copyright assignment through, so here's my first GCC patch, Welcome! This is a one liner removing the unneeded dependency of cp-lang.c on cp/parser.h. This has been tested on Linux. I think you need to attach a ChangeLog entry with every

Re: [patch] Fix problems with -fdebug-types-section and local types

2012-08-29 Thread Cary Coutant
Ping. http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00398.html Because much of this patch was superceded by this recent patch: http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01968.html I'll combine the two and submit a new patch. -cary

Re: [MIPS, committed] Add missing COSTS_N_INSNS call.

2012-08-29 Thread Richard Sandiford
Richard Sandiford rdsandif...@googlemail.com writes: Hans-Peter Nilsson h...@bitrange.com writes: On Tue, 28 Aug 2012, Richard Sandiford wrote: Hans-Peter Nilsson h...@bitrange.com writes: On Sun, 26 Aug 2012, Richard Sandiford wrote: I'm preparing a patch to turn gcc.target/mips into a

[patch] Fix problems with -fdebug-types-section

2012-08-29 Thread Cary Coutant
I've combined these two pending patches into one: http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00398.html http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01968.html The first patch fixed a problem with copying too much of a referenced type into a type unit, by changing clone_tree_hash() to

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Segher Boessenkool
On some systems the timebase runs at a rather low frequency, say 20MHz. This test will spuriously fail there. Waste a million CPU cycles before reading TB the second time? Waste said million cycles portably by calling sched_yield()? (Available only on POSIX systems. :) I was thinking

Re: [google/gcc-4_7, trunk] Fix problem with -fdebug-types-section and template instantiations, take 2

2012-08-29 Thread Cary Coutant
This patch is for trunk and the google/gcc-4_7 branch. 2012-08-28 Cary Coutant ccout...@google.com * gcc/dwarf2out.c (clone_tree_partial): Remove. (copy_decls_walk): Don't copy children of a declaration into a type unit. For trunk, I've submitted a new patch that

Re: [google/gcc-4_7, trunk] Fix problem with -fdebug-types-section and template instantiations, take 2

2012-08-29 Thread Sterling Augustine
On Wed, Aug 29, 2012 at 12:03 PM, Cary Coutant ccout...@google.com wrote: This patch is for trunk and the google/gcc-4_7 branch. 2012-08-28 Cary Coutant ccout...@google.com * gcc/dwarf2out.c (clone_tree_partial): Remove. (copy_decls_walk): Don't copy children of a

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Segher Boessenkool
For things that do mftb with high frequency, maybe you should also add a builtin that does just an mftb, i.e. returns a 32-bit result on 32- bit implementations. Are you thinking in a function that returns only the TBL? On 32-bit, just TBL; on 64-bit, the whole TB (there is no machine

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Hans-Peter Nilsson
On Wed, 29 Aug 2012, Michael Meissner wrote: On Wed, Aug 29, 2012 at 01:56:05PM -0400, Hans-Peter Nilsson wrote: On Wed, 29 Aug 2012, Segher Boessenkool wrote: On some systems the timebase runs at a rather low frequency, say 20MHz. This test will spuriously fail there. Waste a million

Re: [MIPS, committed] Add missing COSTS_N_INSNS call.

2012-08-29 Thread Hans-Peter Nilsson
On Wed, 29 Aug 2012, Richard Sandiford wrote: Richard Sandiford rdsandif...@googlemail.com writes: I'm testing a patch to make the testsuite work out the default -m{no,}synci, which ought to be enough. The usual rules should then kick in and force -mno-synci where necessary. Hopefully.

[wwwdocs] SH 4.8 changes update

2012-08-29 Thread Oleg Endo
Hello, The new SH option -menable-tas has been renamed to -mtas in rev 190782. I have committed the attached patch to reflect this in the changes.html for 4.8. Cheers, Oleg ? sh_mtas_rename.patch Index: htdocs/gcc-4.8/changes.html

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Tulio Magno Quites Machado Filho
Segher Boessenkool seg...@kernel.crashing.org writes: For things that do mftb with high frequency, maybe you should also add a builtin that does just an mftb, i.e. returns a 32-bit result on 32- bit implementations. Are you thinking in a function that returns only the TBL? On 32-bit,

Re: [Fortran] PR37336 - FIINAL patch [1/n]: Implement the finalization wrapper subroutine

2012-08-29 Thread Tobias Burnus
Dear all, that's the revised version of patch at http://gcc.gnu.org/ml/fortran/2012-08/msg00095.html, taking the review comments into account. Reminder: This patch only generates the finalization wrapper, which is in the virtual table. It does not add the required calls; hence, it still

Re: remove dependency on cp/parser.h from cp/lang.c

2012-08-29 Thread Aaron Gray
On 29 August 2012 19:47, Marc Glisse marc.gli...@inria.fr wrote: On Wed, 29 Aug 2012, Aaron Gray wrote: Just got my copyright assignment through, so here's my first GCC patch, Welcome! Thanks Marc ! This is a one liner removing the unneeded dependency of cp-lang.c on cp/parser.h.

[patch] Fix CFG dumping of blocks with no predecessors or successors

2012-08-29 Thread Steven Bosscher
Will commit as obvious. * cfg.c (dump_bb_info): Print a newline if there were no edges to dump. Index: cfg.c === --- cfg.c (revision 190785) +++ cfg.c (working copy) @@ -764,6 +764,8 @@ dump_bb_info (FILE *outf,

[PATCH] Remove dependency of cp/cp-lang.c on cp/parser.h

2012-08-29 Thread Aaron Gray
Patch removing the dependency of cp/cp-lang.c on cp/parser.c. This as been tested on Linux. [gcc/cp] 2012-08-29 Aaron Gray aaronngray.li...@gmail.com * cp/cp-lang.c: removed #include parser.h * cp/Make-lang.in: removed dependency of cp/cp-lang.c on cp/parser.h diff --git

Re: [PATCH] rs6000: Add a builtin to read the time base register on PowerPC

2012-08-29 Thread Segher Boessenkool
Point being, for simulator environments, you may not want the loop that was suggested later. On the other hand, that might not be an observable period, either. I don't think looping a million times would be too slow for the testsuite: there are many tests that do a lot more work than that,

Re: [PATCH] Add counter histogram to fdo summary (issue6465057)

2012-08-29 Thread Teresa Johnson
On Wed, Aug 29, 2012 at 6:12 AM, Jan Hubicka hubi...@ucw.cz wrote: Index: libgcc/libgcov.c === --- libgcc/libgcov.c (revision 190736) +++ libgcc/libgcov.c (working copy) @@ -276,6 +276,78 @@ gcov_version (struct gcov_info *ptr,

Re: [PATCH 1/6] Thread pointer built-in functions, core parts

2012-08-29 Thread Richard Henderson
On 2012-08-28 01:13, Chung-Lin Tang wrote: + icode = optab_handler (get_thread_pointer_optab, Pmode); Until we decide there's no point in the distinction, this should be spelled direct_optab_handler, to match OPTAB_D with which the optab is declared. Otherwise ok. r~

Re: [PATCH 2/6] Thread pointer built-in functions, alpha

2012-08-29 Thread Richard Henderson
On 2012-08-28 01:13, Chung-Lin Tang wrote: Alpha patch updated to use MD pattern. Ok. r~

[middle-end] Add machine_mode to address_cost target hook

2012-08-29 Thread Oleg Endo
Hello, While experimenting a little bit with an idea for an address mode selection RTL pass for SH, I realized that SH's sh_address_cost function is quite broken. When trying to fix it, I ran against a wall, since the mode of the MEM is not passed to the target hook function, as it is e.g. in

Re: [PATCH] C++'ization of cp/parser.c/h

2012-08-29 Thread Gabriel Dos Reis
On Wed, Aug 29, 2012 at 8:01 PM, Aaron Gray aaronngray.li...@gmail.com wrote: Patch to SVN HEAD that initially C++'izes cp/parser.h and cp/parser.c by class'izing the cp_lexer and cp_parser group of functions. For C programmers and for context all method calls are preceded by 'this-' and

Re: [PATCH] limited C++ parsing support for gengtype

2012-08-29 Thread Laurynas Biveinis
Hi - 2012/8/30 Aaron Gray aaronngray.li...@gmail.com: First of two patches for class'ized cp/parser.c|h gives limited support for gengtype to parse C++ classes and enums as first class citizens. Please sync with Diego to avoid duplicate work and/or conflicting designs. Thanks, -- Laurynas

Re: [PATCH] MIPS16 TLS support for GCC

2012-08-29 Thread Chung-Lin Tang
On 2012/8/30 02:44 AM, Richard Sandiford wrote: Chung-Lin Tang clt...@codesourcery.com writes: On 2012/7/6 02:23 PM, Richard Sandiford wrote: Richard Sandiford rdsandif...@googlemail.com writes: (3) Also related to libraries, I edited CRT_CALL_STATIC_FUNCTION to emit a 32-bit code sequence