Ping: [PATCH] Enable bbro for -Os

Ping. Thanks! -Zhenqiang > -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Zhenqiang Chen > Sent: Tuesday, August 14, 2012 2:50 PM > To: gcc-patches@gcc.gnu.org > Subject: [PATCH] Enable bbro for -Os > > Hi, > > Basic block

Re: [PATCH] Add working-set size and hotness information to fdo summary (issue6465057)

> On Tue, Aug 21, 2012 at 6:56 PM, Jan Hubicka wrote: > >> > I can go ahead with the histogram approach. There is some roundoff > >> > error from the working set scaling approach that can affect different > >> > merging orders as you note, although I think this only really affects the > >> > small

[Patch, Fortran, committed] Free loop and gfc_ss data

Committed as Rev. 190586 after successful regtesting. That's the version I also had attached to http://gcc.gnu.org/ml/fortran/2012-08/msg00118.html; as written there: "The patch is incomplete, e.g. "argss" of gfc_conv_procedure_call is not (or not always) freed. Ditto for rss of gfc_trans_ass

Ping^4: Properly handle arg_pointer and frame_pointer in DWARF output

(^4 because I unwittingly submitted the same patch a while back.) Ping for H.J.'s patch to avoid using dwarf extension codes for simple CFA addresses based on arg_pointer_rtx and frame_pointer_rtx in cases where Pmode is wider than the DWARF address size: http://gcc.gnu.org/ml/gcc-patches/2012-04

Re: [PATCH] Add working-set size and hotness information to fdo summary (issue6465057)

On Tue, Aug 21, 2012 at 6:56 PM, Jan Hubicka wrote: >> > I can go ahead with the histogram approach. There is some roundoff >> > error from the working set scaling approach that can affect different >> > merging orders as you note, although I think this only really affects the >> > small counter v

Re: [PATCH 4/4] Reduce the size of optabs representation

On Jul 19, 2012, at 11:24 AM, Richard Henderson wrote: > +# genopinit produces two files. > +insn-opinit.c insn-opinit.h: s-opinit ; @true > +s-opinit: $(MD_DEPS) build/genopinit$(build_exeext) insn-conditions.md > + $(RUN_GEN) build/genopinit$(build_exeext) $(md_file) \ > + insn-conditio

Re: [PATCH] Add working-set size and hotness information to fdo summary (issue6465057)

> > I can go ahead with the histogram approach. There is some roundoff > > error from the working set scaling approach that can affect different > > merging orders as you note, although I think this only really affects the > > small counter values. The other place where saving/merging the histogram

[Patch ARM] Update the test case to differ movs and lsrs for ARM mode and non-ARM mode

Hi, Due to the impact of ARM UAL, the Thumb1 and Thumb2 mode use LSRS instruction while the ARM mode uses MOVS instruction. So the following case is updated accordingly. Is it OK to trunk? BR, Terry 2012-08-21 Terry Guo * gcc.target/arm/combine-movs.c: Check movs for ARM mode

Re: Build static libgcc with hidden visibility even with --disable-shared

On Tue, Aug 21, 2012 at 5:33 PM, Joseph S. Myers wrote: > > 2012-08-21 Joseph Myers > > * Makefile.in (vis_hide, gen-hide-list): Do not make definitions > depend on --enable-shared. > ($(lib1asmfuncs-o)): Use %.vis files independent of > --enable-shared. >

Build static libgcc with hidden visibility even with --disable-shared

As discussed in , it is desirable for the libgcc build with inhibit_libc defined and --disable-shared to be similar enough to that build without inhibit_libc and --enable-shared to be usable to build glibc, producing the same results as if gl

Re: [PATCH] Combine location with block using block_locations

On Tue, Aug 21, 2012 at 6:25 AM, Richard Guenther wrote: > On Mon, Aug 20, 2012 at 3:18 AM, Dehao Chen wrote: >> ping > > Conceptually I like the change. Can a libcpp maintainer please have a 2nd > look? > > Dehao, did you do any compile-time and memory-usage benchmarks? I don't have a memo

Re: [SH] PR 39423 - Add support for SH2A movu.w insn

Oleg Endo wrote: > This adds support for SH2A's movu.w insn for memory addressing cases as > described in the PR. > Tested on rev 190546 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > > and no new failures. > OK? OK. R

Re: [SH] Use more multi-line asm outputs

Oleg Endo wrote: > This mainly converts the asm outputs to multi-line strings and uses tab > chars instead of '\\t' in the asm strings, in the hope to make stuff > easier to read and a bit more consistent. > Tested on rev 190546 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,

