Re: C++ PATCH to add auto return type deduction with -std=c++1y

2012-03-29 Thread Marc Glisse
On Wed, 28 Mar 2012, Jason Merrill wrote: On 03/28/2012 03:20 PM, Marc Glisse wrote: I agree that it works like initialization, and like lambdas, so that ship has sailed. I assume there were good reasons for that, even if they are not obvious, I know things can be trickier than they appear.

[v3] libstdc++/52689

2012-03-29 Thread Benjamin De Kosnik
Seems like libstdc++.a has suffered a bit in the transition to a more modular convenience-library build system. Here's a patch to clean up the compatibility symbols throughout libstdc++, to insure that necessary symbols for versioning purposes are only in shared binaries, ie libstdc++.so. Most

Re: C++ PATCH to add auto return type deduction with -std=c++1y

2012-03-29 Thread Marc Glisse
On Wed, 28 Mar 2012, Marc Glisse wrote: On Sun, 25 Mar 2012, Marc Glisse wrote: - a first goal is simple functions, with a single return statement (which may even often be the only statement). After playing with it a bit, I am not sure how to use it in the simple forwarding case: T

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

2012-03-29 Thread Richard Guenther
On Wed, 28 Mar 2012, Martin Jambor wrote: Hi, when testing a different patch of mine I hit the assert in insert_clobbers_for_var which is there to make sure that there is a call to builtin_stack_save in a BB with or dominating a call to builtin_alloca_with_align. In my case that was not

