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

> -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 > wrote: > >> The pat

Re: [PATCH] MIPS16 TLS support for GCC

On 2012/7/6 02:23 PM, Richard Sandiford wrote: > Richard Sandiford 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 to emit a >>>

[AArch64] Merge from upstream trunk r190706

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

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

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 lim

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

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-l

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

> 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 VAL

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

Hi, On 8/28/12 1:41 PM, Ulrich Drepper wrote: On Tue, Aug 28, 2012 at 4:44 AM, Paolo Carlini 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 facilities like dev/random on uni

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

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 * config/rs6000/rs6000-builtin.def: Add __b

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

Hi Gerald, On Sun, 26 Aug 2012 23:28:49 +0200 (CEST) Gerald Pfeifer 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 :) FWIW, the git comm

Re: faster random number engine

On 8/29/12 4:19 PM, Ulrich Drepper wrote: The 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 instructions

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

On Thu, Aug 2, 2012 at 12:28 PM, Martin Jambor 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 for the pr

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

On Wed, Aug 29, 2012 at 9:48 AM, Paolo Carlini 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 version 3.4.17. This is a

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

On 8/29/12 4:49 PM, Ulrich Drepper wrote: On Wed, Aug 29, 2012 at 9:48 AM, Paolo Carlini 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 symbol

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

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 that

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

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 u

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

On Wed, Aug 29, 2012 at 6:56 AM, Tulio Magno Quites Machado Filho 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. > > [gcc] > 2012-08-29 T

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

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 chan

Re: Loop iterations inline hint

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 stil

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

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 tim

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

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 coo

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

Hi Segher, Segher Boessenkool 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 high frequency, maybe you sh

Re: faster random number engine

On Wed, Aug 29, 2012 at 11:43 AM, Paolo Carlini 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 even object > to the targ

Re: [PATCH] MIPS16 TLS support for GCC

Chung-Lin Tang writes: > On 2012/7/6 02:23 PM, Richard Sandiford wrote: >> Richard Sandiford 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. pat

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

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 p

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

> 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.

Richard Sandiford writes: > Hans-Peter Nilsson writes: >> On Tue, 28 Aug 2012, Richard Sandiford wrote: >>> Hans-Peter Nilsson writes: >>> > On Sun, 26 Aug 2012, Richard Sandiford wrote: >>> >> I'm preparing a patch to turn gcc.target/mips into a torture-like >>> >> testsuite. >>> > >>> > While

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

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 cop

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

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 more

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

> This patch is for trunk and the google/gcc-4_7 branch. > > 2012-08-28 Cary Coutant > > * 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 combines thi

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

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

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

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 instr

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

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 mi

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

On Wed, 29 Aug 2012, Richard Sandiford wrote: > Richard Sandiford 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. > > Here's the patch.

[wwwdocs] SH 4.8 changes update

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

Segher Boessenkool 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, just TBL; on 64-bit, the whole

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

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 doe

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

On 29 August 2012 19:47, Marc Glisse 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. This has

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

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, basic_blo

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

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 * 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 a/gcc/cp/cp-lang.c b/gcc/cp/cp-

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

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, alre

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

On Wed, Aug 29, 2012 at 6:12 AM, Jan Hubicka 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, gcov

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

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

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

[PATCH] limited C++ parsing support for gengtype

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. Patch to SVN HEAD 2012-08-30 Aaron Gray * gengtype-lex.l: Support for FILE Support for C++ single line Comments Support for classe

[middle-end] Add machine_mode to address_cost target hook

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 leg

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

On Wed, Aug 29, 2012 at 8:01 PM, Aaron Gray 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 static method calls by 'c

RE: [Patch, test] Enable to prune warnings for tests defined in one exp file

> -Original Message- > From: Mike Stump [mailto:mikest...@comcast.net] > Sent: Tuesday, August 28, 2012 1:21 AM > To: Terry Guo > Cc: gcc-patches@gcc.gnu.org; Richard Guenther > Subject: Re: [Patch, test] Enable to prune warnings for tests defined > in one exp file > > On Aug 27, 2012, at

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

Hi - 2012/8/30 Aaron Gray : > 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

On 2012/8/30 02:44 AM, Richard Sandiford wrote: > Chung-Lin Tang writes: >> On 2012/7/6 02:23 PM, Richard Sandiford wrote: >>> Richard Sandiford 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 O