Re: [PATCH] Dissociate store_expr's temp from exp so that it is not marked as addressable

2012-03-31 Thread Martin Jambor
Hi, On Fri, Mar 30, 2012 at 10:03:59AM +0200, Richard Guenther wrote: > On Fri, 30 Mar 2012, Martin Jambor wrote: > > > Hi, > > > > when testing a patch of mine on sparc64-linux, I came across an Ada > > bootstrap failure due to a structure DECL which was marked addressable > > but had a registe

Re: [PATCH] Remove bogus assert from CCP's insert_clobbers_for_var

2012-03-31 Thread Martin Jambor
Hi, On Fri, Mar 30, 2012 at 01:23:00PM +0200, Eric Botcazou wrote: > > On IRC I've been told that is OK and the that CCP cannot make such > > assumtions. Since it is only a missed-optimization if the call to the > > builtin is not found and processed (basically PR 51491 again but only > > in case

Re: [PATCH] Remove bogus assert from CCP's insert_clobbers_for_var

2012-03-31 Thread Eric Botcazou
> > Please do that on the 4.7 branch as well if the assertion is incorrect. > > Done. Thanks. I've now reverted the kludge I put for Ada because of it: http://gcc.gnu.org/ml/gcc-patches/2012-03/msg01616.html -- Eric Botcazou

Re: [PATCH][3/n] Cleanup internal interfaces

2012-03-31 Thread Uros Bizjak
Hello! > 2012-03-29 Richard Guenther > > * cgraph.h (cgraph_materialize_all_clones): Remove. > (reset_inline_failed): Likewise. > * cgraphunit.c (cgraph_materialize_all_clones): Make static. > * cgraphbuild.c (reset_inline_failed): Remove. > * rtl.h (cse_main): Rem

[PATCH] Add implicit C linkage for win32-specific entry points

2012-03-31 Thread Jacek Caban
This is my first patch to GCC, so please let me know if I did something wrong. This patch fixes common annoyance on w64-mingw32 targets, where once needs to add explicit "C" linkage to make C++ app work with wmain entry point. * decl.c: Allow custom target implicit C linkage *

Re: [PATCH] Add implicit C linkage for win32-specific entry points

2012-03-31 Thread Eric Botcazou
> This is my first patch to GCC, so please let me know if I did something > wrong. This patch fixes common annoyance on w64-mingw32 targets, where > once needs to add explicit "C" linkage to make C++ app work with wmain > entry point. > > * decl.c: Allow custom target implicit C linkage >

[PATCH] Add implicit C linkage for win32-specific entry points

2012-03-31 Thread Jacek Caban
Fixed Changelog as requested by Eric. Thanks. gcc/ChangeLog * config/i386/mingw-w64.h: Specify entry points with implicit C linkage gcc/cp/ChangeLog: * decl.c: Allow custom target implicit C linkage gcc/testsuite/ChangeLog: * g++.dg/ext/main.C: Added implicit C linkage

[PATCH]: Restore alpha boostrap by partial revert

2012-03-31 Thread Uros Bizjak
Hello! Attached patch restores alpha bootstrap. 2012-03-31 Uros Bizjak Partially revert: 2012-03-29 Richard Guenther * rtl.h (extended_count): Remove. * combine.c (extended_count): Remove. Bootstrapped on alphaev68-pc-linux-gnu. OK for mainline? Uros. In

Re: [Patch ARM] Fix PR51819.

2012-03-31 Thread Ulrich Weigand
Ramana Radhakrishnan wrote: > PR51819 is a case where we were actually putting out alignment hints > for the wrong memory size. The attached patch corrects this and > another latent issue that I spotted. Your patch did: /* Only certain alignment specifiers are supported by the hardware.

Re: [C Patch]: pr52543

2012-03-31 Thread Kenneth Zadeck
New version of the patch, with all of Richard Sandiford's comments applied and retested. Ok for commit? Kenny 2012-03-31 Kenneth Zadeck * toplev.c (backend_init_target): Call initializer for lower-subreg pass. * lower-subreg.c (target_info): New static var. (compute_move_cost,

[PATCH, RFC] Extend __attribute__((format)) with user-specified conversion sequences

2012-03-31 Thread Eddie Kohler
Hi, Projects not uncommonly extend printf and/or scanf with new conversion characters, such gcc's own %< and %> for fancy quotes. Unfortunately, new conversion characters make __attribute__((format)) unusable for these functions, so errors go uncaught. This patch adds an extra optional argum

Re: [IA-64] Work around bug in unwinder

2012-03-31 Thread Eric Botcazou
> Looks ok, given the other ugliness in this macro. Thanks. There is another pending workaround for IA-64: http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00451.html as well as an implementation of stack checking in the back-end: http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00452.html -- Eric

[PATCH] PR c++/40942 - Failure of template specialization partial ordering

2012-03-31 Thread Dodji Seketeli
Hello, G++ compiles the example below without error: struct S { template S (T const *) //#0 { } template S (char const (&)[N]) //#1 { } }; int main() { S s1 ("test"); // #3 This should error out because the

Re: PATCH: Add OPTION_MASK_ISA_X86_64 and support TARGET_BI_ARCH == 2

2012-03-31 Thread H.J. Lu
On Fri, Mar 30, 2012 at 1:36 PM, H.J. Lu wrote: > On Fri, Mar 30, 2012 at 1:23 PM, Jack Howarth > wrote: >> On Fri, Mar 30, 2012 at 11:32:37AM -0700, H.J. Lu wrote: >>> On Fri, Mar 30, 2012 at 11:05 AM, Jack Howarth >>> wrote: >>> > On Fri, Mar 30, 2012 at 09:18:13AM -0700, H.J. Lu wrote: >>>

Re: PATCH: Add OPTION_MASK_ISA_X86_64 and support TARGET_BI_ARCH == 2

2012-03-31 Thread Jack Howarth
On Sat, Mar 31, 2012 at 11:20:27AM -0700, H.J. Lu wrote: > On Fri, Mar 30, 2012 at 1:36 PM, H.J. Lu wrote: > > On Fri, Mar 30, 2012 at 1:23 PM, Jack Howarth > > wrote: > >> On Fri, Mar 30, 2012 at 11:32:37AM -0700, H.J. Lu wrote: > >>> On Fri, Mar 30, 2012 at 11:05 AM, Jack Howarth > >>> wrote

PATCH: PR bootstrap/52812: --enable-targets=all --with-multilib-list=m32,m64,mx32 doesn't work with i686-linux

2012-03-31 Thread H.J. Lu
Hi, I checked in this patch as an obvious fix to handle -mx32 like -m64. Tested on Linux/ia32 with --enable-targets=all --with-multilib-list=m32,m64,mx32 Thanks. H.J. -- Index: libgomp/configure.tgt === --- libgomp/configure.tgt