Re: libstdc++ PATCH to use __cplusplus rather than __GXX_EXPERIMENTAL_CXX0X__

2012-11-10 Thread Paolo Carlini
Ed Smith-Rowland 3dw...@verizon.net ha scritto: Paolo? ... carte blanche to Jason! (but I have a little lexer tweak pending... ;) Paolo

Re: [asan] Patch - fix an ICE in asan.c

2012-11-10 Thread Jakub Jelinek
On Fri, Nov 09, 2012 at 09:36:53PM +0100, Tobias Burnus wrote: * I still have to do an all-language bootstrap and regtesting, though the latter is probably pointless as there is currently not a single -fasan test case. --- gcc/asan.c.orig 2012-11-09 21:26:26.0 +0100 +++ gcc/asan.c

[PATCH, i386]: Fix PR 55247, ICE: Max. number of generated reload insns per insn is achieved

2012-11-10 Thread Uros Bizjak
Hello! Attached patch disparages riF-o alternative of *movti_internal_rex64 insn, as described by Vlad in comment #2 [1] The core of the problem however is, that gcc is unable to detect zero-extended address as offsetable. H.J. will propose a patch for this [2]. 2012-11-10 Vladimir Makarov

Re: [asan] Patch - fix an ICE in asan.c

2012-11-10 Thread Tobias Burnus
Jakub Jelinek wrote: --- gcc/asan.c.orig 2012-11-09 21:26:26.0 +0100 +++ gcc/asan.c 2012-11-09 21:26:00.0 +0100 @@ -1362,6 +1362,8 @@ transform_statements (void) instrument_assignment (i); else if (is_gimple_call (s)) maybe_instrument_call

*ping* [patch, fortran] PR 30146, errors for INTENT(OUT) and INTENT(INOUT) for DO loop variables

2012-11-10 Thread Thomas Koenig
I wrote: after the dicsussion on c.l.f, it become clear that passing a DO loop variable to an INTENT(OUT) or INTENT(INOUT) dummy argument is an error. The attached patch throws an error for both cases. I chose to issue the errors as a front-end pass because we cannot check for formal arguments

Re: libstdc++ PATCH to use __cplusplus rather than __GXX_EXPERIMENTAL_CXX0X__

2012-11-10 Thread Jonathan Wakely
On 9 November 2012 22:09, Jason Merrill wrote: Now that G++ uses the value of __cplusplus specified by the standard, we don't need the other macro anymore. OK for trunk, or should I save it for the next stage 1? Jason I'd been thinking about suggesting that change just the other day - do

[PATCH} AIX Testsuite cleanup

2012-11-10 Thread David Edelsohn
A few more testsuite fixes to address failures on AIX. The only really interesting one is g++.dg/other/anon5.C where Undefined is capitalized in the AIX error message. Thanks, David * c-c++-common/scal-to-vec2.c: Ignore non-standard ABI message. *

Re: PATCH: Handle ZERO_EXTEND offsettable address