[PATCH, ping #1] Don't insert white space in 'orig_option_with_args_text' for OPT_l

2012-03-29 Thread Ludovic Courtès
Ping. ludovic.cour...@inria.fr (Ludovic Courtès) skribis: Joseph S. Myers jos...@codesourcery.com skribis: On Mon, 12 Mar 2012, Ludovic Courtès wrote: The patch below solves the problem in a gfortran-specific way. WDYT? I think that's the right approach for this issue. The previous

Re: [patch] Fix ICE during inlining with SJLJ

2012-03-29 Thread Richard Guenther
On Wed, Mar 28, 2012 at 4:42 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, this is a regression present on the mainline and 4.7 branch for platforms using SJLJ exceptions, e.g. the ARM.  The scenario is as follows: the main procedure calls My_Iterators.Current which has a pragma Inline

Re: [patch] Add support for FP bit fields in varasm.c

2012-03-29 Thread Richard Guenther
On Wed, Mar 28, 2012 at 4:53 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, another kind of bit fields supported in Ada are floating-point bit fields. They work fine, except that varasm.c rejects static constants (CONSTRUCTORs) containing them.  The attached patch plugs this hole.

[PATCH][1/n] Cleanup internal interfaces, GCC modularization

2012-03-29 Thread Richard Guenther
I am playing with doing some internal interface static analysis using the first patch below (and looking at LTO bootstrap results). An example, obvious patch resulting from that is the 2nd patch, resuling from the static analysis output

[PATCH] SH2A: Don't push/pop registers for functions with resbank attribute

2012-03-29 Thread Naveen H. S
Hi, Please find attached the patch resbank.patch which fixes the issue with resbank attribute. Currently, registers used in the routine are also saved on using resbank attribute. These registers are saved with resbank instruction and need not be saved separately. The patch fixes the issue.

[PATCH][configure] Make sure CFLAGS_FOR_TARGET And CXXFLAGS_FOR_TARGET contain -O2

2012-03-29 Thread Christophe Lyon
Hello, According to a comment in configure/configure.ac: # We want to ensure that TARGET libraries (which we know are built with # gcc) are built with -O2 -g, so include those options when setting # CFLAGS_FOR_TARGET and CXXFLAGS_FOR_TARGET. but the current code does not ensure this. I propose

Re: C++ PATCH to add auto return type deduction with -std=c++1y

2012-03-29 Thread Marc Glisse
On Thu, 29 Mar 2012, Marc Glisse wrote: On Wed, 28 Mar 2012, Marc Glisse wrote: On Sun, 25 Mar 2012, Marc Glisse wrote: - a first goal is simple functions, with a single return statement (which may even often be the only statement). After playing with it a bit, I am not sure how to use it

Re: [PATCH, i386]: FIX PR 52698, reload failure with complex address

2012-03-29 Thread Richard Henderson
On 03/28/2012 02:40 AM, Uros Bizjak wrote: What about release branches? While this issue didn't trigger there yet, it can be triggered by some bad RA decision, and the fixup is missing. I have no objection to your current patch being backported. It looked fairly safe. r~

[C++ PATCH] Don't call maybe_apply_pragma_weak for templates (PR c++/52759)

2012-03-29 Thread Jakub Jelinek
Hi! The recently added assert in mangle.c doesn't like mangling to be performed during processing_template_decl. With #pragma weak foo we can hit it unfortunately. This patch fixes that. Perhaps we could call maybe_apply_pragma_weak from pt.c when instantiating, though not sure how much useful

Re: [SMS] Support new loop pattern

2012-03-29 Thread Andrey Belevantsev
Hello, I'd like to ping again those SMS patches once we're back to Stage 1. Ayal, maybe it would remove some burden for you if you'd review the general SMS functionality of those patches, and we'd ask RTL folks to look at the pieces related to RTL pattern matching and generation? Yours,

[libffi] Restore three commits

2012-03-29 Thread Anthony Green
My merge from a few weeks ago accidentally trampled three patches by Kai Teitz and Eric Botcazou from last month: http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00553.html http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00524.html http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01163.html I've just

[PATCH][2/n] Cleanup internal interfaces

2012-03-29 Thread Richard Guenther
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-03-29 Richard Guenther rguent...@suse.de * tree-flow.h (struct pre_expr_d): Remove forward declaration. (add_to_value): Remove. (print_value_expressions): Likewise. *

[PATCH][3/n] Cleanup internal interfaces

2012-03-29 Thread Richard Guenther
Some more stuff. dse.c can probably be simplified with dse_record_singleton_alias_set / dse_invalidate_singleton_alias_set being removed. LTO Bootstrapped on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-03-29 Richard Guenther rguent...@suse.de * cgraph.h

Re: [PATCH][1/n] Cleanup internal interfaces, GCC modularization

2012-03-29 Thread Jan Hubicka
I am playing with doing some internal interface static analysis using the first patch below (and looking at LTO bootstrap results). An example, obvious patch resulting from that is the 2nd patch, resuling from the static analysis output

Re: [C++ Patch] for c++/52465

2012-03-29 Thread Jason Merrill
On 03/08/2012 04:34 PM, Fabien Chêne wrote: * decl.c (grokdeclarator): Call strip_using_decl. I would think we ought to be stripping USING_DECLs at a lower level, when we first look up the name in the parser. They shouldn't make it as far as grokdeclarator. Jason

C++ PATCH for c++/52743 (ICE with list-initialized array)

2012-03-29 Thread Jason Merrill
List-initialization of an array of scalars is not considered a user-defined conversion, so we need to check for it specifically to avoid treating it as a standard conversion. Tested x86_64-pc-linux-gnu, applying to trunk and 4.7. commit efaacb4065cce51b7374c31ddfb4682a880cbfc5 Author: Jason

Re: PATCH: Backport x32 support to libtool

2012-03-29 Thread H.J. Lu
On Sat, Mar 3, 2012 at 9:54 AM, H.J. Lu hongjiu...@intel.com wrote: Hi, This patch backports x32 support to libtool: http://git.savannah.gnu.org/cgit/libtool.git/commit/?id=88992fe6771ec3258bde1b03314ce579da0ac2d5 OK to install? Thanks. H.J. --- ommit

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

2012-03-29 Thread Gabriel Dos Reis
On Thu, Mar 29, 2012 at 8:50 AM, Richard Guenther rguent...@suse.de wrote: Some more stuff.  dse.c can probably be simplified with dse_record_singleton_alias_set / dse_invalidate_singleton_alias_set being removed. Can we also make big functions much smaller? semi :-) -- Gaby

