Re: [PATCH 2/2] mips: Add R4700 scheduling support

2012-06-03 Thread Richard Sandiford
Matt Turner writes: > The R4700 is identical to the R4600 except for the integer and > floating-point multiplication costs. > > See page 4 of http://datasheets.chipdb.org/IDT/MIPS/79RV4700.pdf > > 2012-03-24 Matt Turner > > gcc/ > * config/mips/4600.md (r4700_imul_si): New. >

RFA: Alternative iterator implementation

2012-06-03 Thread Richard Sandiford
As discussed in the context of the AARCH64 submission, this patch rewrites the iterator handling in read-rtl.c so that we record iterator positions using an on-the-side VEC rather than placeholder modes and codes. We then substitute in-place for each sequence of iterator values and take a deep cop

Re: [Fortran, patch] PR 48831 - Constant expression (PARAMETER array element) rejected as nonconstant

2012-06-03 Thread Alessandro Fanfarillo
Hi all, in attachment the patch which includes the review comments provided by Tobias. The patch is bootstrapped and tested on x86_64-unknown-linux-gnu. Regards. Alessandro 2012/5/20 Tobias Burnus : > Hi Alessandro, > > > Alessandro Fanfarillo wrote: >> >> in attachment there's a patch for PR

Re: [Fortran, DRAFT patch] PR 46321 - [OOP] Polymorphic deallocation

2012-06-03 Thread Alessandro Fanfarillo
> Right, the problem is that the _free component is missing. Just as the > _copy component, _free should be present for *every* vtype, no matter > if there are allocatable components or not. If the _free component is > not needed, it should be initialized to EXPR_NULL. With an "empty" _free functi

Document sincos standard pattern name

