Re: PATCH [trunk] gengtype should generate ggc_alloc macros in plugin mode.

Hi Basile, By the way, this patch is enough for the MELT plugin (which probably is the only plugin needing GTY). the dragonegg plugin also uses GTY, but only in a very mild way. Ciao, Duncan.

C++ PATCHes for c++/48481 (overload resolution memory hog)

48481 involves a huge function involving calls to functions taking class arguments. Detailed memory statistics pointed to three primary sources of garbage: 1) BASELINKs from lookup of constructors for considering a user-defined conversion sequence. We can just avoid creating these in the fir

Re: Simple enhancement to -dA dump

Ping? On Sun, Apr 3, 2011 at 5:54 PM, Xinliang David Li wrote: > For an assembly dump, it is very useful to examine the control flow > graph with frequency and profile count information. This requires > dumping the .s file with more control flow annotations. This patch > does just that. The form

FDO usability -- sanity check indirect call target

Hi, due to race conditions, it is common that the value profile information for an indirect call site is 'corrupted' -- resulting in false target function to be recorded. The value profile transformation won't cause runtime problem as the path will never be executed, however it may cause compile ti

Re: PATCH [trunk] gengtype should generate ggc_alloc macros in plugin mode.

On Thu, 7 Apr 2011 05:45:42 +0300 Laurynas Biveinis wrote: > The patch is correct in general. You didn't say "Ok with changes", so attached here is the patch to trunk 172161 gcc/ChangeLog entry ## 2011-04-08 Basile Starynkevitch * gengtype.c (write_typed_alloc_de

Merged mainline to gccgo branch

I merged mainline revision 172121 to the gccgo branch. Ian

C++ PATCH for c++/48500 (C++0x crash on non-type-dependent but non-constant call in template)

constexpr fallout. In fold_non_dependent_expr, we determined that the expression is not type-dependent, is potentially constant, and is not value-dependent, so we try to fold it. But it isn't potentially constant. So this patch fixes that determination. Tested x86_64-pc-linux-gnu, applying

C++ PATCH for use of parameter pack in default argument (48451)

Here deduction was failing because when we tried to substitute {int,int} for Args in decltype(T(std::declval()...)) we still had Args marked as an incomplete/deducible parameter pack, so the expansion wasn't fully resolved. So now we clear the incomplete flag before trying to substitute into l

[x32] PATCH: Add more x32 support to testsuite

Hi, This patch has been in my tree for a couple weeks. I am checking it into x32 branch. H.J. --- diff --git a/gcc/testsuite/ChangeLog.x32 b/gcc/testsuite/ChangeLog.x32 index 9bcf223..0081f4c 100644 --- a/gcc/testsuite/ChangeLog.x32 +++ b/gcc/testsuite/ChangeLog.x32 @@ -1,3 +1,18 @@ +2011-03-16

new option -Wno-maybe-uninitialized

Hi, the following patch implements the option to fine control the emitted warnings -- 1) allow suppressing warnings for use of values that may be uninitialized. Definitely uninitialized values that may be used warning is not affected 2) allow fine grain control on promotion of warnings to errors:

FDO usability patch -- correct insane profile data

Hi please review the attached patch. Ok when bootstrap and test finish? Thanks, David 2011-04-07 Xinliang David Li * ipa-cp.c (ipcp_update_profiling): Correct negative scale factor due to insane profile data. Index: ipa-cp.c

Avoid global state in arm_handle_option

This patch stops the ARM handle_option hook from using global state - in fact removing the hook altogether by using .opt Enum facilities for what it did. The -march, -mcpu and -mtune option values are made to use Enum. The Enum definitions go in a new generated file arm-tables.opt; for -mcpu/-mtu

[lipo] merge from trunk

Merged lw-ipo branch from trunk 163718:170935, committed as revision 172131. Tested spec2k-int with lipo. thanks, raksit

Re: Avoid global state in rx_handle_option