Re: [C++ PATCH] Don't call maybe_apply_pragma_weak for templates (PR c++/52759)

2012-03-29 Thread Jason Merrill
On 03/29/2012 08:30 AM, Jakub Jelinek wrote: The recently added assert in mangle.c doesn't like mangling to be performed during processing_template_decl. With #pragma weak foo we can hit it unfortunately. This patch fixes that. Perhaps we could call maybe_apply_pragma_weak from pt.c when

[PATCH] Fix SLP miscompilation with constant shifts (PR tree-optimization/52760)

2012-03-29 Thread Jakub Jelinek
Hi! The shift count operand of shifts/rotates uses int type which is different from lhs/rhs1 type. If shift/rotate counts are constant, vectorizable_shift converts it to the right type, but then the types in vect_get_constant_vectors aren't compatible and we end up folding a ushort VCE of int,

Re: [C++ RFC / Patch] Implementing Deducing noexcept for destructors

2012-03-29 Thread Paolo Carlini
... attached the testsuite changes I have so far (seem all rather straightforward to me). Thanks, Paolo. Index: testsuite/g++.old-deja/g++.eh/cleanup1.C === --- testsuite/g++.old-deja/g++.eh/cleanup1.C

C++ PATCH for c++/52685 (ICE with dependent/non-dependent bases)

2012-03-29 Thread Jason Merrill
In this testcase, C has a base BU which is a non-dependent scope because it is the current instantiation. But BU has a dependent base ABU, so we need to be able to handle a non-dependent base that itself has a dependent base. Tested x86_64-pc-linux-gnu, applying to trunk and 4.7. commit

patch to fix a bug in coloring in multi-register allocnos presence.

2012-03-29 Thread Vladimir Makarov
Frederic Riss sent me IRA dump files for an in-house GCC back-end which helped to find a bug in coloring in presence of multi-register allocnos. The bug resulted in making allocno colorable too earlier and as a consequence in worse code. The problem was in preventing processing of all

Re: [PATCH, i386, Android] -mandroid support for i386 target

2012-03-29 Thread Jan Hubicka
2012-02-27 Enkovich Ilya ilya.enkov...@intel.com * gcc/config/i386/gnu-user.h (GNU_USER_TARGET_CC1_SPEC): New. (CC1_SPEC): Use GNU_USER_TARGET_CC1_SPEC. (GNU_USER_TARGET_LINK_SPEC): New. (LINK_SPEC): Use GNU_USER_TARGET_LINK_SPEC.

Re: [PATCH, i386, Android] Enable __ANDROID__ macro for Android i386 target

2012-03-29 Thread Jan Hubicka
Undef TARGET_OS_CPP_BUILTINS and define TARGET_OS_CPP_BUILTINS in linux.h with GNU_USER_TARGET_OS_CPP_BUILTINS and ANDROID_TARGET_OS_CPP_BUILTINS. -- H.J. Hello, Here is a variant with linux.h modification. Does it look fine? OK, Honza

Re: [PATCH] eh_personality.cc: unwinding on ARM

2012-03-29 Thread Peter Waechtler
On 19.03.2012 17:38, Daniel Jacobowitz wrote: On Mon, Mar 19, 2012 at 12:12 PM, Andrew Stubbsa...@codesourcery.com wrote: On 16/03/12 13:29, EXTERNAL Waechtler Peter (Fa. TCP, CM-AI/PJ-CF31) wrote: The CodeSourcery toolchain contains a fix like the following, please consider for adding it.

Re: [C++ RFC / Patch] Implementing Deducing noexcept for destructors