2012-06-03 Thread Oleg Endo
Hi, The attached patch adds some documentation for the sincos standard pattern name. Tested with 'make info dvi pdf'. Is the text correct and OK to apply? Maybe it would also make sense to apply it to the 4.7 branch? Cheers, Oleg ChangeLog: * gcc/doc/md.texi (Standard Pattern Names For

Re: [gcov] a few improvements

2012-06-03 Thread Nathan Sidwell
On 06/03/12 05:51, Xinliang David Li wrote: On Fri, Dec 30, 2011 at 10:25 AM, Nathan Sidwell wrote: I've committed this patch to fix and improve coverage reporting: 1) the time stamp local_tick will be -1 if the user overrides the random seed. In such cases the gcov data file should be deleted

Re: [google] Add options to pattern match function name for hotness attributes

2012-06-03 Thread Dehao Chen
Thank you guys for the comments, I'll update the patch to : 1. generalize the flag to enable other annotations such always_inline. 2. change to use deferred option. Thanks, Dehao On Sun, Jun 3, 2012 at 12:40 PM, Xinliang David Li wrote: > On Sat, Jun 2, 2012 at 11:11 AM, Jan Hubicka wrote: >>>

[committed] Reduce maximum PCREL17F branch offsets for PIC code

2012-06-03 Thread John David Anglin
Sometimes when generating PIC code, a call would exceed its maximum branch offset and link would fail. This occurs because PIC stubs are larger than non PIC stubs. Most recently this occurred building the Debian kde4libs package. This change adjusts the maximum branch offsets for PIC code so tha

Re: [Fortran, patch] PR 48831 - Constant expression (PARAMETER array element) rejected as nonconstant

2012-06-03 Thread Tobias Burnus
Hi Alessandro, hi all, Alessandro Fanfarillo wrote: in attachment the patch which includes the review comments provided by Tobias. Thanks for the patch, which I committed as Rev. 188152. Congratulation to your second committed patch. Nit: You forgot twice to add the prefix "gfc_" in the Cha

Re: [Fortran, patch] PR 48831 - Constant expression (PARAMETER array element) rejected as nonconstant

2012-06-03 Thread Alessandro Fanfarillo
Thank you Tobias, I thought that "Change name in gfc_check_init_expr" was sufficient. 2012/6/3 Tobias Burnus : > Hi Alessandro, hi all, > > > Alessandro Fanfarillo wrote: >> >> in attachment the patch which includes the review comments provided by >> Tobias. > > > Thanks for the patch, which I com

[wwwdocs] Buildstat update for 4.7

2012-06-03 Thread Tom G. Christensen
Latest results for 4.7.x -tgc Testresults for 4.7.0: hppa64-hp-hpux11.11 i386-pc-solaris2.8 x86_64-apple-darwin11.3.0 Index: buildstat.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/buildstat.html,v retrieving revision 1.4

RE: [Patch,AVR]: Fix PR46261

2012-06-03 Thread Weddington, Eric
> -Original Message- > From: Georg-Johann Lay [mailto:a...@gjlay.de] > Sent: Thursday, May 31, 2012 8:56 AM > To: gcc-patches@gcc.gnu.org > Cc: Denis Chertykov; Weddington, Eric; Richard Henderson > Subject: [Patch,AVR]: Fix PR46261 > > This fixes ICE on any source compiled with -mint8. >

[wwwdocs] Buildstat update for 4.6

2012-06-03 Thread Tom G. Christensen
Latest results for 4.6.x -tgc Testresults for 4.6.3: i386-pc-solaris2.8 i386-pc-solaris2.10 x86_64-unknown-linux-gnu Index: buildstat.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.6/buildstat.html,v retrieving revision 1.11

Re: [C++ Patch] for c++/51214

2012-06-03 Thread Fabien Chêne
2012/6/3 Jason Merrill : > On 05/24/2012 09:18 AM, Jason Merrill wrote: >> >> OK, thanks. > > > I notice you haven't checked the patch in yet, is there a problem? Not at all, just lack of time, so many problems/holidays to tackle at the moment... That is May month in France ;-) I'll be checking in

Re: [google] Add options to pattern match function name for hotness attributes

2012-06-03 Thread Xinliang David Li
Where is the patch -- the code review link is missing (in the original post). David On Sun, Jun 3, 2012 at 6:14 AM, Dehao Chen wrote: > Thank you guys for the comments, I'll update the patch to : > > 1. generalize the flag to enable other annotations such always_inline. > 2. change to use deferr

Re: [gcov] a few improvements

2012-06-03 Thread Xinliang David Li
On Sun, Jun 3, 2012 at 6:10 AM, Nathan Sidwell wrote: > On 06/03/12 05:51, Xinliang David Li wrote: >> >> On Fri, Dec 30, 2011 at 10:25 AM, Nathan Sidwell  wrote: >>> >>> I've committed this patch to fix and improve coverage reporting: >>> >>> 1) the time stamp local_tick will be -1 if the user ov

[PATCH] Option to build bare-metal ARM cross-compiler for arm-none-eabi target without libunwind

2012-06-03 Thread Larry Doolittle
This is an update to Fredrik Hederstierna's mail and patch from 12 Apr 2012. I think he missed one place where -fexceptions needs to be changed to -fno-exceptions. With the attached patch, two of us (a friend and I, one on Linux and one on Mac) can build Rockbox with gcc-4.7.0. I'll admit I have

Re: [C++ Patch] for c++/51214

2012-06-03 Thread Gabriel Dos Reis
On Sun, Jun 3, 2012 at 10:56 AM, Fabien Chêne wrote: > 2012/6/3 Jason Merrill : >> On 05/24/2012 09:18 AM, Jason Merrill wrote: >>> >>> OK, thanks. >> >> >> I notice you haven't checked the patch in yet, is there a problem? > > Not at all, just lack of time, so many problems/holidays to tackle at

Re: [gcov] a few improvements

2012-06-03 Thread Nathan Sidwell
On 06/03/12 17:16, Xinliang David Li wrote: Basically it makes it very difficult to rebuild the file with the profile data --- which makes problem triaging impossible. What is Can you explain this more -- what exactly are trying to do? Are you trying to rebuild multiple times with the same c

Re: [gcov] a few improvements

2012-06-03 Thread Xinliang David Li
On Sun, Jun 3, 2012 at 10:24 AM, Nathan Sidwell wrote: > On 06/03/12 17:16, Xinliang David Li wrote: > >> Basically it makes it very difficult to rebuild the file with the >> profile data --- which makes problem triaging impossible. What is > > > Can you explain this more -- what exactly are tryin

Re: [PATCH 2/2] Better system header location detection for built-in macro tokens

2012-06-03 Thread Paolo Carlini
On 06/03/2012 05:27 AM, Jason Merrill wrote: On 06/02/2012 12:40 PM, Paolo Carlini wrote: That said, the tricks we are playing with the global input_location vs the loc we are passing around still confuse me quite a lot. Actually any *assignment* to input_location makes me a bit more nervous t

Re: [PATCH 2/2] Better system header location detection for built-in macro tokens

2012-06-03 Thread Jason Merrill
On 06/03/2012 06:04 PM, Paolo Carlini wrote: if I apply to below, thus passing an actual loc to build_min_non_dep_loc, there is a diagnostic regression for the locations of libstdc++-v3/testsuite/20_util/bind_ref_neg.cc. If you see something obviously wrong somewhere, just let me know... Nope,

Re: [google] Add options to pattern match function name for hotness attributes

2012-06-03 Thread Andi Kleen
Dehao Chen writes: > Hi, > > This patch adds 4 flags to enable user to type in a list of name > patterns. Compiler will match the function name with the given > patterns, and add "hot", "cold", "likely_hot", "likely_cold" > attributes to function declaration. The static branch prediction > checks

[PATCH] Fix bug added to sparc 64-bit sub_ddmmss() implementation.

2012-06-03 Thread David Miller
Testing in glibc found a bug in one of my longlong.h sparc changes, fixed thusly and committed to master. libgcc/ * longlong.h [SPARC] (sub_ddmmss): Fix thinko in previous 64-bit change. --- libgcc/ChangeLog |5 + libgcc/longlong.h |2 +- 2 files changed, 6 insertio

Re: [google] Add options to pattern match function name for hotness attributes

2012-06-03 Thread Dehao Chen
On Mon, Jun 4, 2012 at 9:34 AM, Andi Kleen wrote: > Dehao Chen writes: > >> Hi, >> >> This patch adds 4 flags to enable user to type in a list of name >> patterns. Compiler will match the function name with the given >> patterns, and add "hot", "cold", "likely_hot", "likely_cold" >> attributes to

[line-map] simple oneliner that speeds up track-macro-expansion

2012-06-03 Thread Dimitrios Apostolou
Hello list, the attached simple patch is a small part of the fix in Bug #53525, but it is independant and offers a noticeable performance improvement for track-macro-expansion. No regressions tested on x86, besides memory usage going up some KB. I don't get why the max_column_hint was zeroed

libgo patch committed: Better SWIG interface for allocating memory

2012-06-03 Thread Ian Lance Taylor
This patch to libgo makes a better interface available to SWIG when C/C++ code wants to allocate memory that is garbage collected by Go. Previously if the only pointer to the memory were held in the C/C++ code, it was possible for the garbage collector to discard the memory before the C/C++ functio

Re: [google] Add options to pattern match function name for hotness attributes

2012-06-03 Thread Dehao Chen
I've updated the patch using deferred option. http://codereview.appspot.com/6281047 Thanks, Dehao The new patch: 2012-06-01 Dehao Chen * gcc/cgraph.c (cgraph_match_attribute_by_name): New function. (cgraph_node): Add attribute to function decl. * gcc/opts-global.c (a