There was a bug in how the conversion to Enum was done in this patch; I've applied this followup patch to fix it. Index: ChangeLog === --- ChangeLog (revision 172137) +++ ChangeLog (working copy) @@ -1,5 +1,10 @@ 2011-04-07 Jos

C++/c-common PATCH to conversion from scoped enum to bool (part of 48450)

PR 48450 has to do with SFINAE bugs, but one of them turns out to be a different sort of bug: we were failing to convert from a non-constant value of scoped enum type to bool, because that conversion was doing a != 0, which requires an implicit conversion to int. So now we explicitly convert t

Re: [gc-improv] Permanent vs function RTL obstack fix

On Thu, Apr 7, 2011 at 8:17 AM, Laurynas Biveinis wrote: > Fixes a bunch of C testsuite failures. Committed to gc-improv. > > 2011-04-07  Laurynas Biveinis   > >        * stmt.c (label_rtx): Allocate RTX in permanent RTL memory. That looks strange, labels should be function specific, except non-l

[PATCH] 4.4 backports

Hi! And here are 3 backported patches plus one testcase to 4.4 branch, committed after bootstrap/regtest on x86_64-linux and i686-linux. Jakub 2011-04-07 Jakub Jelinek PR tree-optimization/46491 Backported from mainline 2010-05-14 Jan Hubicka * ipa-p

Re: [RFC] Remove kludge in commit_edge_insertions