2012-11-10 Thread Paolo Bonzini
Il 10/11/2012 07:44, H.J. Lu ha scritto: Hi, In (insn 19 17 20 2 (set (reg:TI 85 [ *_15 ]) (mem:TI (zero_extend:DI (reg:SI 82)) [0 *_15+0 S16 A32])) x.i:29 61 {*movti_internal_rex64} (expr_list:REG_DEAD (reg:SI 82) (expr_list:REG_EQUIV (mem/c:TI (plus:DI (reg/f:DI

Re: [PATCH] Fix combined tree for LTO

2012-11-10 Thread Paolo Bonzini
Il 10/11/2012 05:30, Andrew Pinski ha scritto: Hi, The problem here is that set PLUGIN_LD_SUFFIX to ld-new which is not the final installed binary name. This patch fixes the problem by changing if we got ld-new to just ld. Note this issue has been around since 4.6 but not many people test

Re: [asan] Patch - fix an ICE in asan.c

2012-11-10 Thread Tobias Burnus
Tobias Burnus wrote: So untested: Thanks for the patch! It fixed the problem half way: It fixes the second issue I had (fail10.ii, http://gcc.gnu.org/ml/gcc-patches/2012-11/msg00791.html ). However, it didn't fix the original problem: As the call for strlen directly returns, it never

Re: [PATCH] PR tree-optimization/55079: Don't remove all exits of a loop during loop unroll

2012-11-10 Thread Siddhesh Poyarekar
On Fri, 9 Nov 2012 22:51:45 +0530, Siddhesh wrote: I had reckoned that the behaviour could be reverted to what was before while I figure out a way to get the warning in place for both cases, i.e. with tree-vrp (where max_loop_iterations now causes the loop to be folded away in -O2) and this

Fix fallout of patch for PR rtl-optimization/54315

2012-11-10 Thread Eric Botcazou
In the patch I installed for PR rtl-optimization/54315: http://gcc.gnu.org/ml/gcc-patches/2012-10/msg00745.html the special code dealing with BLKmode in registers at the beginning of store_field is disabled for CALL_EXPR: /* If we are storing into an unaligned field of an aligned union that

Invitation to use Google Talk

2012-11-10 Thread Google Talk
--- You've been invited by Claudiu Zissulescu to use Google Talk. If you already have a Google account, login to Gmail and accept this chat invitation:

Re: [1/8] Handle TRUNCATE in make_extraction

2012-11-10 Thread Eric Botcazou
Tested as described in the covering note. OK to install? Richard gcc/ * combine.c (make_extraction): Handle TRUNCATEd INNERs. OK, thanks. -- Eric Botcazou

Re: [2/8] Add adjust_bitfield_address_size

2012-11-10 Thread Eric Botcazou
Tested as described in the covering note. OK to install? Richard gcc/ * expr.h (adjust_address_1): Add a size parameter. (adjust_address, adjust_address_nv, adjust_bitfield_address) (adjust_bitfield_address_nv): Adjust accordingly.

Re: [3/8] Add narrow_bit_field_mem

2012-11-10 Thread Eric Botcazou
Tested as described in the covering note. OK to install? Richard gcc/ * expmed.c (narrow_bit_field_mem): New function. (store_bit_field_using_insv, store_bit_field_1, store_fixed_bit_field) (extract_bit_field_1): Use it. This looks better now, thanks. -- Eric

Re: [PATCH] Fix combined tree for LTO

2012-11-10 Thread Andrew Pinski
On Sat, Nov 10, 2012 at 6:46 AM, Paolo Bonzini bonz...@gnu.org wrote: Il 10/11/2012 05:30, Andrew Pinski ha scritto: Hi, The problem here is that set PLUGIN_LD_SUFFIX to ld-new which is not the final installed binary name. This patch fixes the problem by changing if we got ld-new to just

Re: [PATCH, i386]: Fix PR 55247, ICE: Max. number of generated reload insns per insn is achieved

2012-11-10 Thread H.J. Lu
On Sat, Nov 10, 2012 at 3:43 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! Attached patch disparages riF-o alternative of *movti_internal_rex64 insn, as described by Vlad in comment #2 [1] The core of the problem however is, that gcc is unable to detect zero-extended address as

libgo patch committed: Fix bug comparing struct field types

2012-11-10 Thread Ian Lance Taylor
This patch fixes a bug comparing struct field types in the reflect package. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch. Ian diff -r 8b1f2a35ded1 libgo/go/reflect/type.go --- a/libgo/go/reflect/type.go Tue Nov 06 10:44:51 2012 -0800 +++

Re: PATCH: Handle ZERO_EXTEND offsettable address

2012-11-10 Thread H.J. Lu
On Sat, Nov 10, 2012 at 10:38:55AM -0800, H.J. Lu wrote: On Sat, Nov 10, 2012 at 6:41 AM, Paolo Bonzini bonz...@gnu.org wrote: Il 10/11/2012 07:44, H.J. Lu ha scritto: Hi, In (insn 19 17 20 2 (set (reg:TI 85 [ *_15 ]) (mem:TI (zero_extend:DI (reg:SI 82)) [0 *_15+0 S16 A32]))

Re: *ping* [patch, fortran] PR 30146, errors for INTENT(OUT) and INTENT(INOUT) for DO loop variables

2012-11-10 Thread Steven Bosscher
On Sat, Nov 10, 2012 at 3:00 PM, Thomas Koenig wrote: I wrote: after the dicsussion on c.l.f, it become clear that passing a DO loop variable to an INTENT(OUT) or INTENT(INOUT) dummy argument is an error. The attached patch throws an error for both cases. But should we really isse an error

[patch] PR55263

2012-11-10 Thread Steven Bosscher
Hello, This is another ICE in pre_and_rev_post_order_compute, called from alias.c after register allocation. The problem is that purge_all_dead_edges can make basic blocks unreachable. Before my patch of r190602, alias.c handled unreachable blocks (resulting in missed disambiguations). Now that

[doc] extend.texi copy-editing, 2/N (which/that usage)

2012-11-10 Thread Sandra Loosemore
This patch continues my series of copy-edits to the GCC user documentation. Here I've fixed a number of problems in extend.texi with confusion between which and that, as I previously did for invoke.texi. Committed as obvious since there are no changes to content, just grammar. -Sandra

Re: RFC: PATCH to add abi_tag attribute

2012-11-10 Thread Jason Merrill
The demangler change was handling the tags in the wrong place; I'm writing them out with unqualified names, so the demangler should expect them in the same place. Tested x86_64-pc-linux-gnu, applied to trunk. commit 75eef303e5494f27a6d9bbef68aaf3200978a8f1 Author: Jason Merrill

libstdc++ PATCH to add abi tag to complex::real/imag

2012-11-10 Thread Jason Merrill
As mentioned in http://gcc.gnu.org/wiki/Cxx11AbiCompatibility, C++11 changes the return type of complex::real and imag, leading to a binary incompatibility between C++98 and C++11 code if the functions are used without inlining. This patch adds an ABI tag to the C++11 variants so that they

[doc] extend.texi copy-editing, 3/N (hyphenated phrases)

2012-11-10 Thread Sandra Loosemore
I've checked in this patch to fix various problems with hyphenated phrases in extend.texi. This exactly parallels similar copy edits I made earlier this year to invoke.texi. To recap, in phrases like 64-bit types, 64-bit is a compound adjective phrase that immediately precedes a noun and

[doc] extend.texi copy-editing, 4/N (bit-fields)

2012-11-10 Thread Sandra Loosemore
I've checked in this patch to consistently use bit-field in extend.texi instead of bitfield or bit field. Bit-field is listed in the GCC Coding Conventions as the preferred terminology, for consistency with the C and C++ standards. -Sandra 2012-11-10 Sandra Loosemore

[Obj-C++] Found a small paste-o in parser.c?

2012-11-10 Thread Ed Smith-Rowland
I found this suspicious looking line in cp/parser.c () while looking at __thread and thread_local. Look at the patterns of the if blocks above the line in question to verify. Built and tested on x86_64-linux. Ed 2012-11-11 Ed Smith-Rowland 3dw...@verizon.net * parser.c