2012-03-29 Thread Jason Merrill
On 03/28/2012 06:40 PM, Paolo Carlini wrote: + /* 12.4/3 */ + if (cxx_dialect= cxx0x + DECL_DESTRUCTOR_P (decl) + TYPE_RAISES_EXCEPTIONS (TREE_TYPE (old_decl))) + deduce_noexcept_on_destructor (decl); The exception specification on old_decl

[C++ Patch] PR 52718

2012-03-29 Thread Paolo Carlini
Hi, this fixes the PR by checking for zero in check_default_argument and, in case, replacing it with nullptr to avoid further warnings later on. Should be safe enough for 4.7.1 too. Tested x86_64-linux. Thanks, Paolo. / /cp 2012-03-29 Paolo Carlini

Re: [C++ RFC / Patch] Implementing Deducing noexcept for destructors

2012-03-29 Thread Paolo Carlini
Hi, On 03/28/2012 06:40 PM, Paolo Carlini wrote: + /* 12.4/3 */ + if (cxx_dialect= cxx0x + DECL_DESTRUCTOR_P (decl) + TYPE_RAISES_EXCEPTIONS (TREE_TYPE (old_decl))) +deduce_noexcept_on_destructor (decl); The exception specification on old_decl doesn't matter; we can drop

Re: [C++ RFC / Patch] Implementing Deducing noexcept for destructors

2012-03-29 Thread Paolo Carlini
Hi, On 03/28/2012 06:40 PM, Paolo Carlini wrote: + /* 12.4/3 */ + if (cxx_dialect= cxx0x + DECL_DESTRUCTOR_P (decl) + TYPE_RAISES_EXCEPTIONS (TREE_TYPE (old_decl))) +deduce_noexcept_on_destructor (decl); The exception specification on old_decl doesn't matter; we can drop

Re: [C++ RFC / Patch] Implementing Deducing noexcept for destructors

2012-03-29 Thread Jason Merrill
On 03/29/2012 03:06 PM, Paolo Carlini wrote: The exception specification on old_decl doesn't matter; we can drop that test. I seem to remember something going wrong with templates otherwise, because implicitly_declare_fn has gcc_assert (!dependent_type_p (type)); We shouldn't be doing this

Re: [C++ Patch] PR 52718

2012-03-29 Thread Jason Merrill
OK. Jason

[PATCH] ARM: Use different linker path for hardfloat ABI

2012-03-29 Thread dann frazier
This is an updated version of a patch Debian and Ubuntu are using to use an alternate linker path for hardfloat binaries. The difference with this one is that it covers the case where no float flag was passed in, defaulting to the softfloat path. 2012-03-29 dann frazier

Re: [C++ RFC / Patch] Implementing Deducing noexcept for destructors

2012-03-29 Thread Paolo Carlini
On 03/29/2012 09:27 PM, Jason Merrill wrote: On 03/29/2012 03:06 PM, Paolo Carlini wrote: The exception specification on old_decl doesn't matter; we can drop that test. I seem to remember something going wrong with templates otherwise, because implicitly_declare_fn has gcc_assert

Re: [PATCH H8300] Add function_vector attribute

2012-03-29 Thread Jeff Law
On 03/28/2012 09:10 AM, Ajinkya Dhobale wrote: Hi Jeff, Thank you for replying to the post. How is this different than the function vector support that is already in GCC for the H8/300 series processors? Current H8/300 implementation of function vector seems inappropriate. This patch fixes

Re: [patch][rfa] Do not call output_constant from the front end

2012-03-29 Thread Steven Bosscher
On Wed, Mar 28, 2012 at 12:02 PM, Richard Guenther richard.guent...@gmail.com wrote: Therefore, an RFA for the attached patch. Bootstrappedtested on powerpc64-unknown-linux-gnu. OK? Ok. Thanks. Committed as r185977. Ciao! Steven

[C Patch]: pr52543

