C++ (RFC) PATCH fix for PR55076 and PR53921

2012-10-27 Thread Markus Trippelsdorf
The problem here is that we end up with an INDIRECT_REF TREE_CODE with a null TREE_TYPE in lvalue_kind. Is this possible at that point, or does it point to a deeper underlying problem? Bootstraped and tested on x86_64-pc-linux-gnu. 2012-10-27 Markus Trippelsdorf mar...@trippelsdorf.de

Re: [PATCH] PR c++/54955 - Fail to parse alignas expr at the beginning of a declaration

2012-10-27 Thread Dodji Seketeli
Jason Merrill ja...@redhat.com writes: On 10/26/2012 01:37 PM, Dodji Seketeli wrote: cp_next_tokens_can_be_std_attribute_p (cp_parser *parser) { - return cp_nth_tokens_can_be_std_attribute_p (parser, 1); + cp_token *token = cp_lexer_peek_token (parser-lexer); + + return (cxx_dialect

[PATCH, testsuite]: Add a testcase for PR34283, RA problem fixed by LRA

2012-10-27 Thread Uros Bizjak
Hello! Attached patch adds a testcase for RA problem that was fixed by LRA. 2012-10-27 Uros Bizjak ubiz...@gmail.com PR target/34283 * gcc.target/i386/pr34283.c: New test. Tested on x86_64-pc-linux-gnu, committed to mainline SVN. Uros. Index: gcc.target/i386/pr34283.c

[PATCH] PR c++/54466 - ICE with alias template which type-id is const qualified

2012-10-27 Thread Dodji Seketeli
Hello, Consider this short example: templatetypename T struct X { }; templatetypename T using Y = const XT; using Z = Yint; G++ crashes in lookup_class_template_1 while trying to build the alias template instantiation Yint. I think this is indirectly due to the fact

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

2012-10-27 Thread Manuel López-Ibáñez
On 25 April 2012 13:54, Joseph S. Myers jos...@codesourcery.com wrote: On Wed, 25 Apr 2012, Manuel López-Ibáñez wrote: On 25 April 2012 00:01, Joseph S. Myers jos...@codesourcery.com wrote: On Sun, 22 Apr 2012, Manuel López-Ibáñez wrote: Wshadow warns whenever any declaration shadows a

Re: *ping* [patch, fortran] Handle -Wextra, -fcompare-reals is implied with -Wextra

2012-10-27 Thread Thomas Koenig
Am 27.10.2012 01:41, schrieb Andreas Schwab: Thomas Koenig tkoe...@netcologne.de writes: Index: trans.c === --- trans.c (revision 192638) +++ trans.c (working copy) @@ -814,26 +814,23 @@ gfc_allocate_allocatable

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

2012-10-27 Thread Uros Bizjak
Hello! 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 attached RFC prototype patch that will trigger an ICE

Re: unordered map design modification

2012-10-27 Thread François Dumont
On 10/26/2012 10:41 AM, Paolo Carlini wrote: On 10/25/2012 10:15 PM, François Dumont wrote: Here is the patch to apply the same modification applied to unordered_set and unordered_multiset. It also use default implementation for unordered_set/unordered_multiset copy/move

[SH, committed] PR 55042

2012-10-27 Thread Oleg Endo
Hello, I've committed the obvious fix for PR 55042 as rev 192877. Cheers, Oleg gcc/ChangeLog: PR target/55042 * config/sh/sh.c (sh1_builtin_p): Comment out unused function. Index: gcc/config/sh/sh.c === ---

[Patch,AVR,applied]: Fix PR55034

2012-10-27 Thread Georg-Johann Lay
http://gcc.gnu.org/viewcvs?view=revisionrevision=192878 Applied as obvios. Johann PR target/55034 * config/avr/avr.c (avr_out_lpm): Remove unused regno_dest.

[wwwdocs,Java] Remove FAQ entry releated to 12 year old problem

2012-10-27 Thread Gerald Pfeifer
I doubt this has been an FAQ in the last decade. :-) Applied. Gerald 2012-10-27 Gerald Pfeifer ger...@pfeifer.com * faq.html (My program seems to hang): Remove. Index: faq.html === RCS file:

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

2012-10-27 Thread Joern Rennecke
Bootstrapped (c,c++) on i686-pc-linux-gnu. Committed as obvious. 2012-10-27 Joern Rennecke joern.renne...@embecosm.com * lra-assigns.c (find_hard_regno_for): Fix use of WORDS_BIG_ENDIAN. Index: lra-assigns.c === ---

Re: [patch, fortran] PR 54833 Don't wrap calls to free(a) in if (a != NULL)

2012-10-27 Thread Andreas Schwab
Thomas Koenig tkoe...@netcologne.de writes: PR fortran/54833 * trans.c (gfc_call_free): Do not wrap the call to __builtin_free in check for NULL. (gfc_deallocate_with_status): For automatic deallocation without status for non-coarrays, don't wrap call

Re: PR c/53063 Handle Wformat with LangEnabledBy

2012-10-27 Thread Manuel López-Ibáñez
On 19 October 2012 18:17, Joseph S. Myers jos...@codesourcery.com wrote: On Wed, 17 Oct 2012, Manuel López-Ibáñez wrote: documentation but I can also implement -Wformat=0 being an alias for -Wno-format and -Wformat=1 an alias for -Wformat and simply reject -Wno-format=. I think that's

PR c/53063 more LangEnabledBy conversions.

2012-10-27 Thread Manuel López-Ibáñez
More mostly easy conversions to LangEnabledBy. Regtested on x86_64-linux-gnu. OK? 2012-10-27 Manuel López-Ibáñez m...@gcc.gnu.org PR c/53063 c-family/ * c.opt(Warray-bounds,Wdelete-non-virtual-dtor,Wenum-compare, Wmain,Woverlength-strings,

Re: [patch, fortran] PR 54833 Don't wrap calls to free(a) in if (a != NULL)

2012-10-27 Thread Thomas Koenig
Hi Andreas, Thomas Koenig tkoe...@netcologne.de writes: PR fortran/54833 * trans.c (gfc_call_free): Do not wrap the call to __builtin_free in check for NULL. (gfc_deallocate_with_status): For automatic deallocation without status for non-coarrays,

Re: [patch, fortran] PR 54833 Don't wrap calls to free(a) in if (a != NULL)

2012-10-27 Thread Andreas Schwab
Thomas Koenig tkoe...@netcologne.de writes: Does not happen on the architectures that I used, and I do not have access to that architecture. A cross compiler is enough. Does a (rougly) equivalent C case compile? How does a (rougly) equivalent C case look like? Andreas. -- Andreas Schwab,

Re: [patch, fortran] PR 54833 Don't wrap calls to free(a) in if (a != NULL)

2012-10-27 Thread Steven Bosscher
On Sat, Oct 27, 2012 at 7:41 PM, Andreas Schwab sch...@linux-m68k.org wrote: How does a (rougly) equivalent C case look like? Usually something can be constructed from the -fdump-tree-gimple dump. Ciao! Steven

Re: [PATCH] Fix debug info for expr and jump stmt

2012-10-27 Thread Richard Biener
On Sat, Oct 27, 2012 at 12:53 AM, Dehao Chen de...@google.com wrote: Hi, I've updated the patch: 1. abandon the changes in cfgexpand.c 2. set the block for trees when lowering gimple stmt. 3. add a unittest. Index: gcc/gimple-low.c

Re: [PATCH] Fix debug info for expr and jump stmt

2012-10-27 Thread Dehao Chen
On Sat, Oct 27, 2012 at 11:07 AM, Richard Biener richard.guent...@gmail.com wrote: On Sat, Oct 27, 2012 at 12:53 AM, Dehao Chen de...@google.com wrote: Hi, I've updated the patch: 1. abandon the changes in cfgexpand.c 2. set the block for trees when lowering gimple stmt. 3. add a unittest.

Re: [patch, fortran] PR 54833 Don't wrap calls to free(a) in if (a != NULL)

2012-10-27 Thread Andreas Schwab
Steven Bosscher stevenb@gmail.com writes: On Sat, Oct 27, 2012 at 7:41 PM, Andreas Schwab sch...@linux-m68k.org wrote: How does a (rougly) equivalent C case look like? Usually something can be constructed from the -fdump-tree-gimple dump. Doesn't look like. Andreas. -- Andreas Schwab,

[wwwdocs,Java] point java/done.html to sourceware.org

2012-10-27 Thread Gerald Pfeifer
Applied, based an another reminder of these two no longer being synonymous. Gerald 2012-10-27 Gerald Pfeifer ger...@pfeifer.com * done.html: Adjust URL for rhug. Index: done.html === RCS file:

Re: [wwwdocs,Java] point java/done.html to sourceware.org

2012-10-27 Thread Gerald Pfeifer
On Sat, 27 Oct 2012, Gerald Pfeifer wrote: 2012-10-27 Gerald Pfeifer ger...@pfeifer.com * done.html: Adjust URL for rhug. And here is a fix for a long standing issue... Gerald Index: done.html === RCS file:

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

2012-10-27 Thread Steven Bosscher
On Sat, Oct 13, 2012 at 11:12 PM, Vladimir Makarov wrote: On 12-10-13 11:37 AM, Steven Bosscher wrote: LIVE is not used on purpose. I added LIVE usage to the old RA about 10 years ago (it was before DF-infrastructure). Everything was ok until, somebody reported compiler crash on semantically

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

2012-10-27 Thread Steven Bosscher
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. * ira.c (ira): Remove DF_LIVE if the problem is in the stack. (do_reload): Add it back at the end.

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

2012-10-27 Thread H.J. Lu
On Sat, Oct 27, 2012 at 5:12 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! 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

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

2012-10-27 Thread H.J. Lu
On Sat, Oct 27, 2012 at 4:36 PM, H.J. Lu hjl.to...@gmail.com wrote: On Sat, Oct 27, 2012 at 5:12 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! 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

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

2012-10-27 Thread Joern Rennecke
With the lra branch merged, I had to update the patch, because of new uses of HAVE_ATTTR_enabled. While looking at the conflicts, I also found that it was often hard to tell if macros were used for performance reasons, or just because there was no other interface available. lra itself is

Re: [PATCH] Fix libbacktrace on 32-bit sparc

2012-10-27 Thread Ian Lance Taylor
On Fri, Oct 26, 2012 at 9:27 PM, David Miller da...@davemloft.net wrote: I'm getting a SIGBUS on every backtrace libbacktrace generates on 32-bit sparc builds. The crashes usually happen in add_function_range(), where 'p' is not 8-byte aligned. It seems that the vector code doesn't take

Re: [PATCH] Fix libbacktrace on 32-bit sparc

2012-10-27 Thread David Miller
From: Ian Lance Taylor i...@google.com Date: Sat, 27 Oct 2012 21:06:59 -0700 The struct used by add_function_range is struct function_addrs { uint64_t low; uint64_t high; struct function *function; }; So on a 32-bit system, this should have a size of 20 if uint64_t requires

Re: [PATCH] Fix libbacktrace on 32-bit sparc

2012-10-27 Thread David Miller
From: David Miller da...@davemloft.net Date: Sun, 28 Oct 2012 00:31:27 -0400 (EDT) The size is 24, and my patch definitely makes the crashes go away. It seems like a vector is being used for a mixed set of objects. I'll try to figure out how that is happening. Ok, the problem seems to have