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 register DECL_RTL

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 cases like

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 rguent...@suse.de * 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

[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 ubiz...@gmail.com Partially revert: 2012-03-29 Richard Guenther rguent...@suse.de * rtl.h (extended_count): Remove. * combine.c (extended_count): Remove. Bootstrapped on

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 zad...@naturalbridge.com * toplev.c (backend_init_target): Call initializer for lower-subreg pass. * lower-subreg.c (target_info): New static

[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

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 typename T S (T const *) //#0 { } template int N 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 hjl.to...@gmail.com wrote: On Fri, Mar 30, 2012 at 1:23 PM, Jack Howarth howa...@bromo.med.uc.edu 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 howa...@bromo.med.uc.edu wrote: On Fri,

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 hjl.to...@gmail.com wrote: On Fri, Mar 30, 2012 at 1:23 PM, Jack Howarth howa...@bromo.med.uc.edu wrote: On Fri, Mar 30, 2012 at 11:32:37AM -0700, H.J. Lu wrote: On Fri, Mar 30, 2012 at

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