2012-03-29 Thread Kenneth Zadeck
This patch takes a different approach to fixing PR52543 than does the patch in http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00641.html This patch transforms the lower-subreg pass(es) from unconditionally splitting wide moves, zero extensions, and shifts, so that it now takes into account the

Re: [PATCH] SH2A: Don't push/pop registers for functions with resbank attribute

2012-03-29 Thread Kaz Kojima
Naveen H. S navee...@kpitcummins.com wrote: Please find attached the patch resbank.patch which fixes the issue with resbank attribute. Currently, registers used in the routine are also saved on using resbank attribute. These registers are saved with resbank instruction and need not be saved

[PATCH, i386]: Fix vhaddpd/vhsubpd AVX patterns

2012-03-29 Thread Uros Bizjak
Hello! My recent patch [1] exposed problem with avx_hplusminus_insnv4df3 AVX pattern. The pattern is defined in a wrong way, since the components of the result should not cross 128bit lane boundary. This is the cause of failures is [2]: FAIL: gcc.target/i386/avx-vhaddpd-256-1.c execution test

Re: [C++ RFC / Patch] Implementing Deducing noexcept for destructors

2012-03-29 Thread Paolo Carlini
On 03/29/2012 09:27 PM, Jason Merrill wrote: On 03/29/2012 03:06 PM, Paolo Carlini wrote: The exception specification on old_decl doesn't matter; we can drop that test. I seem to remember something going wrong with templates otherwise, because implicitly_declare_fn has gcc_assert

Support for Runtime CPU type detection via builtins (issue5754058)

2012-03-29 Thread Sriraman Tallam
Subject:Support for Runtime CPU type detection via builtins Hi, I have made a new patch to only have two builtins : * __builtin_cpu_is (CPUNAME) * __builtin_cpu_supports (FEATURE) apart from the cpu init builtin, __builtin_cpu_init. List of CPU names : * amd * intel * atom * core2 *

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-03-29 Thread Sriraman Tallam
Subject:Support for Runtime CPU type detection via builtins Hi, I have uploaded a new patch to only have two builtins : * __builtin_cpu_is (CPUNAME) * __builtin_cpu_supports (FEATURE) apart from the cpu init builtin, __builtin_cpu_init. List of CPU names : * amd * intel * atom * core2 *

Re: [PATCH] PR c++/52672

2012-03-29 Thread Meador Inge
On 03/26/2012 09:14 AM, Meador Inge wrote: Hi All, This patch fixes an ICE that occurs when attempting to fold nested INDIRECT_REF trees that have conversions in between the indirect references. For example: constexpr unsigned long b = *((ul_ptr)(*((ul_ptr)0x0))); What happens is

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

2012-03-29 Thread Jack Howarth
On Wed, Mar 28, 2012 at 03:40:28PM -0700, H.J. Lu wrote: On Wed, Mar 28, 2012 at 3:07 PM, Joseph S. Myers jos...@codesourcery.com wrote: On Wed, 28 Mar 2012, H.J. Lu wrote: Here is the updated patch.  I will wait for OK from Joseph. I have no comments on this patch. Given that my

libitm MinGW detect

2012-03-29 Thread niXman
Hello. -- Regards,   niXman libitm-mingw-detect.diff Description: Binary data

[PATCH] SH: Fix m2a-single-only compilation error

2012-03-29 Thread Naveen H. S
Hi, Please find attached the patch crt1.patch which fixes compilation issue with sh2a-single-only target. Currently, compilation generates the following error:- merge of architecture 'sh3e' with architecture 'sh2a' produced unknown architecture The patch fixes the issue. Tested with sh2a-elf.

Go patch committed: Fix order of evaluation of struct literals

2012-03-29 Thread Ian Lance Taylor
This patch to the Go compiler fixes the order of evaluation of struct composite literals. Previously the compiler assumed that all fields in the struct were set in order. However, the composite literal might be written differently. If the composite literal had function calls, Go specifies that