Re: [PATCH, ARM] Don't pull in unwinder for 64-bit division routines

On 17 August 2012 07:29, Julian Brown wrote: > On Thu, 16 Aug 2012 19:56:52 +0100 > Ramana Radhakrishnan wrote: > >> On 07/24/12 13:27, Julian Brown wrote: >> > On Fri, 20 Jul 2012 11:15:27 +0100 >> > Julian Brown wrote: >> > >> >> Anyway: this revised version of the patch removes the strange >>

[Patch, Fortran, committed] free gfc_code of EXEC_END_PROCEDURE

Background: There is currently a memory leakage cleanup in the middle end – and fixing PR 54332 would probably have been also easier without FE leaks. I think we should join in an try to remove some leakage - and try to not introduce new ones. * * * Committed: For EXPR_END_PROCEDURE, I have

Re: [PATCH] Allow dg-skip-if to use compiler flags specified through set_board_info cflags

On Aug 11, 2012, at 10:39 AM, Senthil Kumar Selvaraj wrote: > This patch allows cflags set in board config files using > "set_board_info cflags" to be used in the selectors of > dg-skip-if and other dejagnu commands that use the check-flags > proc. Ok.

Another merge from gcc 4.7 branch to gccgo branch

I merged revision 190574 from the gcc 4.7 branch to the gccgo branch. Ian

PATCH: PR middle-end/54332: [4.8 Regression] 481.wrf in SPEC CPU 2006 takes > 10GB memory to compile

Hi, This patch restores df_free_collection_rec call inside the insn traversal loop and removes the stack allocation check in vec_reserve. It has been approved in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54332#c25 It has been tested on Linux/x86-64 and checked in. Thanks. H.J. --- 2012-08-

libgcc patch committed: Increase non-split stack space

When a -fsplit-stack function calls a non-split-stack function, the gold linker automatically redirects the call to __morestack to call __morestack_non_split instead. I wrote __morestack_non_split to always allocate at least 0x4000 bytes. However, that was unclear thinking; 0x4000 bytes is suffic

[PATCH] Fix some leaks and one uninitialized var read

Hi! The recent change in find_assert_locations from XCNEWVEC to XNEWVEC caused a valgrind warning, because bb_rpo[ENTRY_BLOCK] used to be accessed, but was never initialized. Fixed by ignoring edges from ENTRY_BLOCK altogether. The rest are a couple of memory leak fixes. Bootstrapped/regtested

Re: [PATCH, MIPS] fix MIPS16 jump table overflow

Sandra Loosemore writes: > In config/mips/mips.h, there is presently this comment: > > /* ??? 16-bit offsets can overflow in large functions. */ > #define TARGET_MIPS16_SHORT_JUMP_TABLES TARGET_MIPS16_TEXT_LOADS > > A while ago we had a bug report where a big switch statement did, in > fact, ove

[PATCH] fix wrong-code bug for -fstrict-volatile-bitfields

This patch is a followup to the addition of support for -fstrict-volatile-bitfields (required by the ARM EABI); see this thread http://gcc.gnu.org/ml/gcc-patches/2010-10/msg01889.html for discussion of the original patch. That patch only addressed the behavior when extracting the value of a v

Re: patch for machine independent rtl section to hide case statements for different types of constants.

Kenneth Zadeck writes: > I named it this way CASE_CONST_SCALAR_INTEGER because i need to > introduce in the next patch a predicate that looks like > > /* Predicate yielding true iff X is an rtx for a integer const. */ > #if TARGET_SUPPORTS_WIDE_INT == 1 > #define CONST_INTEGER_P(X) \ >(CONST

[patch] two more bitmap obstacks

Hello, Two more bitmap obstacks, this time in tree-ssa-coalesce.c. The advantage isn't so much in having the bitmaps on the non-default obstack, but more in that the bitmaps can be free'ed all at once by simply releasing the obstack. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk?

[lra] patch to remove -flra option

