Re: Ping / update: RFA: replace #ifdef with if/#if for HAVE_ATTR_*

2012-10-28 Thread Andreas Schwab
Joern Rennecke writes: > Index: gcc/doc/tm.texi > === > --- gcc/doc/tm.texi (revision 192840) > +++ gcc/doc/tm.texi (working copy) > @@ -11333,3 +11333,11 @@ @deftypefn {Target Hook} {unsigned HOST_ > @deftypevr {Target Hook} {u

Move @end to own line

2012-10-28 Thread Andreas Schwab
The texinfo manual says that @end should always be put on a line of its own. Tested with make info dvi pdf html and checked in as obvious. (None of the converters have a problem with this so the output will be the same.) Andreas. * doc/tm.texi.in (Misc): Add newline before @end.

Re: [RFC PATCH, i386]: Remove peephole2s for (subreg (operator (...)(...))) RTXes

2012-10-28 Thread Uros Bizjak
On Sun, Oct 28, 2012 at 2:37 AM, H.J. Lu wrote: >>> As suggested by Richard S. [1], after the patch that converts subreg:M >>> (op:N (...)(...)) to op:M (subreg:M (...) subreg:M (...)), we can >>> remove several peephole2 patterns that handle subregs of PLUS, MINUS >>> and MULT operators. I have

Re: [RFC PATCH, i386]: Remove peephole2s for (subreg (operator (...)(...))) RTXes

2012-10-28 Thread Uros Bizjak
On Sun, Oct 28, 2012 at 9:57 AM, Uros Bizjak wrote: > On Sun, Oct 28, 2012 at 2:37 AM, H.J. Lu wrote: > As suggested by Richard S. [1], after the patch that converts subreg:M (op:N (...)(...)) to op:M (subreg:M (...) subreg:M (...)), we can remove several peephole2 patterns that ha

Fix use of @item vs. @itemx inside @table

2012-10-28 Thread Andreas Schwab
Tested with make info dvi pdf html and checked in as obvious. Andreas. * doc/cppopts.texi: Fix use of @item vs. @itemx inside @table. * doc/extend.texi: Likewise. * doc/generic.texi: Likewise. * doc/invoke.texi: Likewise. * doc/md.texi: Likewise. *

Make inliner to take cgraph SCCs into account

2012-10-28 Thread Jan Hubicka
Hi, this patch implements simple hints on strongly connected components to inliner. It increases badness of inlining functions within the same scc component, since this non-trivial recursive inlining is not win very often and it may blow up stack frames a lot. It also increases the entry points

Re: [m68k] Fix option handling for -m68020-40 and -m68020-60

2012-10-28 Thread Gunther Nikl
Andreas Schwab wrote: > Gunther Nikl writes: > >> The patch should be installed on trunk and on the 4.7 branch. > > Thanks, done. The 4.7 branch required some adjustment, since it's not > compiled as C++. Right. Maybe a better solution would have been then to only change m68k.c/m68k_option_ove

Re: Make inliner to take cgraph SCCs into account

2012-10-28 Thread Marc Glisse
On Sun, 28 Oct 2012, Jan Hubicka wrote: Bootstrapped/regtested x86_64-linux. Honza * ipa-inline.c (edge_badness): Reduce precision; use scc hints. (inline_small_functions): Fix dumps; update all callees after inlining. * ipa-inline.h (INLINE_HINT_in_scc, INLINE_HINT_sam

[C++ Patch] PR 54526 (again)

2012-10-28 Thread Paolo Carlini
Hi, as pointed out in the audit trail, my first patch for this C++11 parsing issue was misguided: I changed cp_parser_template_id but in fact C++11 wants new special *lexing* rules, which must be active outside templates too, as the additional test (and the old manded one) shows. Thus the belo

[wwwdocs] Rewrite glibc list archive references

2012-10-28 Thread Gerald Pfeifer
...from sources.redhat.com to sourceware.org. With that, there are only a few references left in java/ . Gerald Index: c99status.html === RCS file: /cvs/gcc/wwwdocs/htdocs/c99status.html,v retrieving revision 1.58 diff -u -3 -p -r1.

Re: Make inliner to take cgraph SCCs into account

2012-10-28 Thread Jan Hubicka
> On Sun, 28 Oct 2012, Jan Hubicka wrote: > > >Bootstrapped/regtested x86_64-linux. > > > >Honza > > > > * ipa-inline.c (edge_badness): Reduce precision; use scc hints. > > (inline_small_functions): Fix dumps; update all callees after inlining. > > * ipa-inline.h (INLINE_HINT_in_scc, I

Re: Make inliner to take cgraph SCCs into account

2012-10-28 Thread Jan Hubicka
> > On Sun, 28 Oct 2012, Jan Hubicka wrote: > > > > >Bootstrapped/regtested x86_64-linux. > > > > > >Honza > > > > > > * ipa-inline.c (edge_badness): Reduce precision; use scc hints. > > > (inline_small_functions): Fix dumps; update all callees after inlining. > > > * ipa-inline.h (INLINE_HI

PR libstdc++/55041 update python printers

2012-10-28 Thread Jonathan Wakely
This fixes part of PR 55041 by updating the hash table printers to account for Francois's recent changes. PR libstdc++/55041 * python/libstdcxx/v6/printers.py (Tr1UnorderedMapPrinter): Update to handle hashtable as member of unordered_map not base class. (Tr1Unorder

real_zerop for vectors

2012-10-28 Thread Marc Glisse
Hello, this patch lets some predicates on floating point constants answer true for vectors, so optimizations are applied. Tested bootstrap + testsuite (default languages). 2012-10-29 Marc Glisse PR middle-end/55027 gcc/ * tree.c (real_zerop, real_onep, real_twop, real_min

Ping: [PATCH] Install error handler for out-of-memory when using STL containers

2012-10-28 Thread Tobias Schlüter
Ping. This issue stands in the way of a very simple solution of PR fortran/51727. I've re-attached the patch for your convenience. On 15 Oct 2012 at 22:51:05 +0200 Tobias Schlüter wrote: The attached patch adds out-of-memory diagnostics for code using STL containers by using set_new_handler

Re: real_zerop for vectors

2012-10-28 Thread Paolo Carlini
On 10/28/2012 04:14 PM, Marc Glisse wrote: Hello, this patch lets some predicates on floating point constants answer true for vectors, so optimizations are applied. Great. I wonder how are we doing lately in terms of function pointer inlining?! If the current optimizers can already able to s

*ping* Re: [Patch, Fortran] Fix some libgfortran issues found by coverity

2012-10-28 Thread Tobias Burnus
* ping * On 16.10.2012 23:18, Tobias Burnus wrote: In the Bessel-function algorithm, there was the useless code: ret->base_addr = ret->base_addr; And in all files which included ifunction.m4 is such code: iall_i4 (gfc_array_i4 * const restrict retarray, ... if (len <= 0) *

*ping* Re: [Patch, Fortran] PR54958 - Allow ac-implied-do and data-implied-do with INTENT(IN)

2012-10-28 Thread Tobias Burnus
* ping * On 19.10.2012 18:54, Tobias Burnus wrote: gfortran's INTENT(IN) check was too strict for "do" variables. While a variable in the normal do-stmt and in an io-implied-do is in the scope and, hence, the variable may not be modified for a nonpointer intent(in) variable. However, ac-impl

Re: real_zerop for vectors

2012-10-28 Thread Paolo Carlini
On 10/28/2012 04:46 PM, Paolo Carlini wrote: On 10/28/2012 04:14 PM, Marc Glisse wrote: Hello, this patch lets some predicates on floating point constants answer true for vectors, so optimizations are applied. Great. I wonder how are we doing lately in terms of function pointer inlining?! I

Re: real_zerop for vectors

2012-10-28 Thread Marc Glisse
On Sun, 28 Oct 2012, Paolo Carlini wrote: On 10/28/2012 04:46 PM, Paolo Carlini wrote: On 10/28/2012 04:14 PM, Marc Glisse wrote: Hello, this patch lets some predicates on floating point constants answer true for vectors, so optimizations are applied. Great. I wonder how are we doing latel

Re: *ping* Re: [Patch, Fortran] Fix some libgfortran issues found by coverity

2012-10-28 Thread Thomas Koenig
Hi Tobias, * ping * On 16.10.2012 23:18, Tobias Burnus wrote: In the Bessel-function algorithm, there was the useless code: ret->base_addr = ret->base_addr; The patch is OK. Thanks a lot! Thomas

Re: *ping* Re: [Patch, Fortran] PR54958 - Allow ac-implied-do and data-implied-do with INTENT(IN)

2012-10-28 Thread Thomas Koenig
Hi Tobias, * ping * This is OK. Thanks for the patch! Thomas

Re: real_zerop for vectors

2012-10-28 Thread Marc Glisse
On Sun, 28 Oct 2012, Marc Glisse wrote: [there are 4 real_*p that only differ by 1 character] It is true that we could have a single function in tree.c: bool real_intcstp (const_tree, int); The helper function could even take a REAL_VALUE_TYPE as second argument, so the non-inline part doesn

Re: real_zerop for vectors

2012-10-28 Thread Paolo Carlini
Hi, On 10/28/2012 05:51 PM, Marc Glisse wrote: On Sun, 28 Oct 2012, Marc Glisse wrote: [there are 4 real_*p that only differ by 1 character] It is true that we could have a single function in tree.c: bool real_intcstp (const_tree, int); The helper function could even take a REAL_VALUE_TYPE

Re: real_zerop for vectors

2012-10-28 Thread Marc Glisse
On Sun, 28 Oct 2012, Paolo Carlini wrote: I was writing something like the below. I would move the one-liners (real_zerop, etc) to tree.h, but that looks good, yes. -- Marc Glisse

Re: Ping / update: RFA: replace #ifdef with if/#if for HAVE_ATTR_*

2012-10-28 Thread Joern Rennecke
Quoting Andreas Schwab : +These flags are automatically generated; you should not override them in tm.c: Typo: s/:$/./, also @file{tm.c}. Thanks. I have re-bootstrapped the amended patch on i686-pc-linux-gnu. 2012-10-28 Joern Rennecke * doc/md.texi (Defining Attributes): Docu

RFA: hookize ADJUST_INSN_LENGTH (Was: RFA: Add lock_lenth attribute to support the ARC port)

2012-10-28 Thread Joern Rennecke
Quoting Richard Biener : Thus, you can allow the length to vary downwards as well as upwards across iterations with suitable definitions of the @code{length} attribute and/or @code{ADJUST_INSN_LENGTH}. Care has to be taken that this does not lead to infinite loops. I don't see that you can sh

Re: real_zerop for vectors

2012-10-28 Thread Paolo Carlini
On 10/28/2012 06:06 PM, Marc Glisse wrote: On Sun, 28 Oct 2012, Paolo Carlini wrote: I was writing something like the below. I would move the one-liners (real_zerop, etc) to tree.h, but that looks good, yes. Ah great. But please, don't wait on me, I'm in the middle of too many other things,

Re: [PATCH] Fix CDDCE miscompilation (PR tree-optimization/55018)

2012-10-28 Thread Steven Bosscher
On Mon, Oct 22, 2012 at 11:09 PM, Jakub Jelinek wrote: > On Mon, Oct 22, 2012 at 10:51:43PM +0200, Steven Bosscher wrote: > Wouldn't it be way cheaper to just export dfs_find_deadend from cfganal.c > and call it in calc_dfs_tree on each unconnected bb? > I.e. (untested with the exception of the tes

patch to remove temporary clobbers in LRA

2012-10-28 Thread Vladimir Makarov
Working on a problem where var-tracking started to spend more time after LRA is switched on, I found that LRA creates significant # clobbers for x86 code (I saw 20% of all insns). Clobbers are created for correct live range analysis in LRA in cases when matching pseudos of *different modes* s

Re: patch to remove temporary clobbers in LRA

2012-10-28 Thread Steven Bosscher
On Sun, Oct 28, 2012 at 9:06 PM, Vladimir Makarov wrote: > Unfortunately, it does not solve the problem I mentioned. Now, LRA > generates approximately the same # of insns and debug_insn. Moreover, two > functions (canonicalize_values_star and set_slot_part) in var-tracking > spending most of t

Re: [PR38711] Use DF_LIVE in IRA if it available (for -O2 and higher)

2012-10-28 Thread Vladimir Makarov
On 12-10-27 6:07 PM, Steven Bosscher wrote: On Sat, Oct 27, 2012 at 11:49 PM, Steven Bosscher wrote: So I'm going to revert this patch. Or actually better, this lighter-weight patch. Will commit after the usual testing. I returned to this problem several times and every time I failed to use

Re: committed: Fix WORDS_BIG_ENDIAN use in lra-assign.c:find_hard_regno_for

2012-10-28 Thread Vladimir Makarov
On 12-10-27 11:34 AM, Joern Rennecke wrote: Bootstrapped (c,c++) on i686-pc-linux-gnu. Committed as obvious. Thanks, Joern.

[wwwdocs] gcc-4.8/changes.html markup consistency

2012-10-28 Thread Gerald Pfeifer
Installed. Gerald Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.8/changes.html,v retrieving revision 1.48 diff -u -3 -p -r1.48 changes.html --- changes.html21 Oct 2012 11:01:37 - 1.48 +++ changes.ht

[patch, RFC] PR 30146, warning/errors for potentially changing values in DO loops

2012-10-28 Thread Thomas Koenig
Hello world, the attached patch, which is not in its final stage, implements some warnings for index variables of DO loops. For the following situations, errors/warnings are issued when an index loop variable is passed as an actual argument: - If the dummy argument has INTENT(OUT). I think an er

Re: [wwwdocs] gcc-4.8/changes.html markup consistency

2012-10-28 Thread Andreas Schwab
Gerald Pfeifer writes: > + G++ now supports a >-std=c++1y option for experimentation Fixed the typo. Andreas. --- changes.html.~1.49.~2012-10-28 22:11:57.0 +0100 +++ changes.html2012-10-28 22:12:37.0 +0100 @@ -133,7 +133,7 @@ B b(42); // OK - G++ now s

[wwwdocs,Java] Some build-snapshot.html updates

2012-10-28 Thread Gerald Pfeifer
While working on the sources->sourceware transition, I noticed some not directly related items and went ahead and addressed them. Committed. 2012-10-28 Gerald Pfeifer * build-snapshot.html: Refer to GNU/Linux, simplify link to installation documentation, remove reference to a

Re: [wwwdocs] gcc-4.8/changes.html markup consistency

2012-10-28 Thread Gerald Pfeifer
On Sun, 28 Oct 2012, Andreas Schwab wrote: >> + G++ now supports a >-std=c++1y option for experimentation > Fixed the typo. Thanks, Andreas! The validator did not complain -- which shows that a pair of human eyes _is_ valuable. :-) Gerald

Re: [wwwdocs] changes.html update

2012-10-28 Thread Gerald Pfeifer
On Wed, 24 Oct 2012, Jan Hubicka wrote: > few things I think are worth to be mentioned in changes.html. Nice! > Index: changes.html > === > + LTO partitioning has been rewritten for better reliability > + and maintanibilit

Re: [AARCH64] Update maintainers file

2012-10-28 Thread Gerald Pfeifer
On Tue, 23 Oct 2012, Richard Earnshaw wrote: > On 23/10/12 23:36, Steven Bosscher wrote: >> Will you also add an announcement of this to the news page (home page) >> and to gcc-4.8/changes.html? > I'm sure we can... :-) You have to. :-) Please. Gerald

../.././gcc/system.h:28:20: fatal error: stdarg.h: No such file or directory

2012-10-28 Thread Bruce Korb
Say, what? Anyone know what this is about? cp ../.././gcc/../fixincludes/README-fixinc include-fixed/README chmod a+r include-fixed/README echo timestamp > stmp-int-hdrs TARGET_CPU_DEFAULT="" \ HEADERS="auto-host.h ansidecl.h" DEFINES="USED_FOR_TARGET " \ /bin/sh ../.././gcc/mkconfig.sh tconfig.h

Re: [patch] fix outdated path in cpp.texi

2012-10-28 Thread Gerald Pfeifer
On Sun, 21 Oct 2012, Jonathan Wakely wrote: >> I was going to say "Ack", since it's a doc patch, but somehow my >> own tests on various platforms (FreeBSD, GNU/Linux,...) did not >> confirm /usr/include/c++ in the search path. >> >> Or do you mean that it's at the root of some search paths, that >>

[Patch] Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF from

2012-10-28 Thread JonY
Hi, Workaround now exists for trunk mingw-w64 headers. Kai approved over IRC, so anybody with commit rights please push. ChangeLog 2012-10-29 Jonathan Yong * config/os/mingw32-w64/os_defines.h: Remove _GLIBCXX_HAVE_BROKEN_VSWPRINTF as no longer required. Index: libstdc++-v

Make inliner to predict &this->field to be optimized out

2012-10-28 Thread Jan Hubicka
Hi, this patch makes optimizer to predict &this->field to be optimized out. The main motivation for this is to make destructors that only calls destructor of inner type to be inlined, even in the cold regions. Bootstrapped/regtested x86_64-linux. Comitted. 2012-10-28 Jan Hubicka * ipa

Re: [patch] fix outdated path in cpp.texi

2012-10-28 Thread Jonathan Wakely
On 28 October 2012 22:21, Gerald Pfeifer wrote: > On Sun, 21 Oct 2012, Jonathan Wakely wrote: >>> I was going to say "Ack", since it's a doc patch, but somehow my >>> own tests on various platforms (FreeBSD, GNU/Linux,...) did not >>> confirm /usr/include/c++ in the search path. >>> >>> Or do you m

Re: [patch] fix outdated path in cpp.texi

2012-10-28 Thread Jonathan Wakely
On 29 October 2012 00:08, Jonathan Wakely wrote: > Yes, libdir/gcc/target/version/../../../../include/c++/version would > be more accurate, because libdir is not necessarily /usr > > I'll come up with something better ... I've committed this to the trunk. commit 258a736d80af598109fae44e55b0577

patch to fix PR55106

2012-10-28 Thread Vladimir Makarov
The following patch fixes PR55106. A value in GENERAL_REGS is inherited into a move with destination pseudo of SSE_REGS. It results into secondary move for which inheritance is tried again an again. It means cycling LRA passes. The patch was successfully bootstrapped on x86/x86-64. C

Re: patch to fix PR55106

2012-10-28 Thread H.J. Lu
On Sun, Oct 28, 2012 at 5:43 PM, Vladimir Makarov wrote: > The following patch fixes PR55106. A value in GENERAL_REGS is inherited > into a move with destination pseudo of SSE_REGS. It results into secondary > move for which inheritance is tried again an again. It means cycling LRA > passes.

Re: PR c/53066 Wshadow should not warn for shadowing an extern function

2012-10-28 Thread Joseph S. Myers
The code changes seem fine, but I don't think a testcase should depend on the details of what system strings.h declares (or even that it exists) like that. If you want a system header declaration, add a header to the testsuite that uses #pragma GCC system_header to be sure it's handled as such

Trailing white spaces in LRA codes

2012-10-28 Thread H.J. Lu
Hi Vladimir, There are many trialling white spaces in LRA codes. This patch removes them. H.J. --- diff --git a/gcc/lra-assigns.c b/gcc/lra-assigns.c index 1f89338..c558a4d 100644 --- a/gcc/lra-assigns.c +++ b/gcc/lra-assigns.c @@ -149,7 +149,7 @@ init_regno_assign_info (void) { int i, regno

[PATCH GCC]Improving register pressure directed hoist

2012-10-28 Thread Bin Cheng
Hi, This patch improves register pressure directed hoist by accurately monitoring the change of register pressure during code hoisting and use this information to feedback further code hoisting. To know how register pressure is affected when hoisting an expression(x+y) through a basic block, the pa