> * basic-block.h (force_nonfallthru): Move to... > * cfghooks.h (struct cfg_hooks): Add force_nonfallthru hook. > (force_nonfallthru): ...here. > * cfghooks.c (force_nonfallthru): New function. > * cfgrtl.c (force_nonfallthru): Rename into... > (rtl_force_nonfal

fix non-local-goto-5.c

This fixes non-local-goto-5.c on targets that don't have trampolines. Noticed in the 4.6.0 release. 2011-04-07 Mike Stump * gcc.dg/torture/stackalign/non-local-goto-5.c: Fix for targets with no trampolines. Index: gcc.dg/torture/stackalign/non-local-goto-5.c

Re: [patch, fortran] More control over front end optimization

Hello, On Thursday 07 April 2011 21:50:46 Daniel Kraft wrote: > Ok. Just my opinion (as non-native-speaker), though: +@item -ffrontend-optimize +@opindex @code{frontend-optimize} +@cindex Front-end optimization +This option performs front-end optimization, based on the Fortran parse +tree. Ena

[4.5/4.6 PATCH, i386]: Some random fixes in SSE patterns.

Hello! 2011-04-07 Uros Bizjak * config/i386/sse.md: Update copyright year. (avx_cmp3): Add missing output register constraint. (*vec_concatv2sf_avx): Fix wrong register constraint in alternative 3 of operand 1. (*vec_set_0_avx): Avoid combining r

Re: [PATCH] sel-sched: Fix erroneous re-use of pointer to last insn in a BB (PR 48235)

On 04/07/2011 01:45 PM, Alexander Monakov wrote: Hello, This patch fixes a couple of places where code motion machinery wrongly attempts to re-use a pointer to the last insn in a BB after control flow following that BB has been changed (so the last jump might have been removed or replaced). Thi

Re: [PATCH] sel-sched: Fix adding of preheader blocks from inner loops (PR48302)

On 04/07/2011 01:34 PM, Alexander Monakov wrote: Hello, (quoting myself from the PR audit trail) We call sched_scan for preheader blocks from inner regions twice: first during sel_add_loop_preheaders, and then during the subsequent call to sel_init_bbs in setup_current_loop_nest. As a result, w

Re: [PATCH] sel-sched: Don't clone CALLs (PR48273)

On 04/07/2011 01:30 PM, Alexander Monakov wrote: Hi, Selective scheduler does not explicitly forbid creating bookkeeping copies of CALLs, but an assert in create_copy_of_insn_rtx does not expect a CALL. Since there's little benefit from aggressive scheduling of CALLs, it makes sense to restrict

Re: [PATCH] sel-sched: Fix an incorrect assert (PR 48442)

On 04/07/2011 01:16 PM, Alexander Monakov wrote: Hi, This patch fixes an incorrect assert in init_seqno and cleans up adjacent code a bit by removing an unneeded argument. Bootstrapped and regtested together with other recent sel-sched fixes on x86_64-linux with scheduler enabled at -O2. OK fo

Re: [patch, fortran] More control over front end optimization

Hi, On 04/07/11 21:32, Thomas Koenig wrote: following Joost's request, the attached patch gives the user more control over front end optimization - it can now be selected or deselected independent of normal optimization. I have also implemented a warning for eliminated functions. Regression-tes

Re: Patch: add {tree,gimple}-pretty-print.h & realmpfr.h to PLUGIN_HEADERS

On Thu, 7 Apr 2011 21:41:18 +0200 Basile Starynkevitch wrote: > > Hello All, > > The following tiny patch add some files to PLUGIN_HEADERS. > Since they are missing in 4.6, I had to copy them in the MELT plugin > tar ball release candidate 0. Sorry, my mailer wrapped the patch in line. I am

Re: [4.5] Backport various selective scheduler patches to 4.5 branch

On 04/06/2011 07:08 AM, Andrey Belevantsev wrote: Hello, As we discussed in PR 43603, we need to do another round of sel-sched patches' backporting to 4.5 -- there were a few reports about the problems already fixed on trunk. I have successfully bootstrapped and tested on x86-64 and ia64 the

Patch: add {tree,gimple}-pretty-print.h & realmpfr.h to PLUGIN_HEADERS

Hello All, The following tiny patch add some files to PLUGIN_HEADERS. Since they are missing in 4.6, I had to copy them in the MELT plugin tar ball release candidate 0. # tiny patch to trunk 172124 Index: gcc/Makefile.in === -

patch to fix PR 48435

The following patch should solve problem http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48435. It might solve other latest IRA problems too including performance related ones. The patch is for targets which require some values to be placed in hard registers starting with an even (odd) hard regis

[patch, fortran] More control over front end optimization

Hello world, following Joost's request, the attached patch gives the user more control over front end optimization - it can now be selected or deselected independent of normal optimization. I have also implemented a warning for eliminated functions. Regression-tested. Once this is in, I wi

Re: [RFC PATCH, go]: Port to ALPHA arch - sysinfo.go fixup

Ian, > Rainer Orth writes: > >> I've got a similar issue on IRIX 6.5: has >> >> struct timeval { >> #if _MIPS_SZLONG == 64 >> __int32_t :32; >> #endif >> time_t tv_sec; /* seconds */ >> longtv_usec;/* and microseconds */ >> }; >> >> which causes the 64-bit lib

[PATCH] Several backports to 4.5 branch

Hi! I've backported a couple of patches to 4.5 branch and committed them after bootstrapping/regtesting on x86_64-linux and i686-linux. Additionally, I've added testcase for PR fortran/48117 to 4.5 (where it is newly fixed) and 4.6/4.7 (where it has been fixed quite some time ago already by Richa

[pph] Macro Validation (issue4379044)

In my last PPH change, I eliminated the redundancy in the preprocessor identifier lookaside table by removing the name of the identifier from the head of the macro value. This later led to a buffer overrun in libcpp/symtab.c cpp_lt_replay. The buffer was allocated based on the value string size,

[PATCH] sel-sched: Fix erroneous re-use of pointer to last insn in a BB (PR 48235)

Hello, This patch fixes a couple of places where code motion machinery wrongly attempts to re-use a pointer to the last insn in a BB after control flow following that BB has been changed (so the last jump might have been removed or replaced). This is not too frequent, so the solution is to simply

[PATCH] sel-sched: Fix adding of preheader blocks from inner loops (PR48302)

Hello, (quoting myself from the PR audit trail) We call sched_scan for preheader blocks from inner regions twice: first during sel_add_loop_preheaders, and then during the subsequent call to sel_init_bbs in setup_current_loop_nest. As a result, we also call remove_notes for those blocks twice, an

[PATCH] sel-sched: Don't clone CALLs (PR48273)

Hi, Selective scheduler does not explicitly forbid creating bookkeeping copies of CALLs, but an assert in create_copy_of_insn_rtx does not expect a CALL. Since there's little benefit from aggressive scheduling of CALLs, it makes sense to restrict it so that CALLs cannot be duplicated. This patch

[PATCH] sel-sched: Fix an incorrect assert (PR 48442)

Hi, This patch fixes an incorrect assert in init_seqno and cleans up adjacent code a bit by removing an unneeded argument. Bootstrapped and regtested together with other recent sel-sched fixes on x86_64-linux with scheduler enabled at -O2. OK for trunk? PR rtl-optimization/48442

[testsuite] Don't XFAIL gcc.target/i386/asm-6.c on 32-bit Solaris 10+/x86

gcc.target/i386/asm-6.c recently started XPASSing on Solaris 10+/x86. There's evidence in PR rtl-optimization/44174 that this isn't just an accident, so I'm removing the XFAIL to reduce the noise. Tested with the appropriate runtest invocation on i386-pc-solaris2.11, installed on mainline.

gcc-patches@gcc.gnu.org

On Thu, 7 Apr 2011, Gunther Nikl wrote: > On Fri, Oct 08, 2010 at 09:58:14AM -0400, Nathan Froyd wrote: > > The patch below hookizes FUNCTION_ARG and related macros for the m68k > > backend. Nothing special here. > > With the old macro approach it was easy for a subtarget to override > these tar

Re: Convert legacy m68k options to .opt aliases

On Thu, 7 Apr 2011, Gunther Nikl wrote: > > an alias for -mcpu=68332 rather than -mcpu=cpu32, to match the old > > code in m68k_handle_option.) This significantly simplifies the > > multilibs code in t-mlibs, since it no longer needs to handle those > > old-style options (and all cases where two

gcc-patches@gcc.gnu.org

On Fri, Oct 08, 2010 at 09:58:14AM -0400, Nathan Froyd wrote: > The patch below hookizes FUNCTION_ARG and related macros for the m68k > backend. Nothing special here. With the old macro approach it was easy for a subtarget to override these target properties. That doesn't work anymore for target

Re: [PATCH] Cleanup, use add_to_hard_reg_set instead of SET_HARD_REG_BIT loops.

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/06/11 12:21, Anatoly Sokolov wrote: > Hi. > > This patch converts loops of SET_HARD_REG_BIT in to add_to_hard_reg_set > functions call. > > The patch has been bootstrapped on and regression tested on > x86_64-unknown-linux-gnu for c. > >

Re: [PATCH, M68K] Handle -fsched-pressure

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/07/11 09:02, Maxim Kuvyrkov wrote: > On Apr 7, 2011, at 6:55 PM, Jeff Law wrote: > >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On 04/07/11 08:29, Maxim Kuvyrkov wrote: >>> The following simple patch fixes a corner-case in m68k sche

Re: [PATCH, M68K] Handle -fsched-pressure

On Apr 7, 2011, at 6:55 PM, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 04/07/11 08:29, Maxim Kuvyrkov wrote: >> The following simple patch fixes a corner-case in m68k scheduling when >> -fsched-pressure is used. Currently m68k scheduling hooks assume that the >> s

Re: Convert legacy m68k options to .opt aliases

On Wed, Apr 06, 2011 at 10:04:37PM +, Joseph S. Myers wrote: > Similar to , > this patch converts legacy m68k options for non-ColdFire CPUs into > aliases for the corresponding -mcpu= options. (Note that -mcpu32 is While I agree with th

Re: [PATCH, M68K] Handle -fsched-pressure

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/07/11 08:29, Maxim Kuvyrkov wrote: > The following simple patch fixes a corner-case in m68k scheduling when > -fsched-pressure is used. Currently m68k scheduling hooks assume that the > scheduler honors DFA restrictions, which is not always th

[PATCH, testsuite, committed] Fix test for odd-TLS targets

The attached patch fixes gcc.dg/lto/20090210_0.c link test for targets that require C library support for TLS, e.g., m68k [*]. The patch changes the condition on the test from requiring theoretical TLS support, to requiring the toolchain to be able to link a program which uses TLS. [*] M68k TL

[PATCH, M68K] Handle -fsched-pressure

The following simple patch fixes a corner-case in m68k scheduling when -fsched-pressure is used. Currently m68k scheduling hooks assume that the scheduler honors DFA restrictions, which is not always the case when scheduling for register pressure (-fsched-pressure). The patch fixes this corner

Re: PATCH: PR middle-end/48440: [4.7 Regression] FAIL: gcc.c-torture/compile/labels-3.c

On Thu, Apr 7, 2011 at 5:34 AM, Michael Matz wrote: > Hi, > > On Thu, 7 Apr 2011, Richard Guenther wrote: > >> > 5600      newx = simplify_subreg (outermode, op, innermode, byte); >> > (gdb) f 1 >> > #1  0x00708494 in expand_expr_real_2 (ops=0x7fffb0c0, >> > target=0x0, >> >    tmode=

[patch i386]: Cleanup calling convention handling in i386.c and fix PR target/9601 and PR target/11772

Hello, This patch cleans up a bit the calling convention checking in i386 architecture. By this I could fix some pretty old bugs about the -mrtd option. See as reference http://gcc.gnu.org/bugzilla/show_bug.cgi?id=11772 and http://gcc.gnu.org/bugzilla/show_bug.cgi?id=9601 For none-prototyped fun

New German PO file for 'gcc' (version 4.6.0)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.6.0.de.po', has just

[PATCH,c++] fix PR objc++/48479, ICE in cxx_mark_addressable

My recent patch removing DECL_RTL from CONST_DECLs caused regressions in the ObjC++ testsuite on Darwin targets. The problem is that DECL_REGISTER was being called on CONST_DECLs; DECL_REGISTER says: /* In VAR_DECL and PARM_DECL nodes, nonzero means declared `register'. */ #define DECL_REGISTER(

Re: PATCH: PR middle-end/48440: [4.7 Regression] FAIL: gcc.c-torture/compile/labels-3.c

Hi, On Thu, 7 Apr 2011, Richard Guenther wrote: > > 5600      newx = simplify_subreg (outermode, op, innermode, byte); > > (gdb) f 1 > > #1  0x00708494 in expand_expr_real_2 (ops=0x7fffb0c0, > > target=0x0, > >    tmode=VOIDmode, modifier=EXPAND_INITIALIZER) > >    at /export/gnu/imp

Re: [Patch] Bfin: Ensure rotrsi and rotlsi don't accept non-const INTVALS

On 04/04/2011 06:04 PM, Henderson, Stuart wrote: > Yep, I'm seeing this behaviour (getting the error using your patch). > But I'm confused as to why the define_expand is being considered when > the predicate doesn't match. It's just a bug in expand. Your original patch (plus using const_int_operan

Re: [patch, ARM] Fix PR target/48252

On 07/04/11 08:42, Ira Rosen wrote: Hi, This patch makes both outputs of neon_vzip/vuzp/vtrn_internal explicitly dependent on both inputs, preventing incorrect optimization: for (a,b)<- vzip (c,d) and (e,f)<- vzip (g,d) CSE decides that b==f, since b and f depend only on d. Tested on arm-linux-

[Patch,AVR]: Fix PR target/44643

This is fix of PR44643 which is triggerd by http://savannah.nongnu.org/bugs/?32988 i.e. include/avr/pgmspace.h:PSTR has to be changed, too: PSTR has to generate __c as "static const char[]", not as "static char[]". Note that avr-libc has to be changed anyway, no matter if or if not this patch is

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

On 07/04/11 12:08, Carrot Wei wrote: On Thu, Apr 7, 2011 at 5:31 PM, Richard Sandiford wrote: Hi Carrot, Sorry if this has already been reported, but the patch breaks bootstrap of arm-linux-gnueabi (or cross builds with --enable-werror). The problem is that this... uses a statement expressi

[Patch, committed] Add myself to MAINTAINERS

Added myself to MAINTAINERS (write after approval), committed as directed. Stu 2011-04-07 Stuart Henderson * MAINTAINERS (Write After Approval): Add myself. Index: MAINTAINERS === --- MAINTAINERS (revision 172094) +++

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

On Thu, Apr 7, 2011 at 5:31 PM, Richard Sandiford wrote: > Hi Carrot, > > Sorry if this has already been reported, but the patch breaks bootstrap > of arm-linux-gnueabi (or cross builds with --enable-werror).  The problem > is that this... > > uses a statement expression -- i.e. ({ code; result; }

Fix PR47976

PR47976 is a followup to PR47166; the patch there caused this problem. The problem occurs in reload. There are two autoinc addresses which inherit from one another, and we delete an insn that is necessary. We reach this code when reloading the second autoinc address: 6821 if (optimize &

Re: [PATCH, ARM] PR47855 Compute attr "length" for some thumb2 insns

Hi Carrot, Sorry if this has already been reported, but the patch breaks bootstrap of arm-linux-gnueabi (or cross builds with --enable-werror). The problem is that this... Carrot Wei writes: > @@ -10256,7 +10281,29 @@ > > return \"\"; >}" > - [(set_attr "type" "store4")] > + [(set_at

Re: [PATCH] Remove ASM_OUTPUT_BSS macro.

2011/4/6 Anatoly Sokolov : > Hi. > >  No one back end does not use ASM_OUTPUT_BSS macro now, this patch remove it. > The GCC have more flexible ASM_OUTPUT_ALIGNED_BSS macro. > >  The patch has been bootstrapped on and regression tested on > x86_64-unknown-linux-gnu for c. > >  OK to install? Ok.

Re: PATCH: PR middle-end/48440: [4.7 Regression] FAIL: gcc.c-torture/compile/labels-3.c

On Wed, Apr 6, 2011 at 7:03 PM, H.J. Lu wrote: > On Tue, Apr 5, 2011 at 3:29 AM, Richard Guenther > wrote: >> On Tue, Apr 5, 2011 at 8:44 AM, Paolo Bonzini wrote: >> Index: cgraphbuild.c >> === >> --- cgraphbuild.c.orig

[wwwdocs] CSS speaks American English, not Her Majesty's.

'nough said. :-) Thanks to Tobias for the hint, and I have now extended my verifier to also cover CSS. Gerald Index: gcc.css === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc.css,v retrieving revision 1.15 diff -u -r1.15 gcc.css --- gcc.cs

Re: [MIPS] Remove REG_OK_FOR_BASE_P and REG_OK_FOR_INDEX_P macros

Anatoly Sokolov writes: > * config/mips/mips.h (REG_MODE_OK_FOR_BASE_P, REG_OK_FOR_INDEX_P): > Remove macros. OK, thanks. Richard

[patch, ARM] Fix PR target/48252

Hi, This patch makes both outputs of neon_vzip/vuzp/vtrn_internal explicitly dependent on both inputs, preventing incorrect optimization: for (a,b)<- vzip (c,d) and (e,f) <- vzip (g,d) CSE decides that b==f, since b and f depend only on d. Tested on arm-linux-gnueabi. OK for trunk? OK for 4.6 aft