The following patch mostly removes -flra option by defining a machine-dependent hook lra_p. If the hook returns true, LRA is used. Otherwise, reload pass is used. By default the hook returns false. It returns true for 8 targets, lra was ported (i386, rs6000, arm, s390, ia64, sparc, mips, pa

Re: C++ PATCH for c++/51675 (more constexpr unions)

On Wed, Feb 8, 2012 at 1:23 AM, Jason Merrill wrote: > More traffic on PR 51675 demonstrates that my earlier patch didn't fix the > whole problem. This patch improves handling of user-defined constructors. > > Tested x86_64-pc-linux-gnu, applying to trunk. This caused: http://gcc.gnu.org/bugzil

Re: [wwwdocs] Document Runtime CPU detection builtins

Committed after making the changes. One small problem, I am not sure how to fix this: The hyper link I referenced is : http://gcc.gnu.org/onlinedocs/gcc/X86-Built_002din-Functions.html#X86-Built_002din-Functions whereas the committed changes.html is pointing to: http://gcc.gnu.org/onlinedocs/gcc

Re: [PATCH] Add working-set size and hotness information to fdo summary (issue6465057)

On Tue, Aug 21, 2012 at 12:34 AM, Jan Hubicka wrote: >> Teresa has done some tunings for the unroller so far. The inliner >> tuning is the next step. >> >> > >> > What concerns me that it is greatly inaccurate - you have no idea how many >> > instructions given counter is guarding and it can diffe

Re: patch for machine independent rtl section to hide case statements for different types of constants.

I am certainly not going to check it in if there are any issues with the patch. However, this was basically a trivial lexicographical cleanup, and if no one has any comments on it after a reasonable amount of time, then i do feel this is ok.Obviously if anyone has any comments. that is co

Re: [Patch,testsuite] Break gcc.dg/fixed-point/convert.c into manageable parts

On Aug 21, 2012, at 4:32 AM, Georg-Johann Lay wrote: > The patch breaks up convert.c in parts so that an AVR ATmega103 device > with 128KiB for executable code (.text + .data + .rodata) can run them. > > Ok for trunk? Ok, but watch out for any comments from the fixed-point or the C front-end fol

Re: patch for machine independent rtl section to hide case statements for different types of constants.

it would have been tough without the second snippit On 08/21/2012 01:02 PM, Richard Sandiford wrote: Richard Sandiford writes: switch (code) { -case CONST_INT: -case CONST_DOUBLE: -case CONST_FIXED: +CASE_CONST_UNIQUE: case SYMBOL_REF: case CONST: cas

Re: patch for machine independent rtl section to hide case statements for different types of constants.

Richard Sandiford writes: >>switch (code) >> { >> -case CONST_INT: >> -case CONST_DOUBLE: >> -case CONST_FIXED: >> +CASE_CONST_UNIQUE: >> case SYMBOL_REF: >> case CONST: >> case LABEL_REF: > > This was suppsoed to be CASE_CONST_ANY too. The omission of CONS

Re: patch for machine independent rtl section to hide case statements for different types of constants.

Kenneth Zadeck writes: > I plan to commit this in a few days unless someone has some comments. > This is a mostly trivial patch and the changes from that are Richard > Sandiford's and he is an rtl maintainer. Please don't do this. Patches need to be sent for review in their final form. Obvi

Re: [Patch,AVR] PR54222: Add fixed point support

2012/8/13 Georg-Johann Lay : > Denis Chertykov wrote: >> 2012/8/11 Georg-Johann Lay : >>> Weddington, Eric schrieb: > From: Georg-Johann Lay > > > The first step would be to bisect and find the patch that lead to > PR53923. It was not a change in the avr BE, so the question goe

Merge from gcc 4.7 branch to gccgo branch

I've merged gcc 4.7 branch revision 190560 to the gccgo branch. Ian

Re: [google/gcc-4_7] Fix regression - SUBTARGET_EXTRA_SPECS overridden by LINUX_GRTE_EXTRA_SPECS

Hi Jing, the crosstool test passed. You can start the review, thanks! -Han On Wed, Aug 15, 2012 at 3:11 PM, Han Shen(沈涵) wrote: > Hi Jing, ping? > > On Mon, Aug 13, 2012 at 10:58 AM, Han Shen(沈涵) wrote: >> Hi, the google/gcc-4_7 fails to linking anything (on x86-generic), by >> looking into spec

Re: Reproducible gcc builds, gfortran, and -grecord-gcc-switches

On Tue, 21 Aug 2012, Simon Baldwin wrote: > Index: gcc/doc/options.texi > === > --- gcc/doc/options.texi (revision 190535) > +++ gcc/doc/options.texi (working copy) > @@ -468,4 +468,8 @@ of @option{-@var{opt}}, if not explic

RE: [AARCH64] [PATCH 2/3] AArch64 Port

> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- > ow...@gcc.gnu.org] On Behalf Of Joseph S. Myers > Sent: 25 May 2012 15:24 > To: Marcus Shawcroft > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [AARCH64] [PATCH 2/3] AArch64 Port > > On Fri, 25 May 2012, Marcus

PATCH: PR target/54347: REAL_VALUE_TO_TARGET_LONG_DOUBLE shouldn't be used in i386

Hi, long double may not be 80-bit on i386. We can't use REAL_VALUE_TO_TARGET_LONG_DOUBLE for XFmode. This patch replaces REAL_VALUE_TO_TARGET_LONG_DOUBLE with real_to_target. OK to install? Thanks. H.J. --- 2012-08-21 H.J. Lu PR target/54347 * config/i386/i386.c (ix86_spli

RE: [AARCH64] [PATCH 1/3] AArch64 Port

Hi, Thanks for the feedback. I respond here to the remaining issues: > > Index: gcc/doc/extend.texi > > === > > --- gcc/doc/extend.texi (revision 187870) > > +++ gcc/doc/extend.texi (working copy) > > @@ -935,7 +935,8 @@ > >

Re: [PATCH] Add working-set size and hotness information to fdo summary (issue6465057)

> The issue here is holding lock for all the files (that can be many) versus > number of locks limits & possibilities for deadlocking (mind that updating > may happen in different orders on the same files for different programs built > from same objects) lockf typically has a deadlock detector, an

[PATCH][4.7] Backport recent heap leak fixes

This backports the obvious heap leak fixes that have accumulated sofar. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-08-21 Richard Guenther Backport from mainline 2012-08-16 Richard Guenther PR middle-end/54146 * tree-ssa-lo

Re: [PATCH] Combine location with block using block_locations

On Mon, Aug 20, 2012 at 3:18 AM, Dehao Chen wrote: > ping Conceptually I like the change. Can a libcpp maintainer please have a 2nd look? Dehao, did you do any compile-time and memory-usage benchmarks? Thanks, Richard. > Thanks, > Dehao > > On Tue, Aug 14, 2012 at 10:13 AM, Dehao Chen wr

Re: [wwwdocs] Document Runtime CPU detection builtins

On 2012-08-20 22:41 , Sriraman Tallam wrote: Hi Gerald / Diego, I have made all the mentioned changes. I also shortened the description like Diego mentioned by removing all the strings but kept the caveats. I have not added a reference to the documentation because i do not know what link t

Re: patch for machine independent rtl section to hide case statements for different types of constants.

Now that I have had a chance to talk to Richard, I have now done everything that he requested in his email. Here is the new patch and changelog. Everything was tested on x86-64. 2012-08-21 Kenneth Zadeck * alias.c (rtx_equal_for_memref_p): Convert constant cases. * combine.c (find_

Re: Reproducible gcc builds, gfortran, and -grecord-gcc-switches

On 20 August 2012 16:45, Joseph S. Myers wrote: > > On Mon, 20 Aug 2012, Simon Baldwin wrote: > > > > OPT_* for Fortran options only exist when the Fortran front-end is in the > > > source tree (whether or not enabled). I think we try to avoid knowingly > > > breaking use cases where people remov

[Patch,testsuite] Break gcc.dg/fixed-point/convert.c into manageable parts

Just as the title says: gcc.dg/fixed-point/convert.c is much too big to run on embedded targets like AVR. Note that embedded systems are a main audience of ISO/IEC TR 18037, and that these systems might have limited resources. The original convert.c inflates to thousands of functions and set -O0.

Re: [PATCH] Set current_function_decl in {push,pop}_cfun and push_struct_function

On Tue, Aug 21, 2012 at 1:27 PM, Martin Jambor wrote: > On Wed, Aug 15, 2012 at 05:21:04PM +0200, Martin Jambor wrote: >> Hi, >> >> On Fri, Aug 10, 2012 at 04:57:41PM +0200, Eric Botcazou wrote: >> > > - ada/gcc-interface/utils.c:rest_of_subprog_body_compilation calls >> > > dump_function which

Re: [PATCH] Set current_function_decl in {push,pop}_cfun and push_struct_function

On Wed, Aug 15, 2012 at 05:21:04PM +0200, Martin Jambor wrote: > Hi, > > On Fri, Aug 10, 2012 at 04:57:41PM +0200, Eric Botcazou wrote: > > > - ada/gcc-interface/utils.c:rest_of_subprog_body_compilation calls > > > dump_function which in turns calls dump_function_to_file which calls > > > push

Re: [PATCH] Document tree.h flags more, fixup valgrind alloc-pool.c

On Tue, 21 Aug 2012, Richard Guenther wrote: > > Testing in progress. > > Richard. > > 2012-08-21 Richard Guenther > > * alloc-pool.c (pool_alloc): Fix valgrind annotation. > * tree.h: Complete flags documentation. > (CLEANUP_EH_ONLY): Check documented allowed tree codes.

[PATCH] Document tree.h flags more, fixup valgrind alloc-pool.c

Testing in progress. Richard. 2012-08-21 Richard Guenther * alloc-pool.c (pool_alloc): Fix valgrind annotation. * tree.h: Complete flags documentation. (CLEANUP_EH_ONLY): Check documented allowed tree codes. Index: gcc/alloc-pool.c ===

[PATCH] Fix more leaks

This fixes a few more heap leaks. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2012-08-21 Richard Guenther * tree-ssa-loop-im.c (tree_ssa_lim_finalize): Properly free the affine expansion cache. * tree-ssa-dom.c (free_expr_hash_elt_contents

[SH] Use more multi-line asm outputs

Hello, This mainly converts the asm outputs to multi-line strings and uses tab chars instead of '\\t' in the asm strings, in the hope to make stuff easier to read and a bit more consistent. Tested on rev 190546 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-

[C++ PATCH] Add overflow checking to __cxa_vec_new[23]

I don't think there are any callers out there, but let's fix this for completeness. A compiler emitting code to call this function would still have to perform overflow checks for the new T[n][m] case, so this interface is not as helpful as it looks at first glance. Tested on x86_64-redhat-li

Re: [PATCH] Add valgrind support to alloc-pool.c

On Sat, Aug 18, 2012 at 9:56 AM, Richard Guenther wrote: > On Sat, Aug 18, 2012 at 6:17 AM, Andrew Pinski wrote: >> Hi, >> I implemented this patch almost 6 years ago when the df branch was >> being worked on. It adds valgrind support to alloc-pool.c to catch >> cases of using memory after fr

Re: [PATCH][RFC] Move TREE_VEC length and SSA_NAME version into tree_base

On 21 August 2012 10:58, Richard Guenther wrote: > Index: trunk/gcc/tree.h > === > *** trunk.orig/gcc/tree.h 2012-08-20 12:47:47.0 +0200 > --- trunk/gcc/tree.h2012-08-21 10:32:47.717394657 +0200 > *** enu

Re: [PATCH][RFC] Move TREE_VEC length and SSA_NAME version into tree_base

On Mon, 20 Aug 2012, Richard Guenther wrote: > > This shrinks TREE_VEC from 40 bytes to 32 bytes and SSA_NAME from > 80 bytes to 72 bytes on a 64bit host. Both structures suffer > from the fact they need storage for an integer (length and version) > which leaves unused padding. Both data struct

[SH] PR 39423 - Add support for SH2A movu.w insn

Hello, This adds support for SH2A's movu.w insn for memory addressing cases as described in the PR. Tested on rev 190546 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK? Cheers, Oleg ChangeLog:

Re: Fix Solaris 9/x86 bootstrap

On Tue, Aug 21, 2012 at 10:53 AM, Rainer Orth wrote: > Solaris 9/x86 bootstrap was broken after the cxx-conversion merge: > > In file included from /vol/gcc/src/hg/trunk/local/gcc/gengtype.c:957: > /vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected identifier > before n > umeric constan

Fix Solaris 9/x86 bootstrap

Solaris 9/x86 bootstrap was broken after the cxx-conversion merge: In file included from /vol/gcc/src/hg/trunk/local/gcc/gengtype.c:957: /vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected identifier before n umeric constant /vol/gcc/src/hg/trunk/local/gcc/rtl.def:347: error: expected '}'

Re: [PATCH, ARM] Don't pull in unwinder for 64-bit division routines

On Fri, Aug 17, 2012 at 9:13 AM, Ian Lance Taylor wrote: > > Looks fine to me. > > Ian Will backport to arm/embedded-4_7-branch. No sure if appropriate for 4.7 branch since it is not a stability problem. - Joey

Re: [PATCH] Add working-set size and hotness information to fdo summary (issue6465057)

> Teresa has done some tunings for the unroller so far. The inliner > tuning is the next step. > > > > > What concerns me that it is greatly inaccurate - you have no idea how many > > instructions given counter is guarding and it can differ quite a lot. Also > > inlining/optimization makes working

Re: [PATCH] Add working-set size and hotness information to fdo summary (issue6465057)

On Mon, Aug 20, 2012 at 11:33 PM, Jan Hubicka wrote: >> >> This is useful for large applications with a long tail. The >> instruction working set for those applications are very large, and >> inliner and unroller need to be aware of that and good heuristics can >> be developed to throttle aggressi