[google] Use R_ARM_GOT_PREL to simplify global address loading from GOT (issue4433079)

Hi This patch is for google/main. The bug http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43129 describes how arm backend generates ineffecient code to load global variable's address in terms of both code size and performance when option -fpic is specified. It also describes how it can be improved by

Re: Toplevel cleanup: disable Java when libffi not supported

On 04/27/2011 06:33 PM, Joseph S. Myers wrote: This patch, relative to a tree with applied, continues the cleanup of toplevel cases relating to disabling Java or Java libraries by arranging for Java to be disabled (via unsupported_languag

Re: Toplevel cleanup: split out libgcj disabling

On 04/27/2011 05:50 PM, Joseph S. Myers wrote: Continuing the toplevel cleanups separating the cases disabling different subdirectories - on the basis that eventually this information should come from the subdirectories, and that as previously discussed much of the default disabling of ${libgcj}

Re: [C++ Patch] PR 48771

On 04/27/2011 04:50 AM, Paolo Carlini wrote: Tested x86_64-linux. Ok for mainline? Yes. (not sure if it qualifies for the branch or we want to reconsider it in a couple of weeks?!?) The latter, I think. Jason

Re: C++ PATCH for libstdc++/48760 (list-initialization of complex)

On 4/27/11, Jason Merrill wrote: > On 04/27/2011 09:36 PM, Jason Merrill wrote: >> A a = { 1, 2 }; >> >> a.c is initialized to 1+0i rather than 1+2i as it has been previously, > > This should have said "a.c is initialized to 1+0i, as it has been > previously, rather than 1+2i, as it would be if it

[patch, ARM] Fix PR42017, LR not used in leaf functions

Hi, this patch tries to solve the problem of the LR register not being used in leaf functions on ARM. Looking at the dumps, it shows that register 14 (lr) conflicts with all allocnos throughout the entire leaf procedure. A little digging shows that lr is present in the OBJECT_CONFLICT_HARD_REGS()

Re: restore bootstrap with a C++ compiler

Andrew Pinski writes: | On Wed, Apr 27, 2011 at 6:48 PM, Gabriel Dos Reis wrote: | > A local `extern' declaration does not give the entity an external linkage | > -- irrespective of the linkage of the function enclosing the declaration. | > It just makes the name locally available for name looku

Here is an updated patch. (issue4438079)

Hi Diego, Thanks for the quick feedback. Here is a an updated version of the patch. 2011-04-27 Sharad Singhai ChangeLog.google-main * params.def: Add new parameters to control peeling. * tree-ssa-loop-ivcanon.c (try_unroll_loop_completely): Use different peelin

Re: restore bootstrap with a C++ compiler

On Wed, Apr 27, 2011 at 6:48 PM, Gabriel Dos Reis wrote: > A local `extern' declaration does not give the entity an external linkage > -- irrespective of the linkage of the function enclosing the declaration. > It just makes the name locally available for name lookup > purpose.   A variable declar

Re: C++ PATCH for libstdc++/48760 (list-initialization of complex)

On 04/27/2011 09:36 PM, Jason Merrill wrote: A a = { 1, 2 }; a.c is initialized to 1+0i rather than 1+2i as it has been previously, This should have said "a.c is initialized to 1+0i, as it has been previously, rather than 1+2i, as it would be if it were an aggregate". Jason

Re: C++ PATCH for libstdc++/48760 (list-initialization of complex)

Jason Merrill writes: | In the discussion of 48760, Gaby suggested that we allow | list-initialization of built-in complex numbers. This made a lot of | sense to me, so I've gone ahead and implemented it. | | Basically it works as though complex were an aggregate except for one | thing: for bac

Re: restore bootstrap with a C++ compiler

Andrew Pinski writes: | On Wed, Apr 27, 2011 at 6:30 PM, Gabriel Dos Reis wrote: | > | > Richard, | > | > Your recent patch broke --enable-build-with-cxx because internal-fn.h | > was using good 'ol C-style name lookup trick that behaves badly with | > C++ linkage rules.  Fixed thusly.  Applied

C++ PATCH for c++/40975 (ICE with new in default arg)

When we expand a default argument, we need to rebuild any TARGET_EXPRs so that we don't run into issues with invalid tree sharing. But copy_tree_r isn't prepared to deal with the STATEMENT_LIST created by array initialization. Since I already introduced VEC_INIT_EXPR in the constexpr work, it

[PATCH][libstdc++] Also install cxxabi_tweaks.h in freestanding mode.

Hi, This patch causes cxxabi_tweaks.h to be installed in freestanding mode. We need this because libsupc++ installs cxxabi.h, which includes cxx_tweaks.h. -Doug 2011-04-27 Doug Kwan * include/Makefile.am (install-freestanding-headers): Also install cxxabi_tweaks.h.

C++ PATCH for libstdc++/48760 (list-initialization of complex)

In the discussion of 48760, Gaby suggested that we allow list-initialization of built-in complex numbers. This made a lot of sense to me, so I've gone ahead and implemented it. Basically it works as though complex were an aggregate except for one thing: for backwards compatibility, we never a

Re: restore bootstrap with a C++ compiler

On Wed, Apr 27, 2011 at 6:30 PM, Gabriel Dos Reis wrote: > > Richard, > > Your recent patch broke --enable-build-with-cxx because internal-fn.h > was using good 'ol C-style name lookup trick that behaves badly with > C++ linkage rules.  Fixed thusly.  Applied as obvious. Can you explain some more

restore bootstrap with a C++ compiler

Richard, Your recent patch broke --enable-build-with-cxx because internal-fn.h was using good 'ol C-style name lookup trick that behaves badly with C++ linkage rules. Fixed thusly. Applied as obvious. -- Gaby 2011-04-27 Gabriel Dos Reis * internal-fn.h (internal_fn_name_array): De

[PATCH 4/4] Docs: extend.texi: Reword and rearrange attribute node introductions

The rearrangement performed in the previous patch left the text in somewhat of an inconsistent state in terms of the flow of concepts; this patch improves the flow of concepts to something more sane by editing the introductions to these nodes: Attribute Syntax Type Attributes Variable Attrib

[PATCH 2/4] Docs: extend.texi: Remove trailing blanks from lines

sed -i "s/[ $(printf '\t')]\{1,\}\$//" trunk/gcc/doc/extend.texi --- trunk/gcc/doc/extend.texi | 82 ++-- 1 files changed, 41 insertions(+), 41 deletions(-) diff --git a/trunk/gcc/doc/extend.texi b/trunk/gcc/doc/extend.texi index c154958..cdbf69f 100644

[PATCH 1/4] Docs: extend.texi: Add missing semicolon for consistency

--- trunk/gcc/doc/extend.texi |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/trunk/gcc/doc/extend.texi b/trunk/gcc/doc/extend.texi index eddff95..c154958 100644 --- a/trunk/gcc/doc/extend.texi +++ b/trunk/gcc/doc/extend.texi @@ -3997,7 +3997,7 @@ @smallexample __attrib

[PATCH 0/4] Docs: extend.texi

See the following emails for a few inlined patches to /trunk/gcc/doc/extend.texi (revision 172911): [1] Docs: extend.texi: Add missing semicolon for consistency [2] Docs: extend.texi: Remove trailing blanks from lines [3] Docs: extend.texi: Rearrange nodes; no text was removed or added [4]

Re: [google] Use different peeling parameters with available profile (issue4438079)

On Wed, Apr 27, 2011 at 20:52, Sharad Singhai wrote: > This patch adds new parameters to control peeling when profile > feedback information is available. > > For google/main. > > Tested: >  bootstrapped on x86_64. > Looks OK with a couple of minor nits below. > 2011-04-27  Sharad Singhai   > >

[google] Use different peeling parameters with available profile (issue4438079)

This patch adds new parameters to control peeling when profile feedback information is available. For google/main. Tested: bootstrapped on x86_64. 2011-04-27 Sharad Singhai * gcc/params.def: Add new parameters to control peeling. * gcc/tree-ssa-loop-ivcanon.c (try_unroll_lo

Re: trial fix to null pointer free

Ok -- false alarm ... David On Wed, Apr 27, 2011 at 5:30 PM, Nathan Froyd wrote: > On Wed, Apr 27, 2011 at 04:23:42PM -0700, Xinliang David Li wrote: >> This can happen when the module does not have function bodies. >> >> -   VEC_free (cgraph_node_ptr, heap, cgraph_node_map); >> -   cgraph_node_

Re: trial fix to null pointer free

On Wed, Apr 27, 2011 at 19:23, Xinliang David Li wrote: > This can happen when the module does not have function bodies. > > > Ok after testing? > > Thanks, > > David > > > 2011-04-27  Xinliang David Li   > >        * value-prof.c (del_node_map): Do not free >        null pointer. > Index: value-

Re: trial fix to null pointer free

On Wed, Apr 27, 2011 at 04:23:42PM -0700, Xinliang David Li wrote: > This can happen when the module does not have function bodies. > > - VEC_free (cgraph_node_ptr, heap, cgraph_node_map); > - cgraph_node_map = NULL; > + if (cgraph_node_map) > +VEC_free (cgraph_node_ptr, heap, cgraph_node_

[google] Disable node map creation for lipo mode

Committed the trial change r173049 that disables cgraph node map creation for lipo -- the latter uses Global id map. David

trial fix to null pointer free

This can happen when the module does not have function bodies. Ok after testing? Thanks, David 2011-04-27 Xinliang David Li * value-prof.c (del_node_map): Do not free null pointer. Index: value-prof.c === ---

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

On Wednesday 27 April 2011 23:10:14 Steve Kargl wrote: > On Wed, Apr 27, 2011 at 10:54:37PM +0200, Mikael Morin wrote: > > On Wednesday 27 April 2011 01:06:26 Steve Kargl wrote: > > > It's an 'if -- else if' construct. If gfc_notify_std == FAILURE, then > > > the error message is issues and the fu

Re: [PATCH] Add FRE pass to the early optimization pipeline

On Wed, Mar 23, 2011 at 7:17 AM, Richard Guenther wrote: > > For quite some releases we'd wanted some (memory) CSE optimization > during early opts.  Currently we rely heavily on SRA which doesn't > always fulfill the "make code only smaller" promises which is > what early opts should do to improv

Re: [PATCH] C++0x, teach the parser to parse virt-specifier-seq for member functions

On Wed, Apr 27, 2011 at 4:58 PM, Ville Voutilainen wrote: > On 28 April 2011 00:56, Gabriel Dos Reis > wrote: >>> Seems to fall over itself in a bunch of places. C++ bootstrap smells >>> like a red herring >>> to me at this point. >> Hmm, do you mean GCC fails to bootstrap with C++ a compiler be

Re: [PATCH] C++0x, teach the parser to parse virt-specifier-seq for member functions

On 28 April 2011 00:56, Gabriel Dos Reis wrote: >> Seems to fall over itself in a bunch of places. C++ bootstrap smells >> like a red herring >> to me at this point. > Hmm, do you mean GCC fails to bootstrap with C++ a compiler before your mods? I did make distclean, followed by CC=g++ GCC=g++ CF

Re: [PATCH] C++0x, teach the parser to parse virt-specifier-seq for member functions

On Wed, Apr 27, 2011 at 4:05 PM, Ville Voutilainen wrote: > On 27 April 2011 10:14, Ville Voutilainen wrote: >> The -fpermissive because there are conversions from literals to char*, >> and c++ nowadays doesn't like that. I'm not at the build machine right >> now, so I'll check >> later how it fa

Re: [Patch, Fortran] PRs 48112/48279 - fix interface related regressions

On Tuesday 26 April 2011 23:41:17 Tobias Burnus wrote: > This patch fixes three issues, (a) and (b) are 4.6/4.7 regressions. > > a) PR48112: Due to incomplete resolution, there was an ICE when writing > the module file. Solution: Back-out the patch which introduced the > incomplete resolution - an

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

On Wed, Apr 27, 2011 at 10:54:37PM +0200, Mikael Morin wrote: > On Wednesday 27 April 2011 01:06:26 Steve Kargl wrote: > > > > It's an 'if -- else if' construct. If gfc_notify_std == FAILURE, then > > the error message is issues and the function returns. If it is TRUE, > > then there should be

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

On Wednesday 27 April 2011 01:06:26 Steve Kargl wrote: > > > Index: primary.c > > > === > > > --- primary.c (revision 172974) > > > +++ primary.c (working copy) > > > @@ -541,6 +541,17 @@ match_real_constant (gfc_expr **result, > >

[google] Moved ChangeLog entries to their proper place (issue4453047)

I moved some of your ChangeLog entries to different ChangeLog files. In general, each sub-directory has its own ChangeLog file, so please use those. This is not completely standard, unfortunately. The rule of thumb is to check whether the directory you are modifying has a ChangeLog file. If so,

[Patch, Fortran, committed] PR 48788 - Avoid segfault after gfc_error for whole-file diagnostic

First, I'd be glad if someone could review my patch at http://gcc.gnu.org/ml/fortran/2011-04/msg00271.html * * * Secondly, the attached patch fixes a segfault after printing an error about the invalid function-return type. It has been committed as Rev. 173048. As it is a 4.6/4.7 regression,

Re: [patch gimplifier]: Do folding on truth and/or trees

On Wed, Apr 27, 2011 at 10:26 PM, Richard Guenther wrote: > On Wed, Apr 27, 2011 at 10:03 PM, Kai Tietz wrote: >> Hello, >> >> this patch adds the ability to gimple-fold to operate for truth and/or >> operations >> on folded binary-and/or optimized truth trees - as done by fold-const.  As >> fol

Re: [patch gimplifier]: Do folding on truth and/or trees

On Wed, Apr 27, 2011 at 10:03 PM, Kai Tietz wrote: > Hello, > > this patch adds the ability to gimple-fold to operate for truth and/or > operations > on folded binary-and/or optimized truth trees - as done by fold-const.  As > fold > converts trivial operations like (A && B) to (A & B) != 0, in m

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

>> set_debug_level should not use global state; this needs to check >> opts->x_debug_info_level (not the global debug_info_level) and set >> opts->x_generate_debug_line_table. > > Oops, missed that. Thanks! I've uploaded the revised patch to http://codereview.appspot.com/4440072 -- should I also p

Re: [patch gimplifier]: Do folding on truth and/or trees

On Wed, Apr 27, 2011 at 10:03:34PM +0200, Kai Tietz wrote: > --- /dev/null 1970-01-01 00:00:00.0 + > +++ gcc/gcc/testsuite/gcc.dg/binop-tand1.c2011-04-27 21:31:19.276726900 > +0200 > @@ -0,0 +1,14 @@ > +/* { dg-do compile } */ > +/* { dg-options "-O2 -fdump-tree-optimized" } */ > +

[patch gimplifier]: Do folding on truth and/or trees

Hello, this patch adds the ability to gimple-fold to operate for truth and/or operations on folded binary-and/or optimized truth trees - as done by fold-const. As fold converts trivial operations like (A && B) to (A & B) != 0, in most cases further folding of truth and/or trees wasn't done. foldi

Re: [patch, libgfortran] Fix numerous formatting bugs

On Wed, Apr 27, 2011 at 08:53, Janne Blomqvist wrote: > On Wed, Apr 27, 2011 at 07:09, Jerry DeLisle wrote: >> On 04/25/2011 07:36 AM, Janne Blomqvist wrote: >>> >>> On Mon, Apr 25, 2011 at 14:44, Jerry DeLisle >>>  wrote: On 04/25/2011 03:48 AM, Janne Blomqvist wrote: > > Now,

Re: [google] GRTE static link spec (issue4426065)

On Wed, Apr 27, 2011 at 15:13, Ollie Wild wrote: > ChangeLog: > 2011-04-27  Ollie Wild   > >        * config.gcc: Add GRTE detection. >        * config/i386/linux.h (LINUX_GRTE_EXTRA_SPECS): New macro. >        (SUBTARGET_EXTRA_SPECS): Add LINUX_GRTE_EXTRA_SPECS. >        * config/i386/linux64.h

Ping^2 Re: Don't use linux.h for non-Linux targets

Ping^2. This patch is still pending review (except insofar as it relates to GNU Hurd). -- Joseph S. Myers jos...@codesourcery.com

[google] GRTE static link spec (issue4426065)

commit 44417f1103b0823f950f539a38a5539b8ff74330 Author: Ollie Wild Date: Mon Apr 25 21:55:44 2011 -0400 When using GRTE (libc) and linking statically, instead of just -lc add: -lc -lnss_borg -lnss_cache -lnss_dns -lnss_files -lresolv to the link by modifying LIB_SPEC

Re: [patch, 4.5] Fix PR middle-end/43085 (make profiledbootstrap crashes due to dataflow bug)

On Wed, Apr 27, 2011 at 6:27 PM, Ulrich Weigand wrote: > Hello, > > PR middle-end/43085 is a crash of cc1plus when building libstdc++ during > make profiledbootstrap on the 4.5 branch.  The crash is caused by a > mis-compile of the add_method routine with -fprofile-use due to a > dataflow bug duri

Re: [patch] Split Parse Timevar (issue4378056)

This discussion continues in the thread "[patch] Split Parse Timevar (rev 2) (issue4433076)" which has a new uploaded patch. On 4/23/11, Jason Merrill wrote: > On 04/22/2011 06:41 PM, Lawrence Crowl wrote: >> On 4/21/11, Jason Merrill wrote: >>> On 04/21/2011 07:17 PM, Lawrence Crowl wrote: > >>

[v3] Add move_if_noexcept

Hi, tested x86_64-linux, committed to mainline. Thanks, Paolo. /// 2011-04-27 Paolo Carlini * include/bits/move.h (move_if_noexcept): Add. * testsuite/20_util/move_if_noexcept/requirements/ explicit_instantiation.cc: New. * testsuite/20_util/m

[PATCH, i386]: Merge and rename SSE/AVX mode attributes

Hello! Several mode SSE/AVX mode attributes can be merged, most notably various SSE insn suffixes for various modes. Attached patch also renames several mode attributes. 2011-04-27 Uros Bizjak * config/i386/i386.md (ssemodesuffix): Merge with ssevecsize, ssemodefsuffix, ssesc

[PING][PATCH] Change rclass argument type in memory_move_cost function from enum reg_class to reg_class_t.

Hello. Patch ping. > This patch change memory_move_cost function to stop using back end specific > type 'enum reg_class' in favor to reg_class_t. Also this allow do small > cleanup in ia64_register_move_cost function. http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01433.html Anatoly.

[SPARC] Hookize PRINT_OPERAND, PRINT_OPERAND_ADDRESS and PRINT_OPERAND_PUNCT_VALID_P

Hello. This patch removes obsolete PRINT_OPERAND, PRINT_OPERAND_ADDRESS and PRINT_OPERAND_PUNCT_VALID_P macros from SPARC back end in the GCC and introduces equivalent TARGET_PRINT_OPERAND, TARGET_PRINT_OPERAND_ADDRESS and TARGET_PRINT_OPERAND_PUNCT_VALID_P target hooks.  Bootstrapped an

Re: Add inline-analysis predicates to edges

Hi, On Wed, 27 Apr 2011, Jan Hubicka wrote: > *** false_predicate (void) > *** 163,168 > --- 166,195 > } > > > + /* Return true if P is (false). */ > + > + static inline bool > + true_predicate_p (struct predicate *p) Comment doesn't match function. > + { > + r

[pph] Enable nested namespaces (issue4431071)

Add a flag -fpph-dump-tree to dump the namespace trees read and written by PPH. This dump could use some clarification, but it can wait for another day. Replace PPH calls to pushdecl_with_scope with pushdecl_into_namespace. It turns out that pushdecl_with_scope always inserts into the current n

FDO patch -- make ic related vars TLS if target allows

Hi please review the trivial patch below. It reduces race conditions in value profiling. Another trivial change (to initialize function_list struct) is also included. Bootstrapped and regression tested on x86-64/linux. Thanks, David 2011-04-27 Xinliang David Li * tree-profile.c (in

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

>> +  generate_debug_line_table = debug_info_level >= DINFO_LEVEL_NORMAL; > > set_debug_level should not use global state; this needs to check > opts->x_debug_info_level (not the global debug_info_level) and set > opts->x_generate_debug_line_table. Oops, missed that. Thanks! -cary

Re: Toplevel cleanup: split out libgcj disabling

On Apr 27, 2011, at 8:50 AM, Joseph S. Myers wrote: > Continuing the toplevel cleanups separating the cases disabling > different subdirectories I've audited the two patches for darwin and they seem safe. Took me a second to figure out why though.

Re: FDO usability patch -- cfg and lineno checksum

On Wed, Apr 27, 2011 at 10:03 AM, Jan Hubicka wrote: >> >> -       else if (entry->checksum != checksum) >> >> +       else if (entry->lineno_checksum != lineno_checksum >> >> +                || entry->cfg_checksum != cfg_checksum) >> >>           { >> >>             error ("coverage mismatch for

Re: FDO usability patch -- cfg and lineno checksum

> >> -       else if (entry->checksum != checksum) > >> +       else if (entry->lineno_checksum != lineno_checksum > >> +                || entry->cfg_checksum != cfg_checksum) > >>           { > >>             error ("coverage mismatch for function %u while reading > >> execution counters", > >>

Re: FDO usability patch -- cfg and lineno checksum

On Wed, Apr 27, 2011 at 8:54 AM, Jan Hubicka wrote: > Please don't forget about changelogs.. >> Index: tree.c >> === >> --- tree.c    (revision 172802) >> +++ tree.c    (working copy) >> @@ -8513,14 +8513,12 @@ dump_tree_statistics (v

[wwwdocs] Remove three obsolete icons

...for a validator never implemented. Andrea Bocci did some work around this in 2003, but we never completed the effort. validate-empty.png, validate-error.png, and validate-ok.png are the three images. Gerald

Toplevel cleanup: disable Java when libffi not supported

This patch, relative to a tree with applied, continues the cleanup of toplevel cases relating to disabling Java or Java libraries by arranging for Java to be disabled (via unsupported_languages) on targets not supporting libffi, and removing

[patch, 4.5] Fix PR middle-end/43085 (make profiledbootstrap crashes due to dataflow bug)

Hello, PR middle-end/43085 is a crash of cc1plus when building libstdc++ during make profiledbootstrap on the 4.5 branch. The crash is caused by a mis-compile of the add_method routine with -fprofile-use due to a dataflow bug during the final if-conversion pass. For more detailed analysis see th

Re: [PATCH][ARM] RTABI half-precision conversion functions

On 21/04/11 16:58, Joseph S. Myers wrote: You need to add %inherit GCC_4.7.0 GCC_4.6.0 GCC_4.7.0 { } Ok, done. Is this OK now? Andrew 2011-04-27 Andrew Stubbs gcc/ * config/arm/arm.c (arm_init_libfuncs): Change __gnu_f2h_ieee to __aeabi_f2h, __gnu_f2h_alternative to __aeabi_f2h_alt, __

Re: [gomp3.1] Update libgomp.texi

On Wed, Apr 27, 2011 at 06:22:16PM +0200, Tobias Burnus wrote: > Jakub, is the patch OK for the GOMP-3_1-branch? With a suitable ChangeLog entry yes. Jakub

[gomp3.1] Update libgomp.texi

Hi Jakub, hello all, the attached patch updates GOMP-3_1-branch's libgomp.texi for OpenMP 3.1 (current draft). Changes: - Change 3.0 to 3.1, including checking/updating the section numbers - Add omp_in_final - Add OMP_PROC_BIND - Update OMP_NUM_THREADS (now that a list is allowed) - Fix a few sma

Add inline-analysis predicates to edges

Hi, this patch finishes the basic predicate infrastructure of ipa-inline by adding predicates to edges. The predicates are combined, so we can work out very simple cases where function call becmes dead after inlining. In tramp3d we get couple hundred cases like that, so it seems to make differen

Re: FDO usability patch -- cfg and lineno checksum

Please don't forget about changelogs.. > Index: tree.c > === > --- tree.c(revision 172802) > +++ tree.c(working copy) > @@ -8513,14 +8513,12 @@ dump_tree_statistics (void) The crc bits was already reviewed, right? > - el

C++ PATCH for c++/48046 (ambiguity diagnostic regression)

This regression was introduced by the fix for PR 35112, but it turns out that fixing it is a simple matter of moving one line up in a function so that we are committed to the tentative parse before we call cp_parser_lookup_name. Tested x86_64-pc-linux-gnu, applying to trunk and 4.6. Will also

Re: FDO usability patch -- cfg and lineno checksum

Honza, any more comments? Thanks, David On Mon, Apr 25, 2011 at 9:35 AM, Xinliang David Li wrote: > Is the patch ok? > > Thanks, > > David > > On Fri, Apr 22, 2011 at 12:48 PM, Jan Hubicka wrote: >>> Please review the new patch which only implements cfg checksum. >>> >>> The auto version gener

Toplevel cleanup: split out libgcj disabling

Continuing the toplevel cleanups separating the cases disabling different subdirectories - on the basis that eventually this information should come from the subdirectories, and that as previously discussed much of the default disabling of ${libgcj} doesn't actually make sense - this patch separate

[gomp3.1] Fix #pragma omp atomic write

Hi! #pragma omp atomic write RHS can be arbitrary expression (not mentioning the LHS), so it is wrong to parse it just as unary expression. Fixed thusly, tested on x86_64-linux, committed to gomp-3_1-branch. 2011-04-27 Jakub Jelinek * c-parser.c (c_parser_omp_atomic): Parse #pragma om

[PATCH][4/n] Alias housekeeping

This moves PTA changed flag handling during solving from using an sbitmap and an explicitly managed change_count to a bitmap. Way easier to get right and also a requirement for pending work that adds variables on-the-fly during solving (which would require growing the sbitmap). Bootstrapped on x8

Commit: RX: Fix some gcc testsuite failures

Hi Guys, I am applying the patch below to the mainline and 4.6 branch sources. It fixes a few GCC testsuite failures for the RX target, specifically: gcc.dg/20020312-2.c gcc.dg/pr28796-2.c gcc.dg/torture/builtin-cproj-1.c gcc.dg/torture/type-generic-1.c gcc.target/rx/

[google][wwwdocs] Add google/gcc-x_y to svn.html

This patch adds the google release branches to the 'distribution branches' section of svn.html. Committed to wwwdocs. Diego. Index: htdocs/svn.html === RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v retrieving revision 1.160 diff -u

[PATCH] Fortran support for OpenMP 3.1 atomics

Hi! This patch adds Fortran support for OpenMP 3.1 atomics. Tested on x86_64-linux, committed to gomp-3_1-branch. 2011-04-27 Jakub Jelinek * gfortran.h (gfc_statement): Add ST_OMP_END_ATOMIC. (gfc_omp_atomic_op): New enum typedef. (struct gfc_code): Add ext.omp_atomic.

Re: [RFC] Context sensitive inline analysis

> > This may have caused: > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48791 Oops, yes, it is mine. The insertion hook at expansion time is incorrectly called after function is expanded, not before. ipa-prop should deregister itself earlier, but that can be done incrementally. I am testi

Re: fix up hot/cold partitioning on ports that don't have long conditional branches

Ping? On Apr 22, 2011, at 3:01 PM, Mike Stump wrote: > On Apr 22, 2011, at 3:28 AM, Eric Botcazou wrote: >>> This patch fixes up hot/cold partitioning on ports that don't have long >>> conditional branches. I'll note that the entire file has lots of other >>> jump optimizations that are suspect.

Re: (build) Patch to fix cp/cfns.gperf building issues

On Apr 22, 2011, at 3:54 PM, Mike Stump wrote: > On Apr 22, 2011, at 8:12 AM, Nicola Pero wrote: >> This patch fixes a building annoyance that I had when building on a new >> machine (an x86_64 gnu/linux box). >> >> The building failed. It was down to two problems: >> >> * due to how I got a cop

[PATCH, i386]: Remove avx_{vpermilp,vperm2f128}_*_operand predicates

Hello! Mode can't be passed to match_parallel predicates, so instead of calling avx_{vpermilp,vperm2f128}_parallel functions via predicates, we can call them from insn condition directly. 2011-04-27 Uros Bizjak * config/i386/predicates.md (avx_vpermilp_*_operand): Remove. (avx

[PATCH][3/n] Alias housekeeping

As preparation for a fix for PR48764 this removes the need for PTA to allocate and preserve new decls. We still need to allocate UIDs from the decl namespace to have something to point to from points-to sets, but that can be done with allocating them directly. Bootstrapped and tested on x86_64-u

Re: [RFC] Context sensitive inline analysis

On Wed, Apr 27, 2011 at 5:16 AM, Jan Hubicka wrote: > Hi, > I don't really have testcase for the HP nor AIX ICE, however I can reproduce > same ICE when I hack x86 to > not use ctors/dtors.  This patch fixes it - the problem is that ipa-prop > ignore newly added functions > (the global ctor buil

Re: [build, lto] Only accept -fuse-linker-plugin if linker supports -plugin (PR lto/46944)

Hello Ralf, > * Rainer Orth wrote on Tue, Apr 26, 2011 at 05:28:19PM CEST: >> it's been a week since I answered your questions on this patch. Could >> you please have a look? > > Sorry for the delay. I'm practically AFK until the weekend or maybe > next weekend, whenever I have connectivity agai

[patch, spu] Fix sched_emit_insn after scheduler changes

Hello, since recent changes to haifa-sched.c, the SPU back-end runs into internal errors due to its use of the sched_emit_insn routine to emit instruction prefetch instructions during final scheduling. Now that scheduled instructions aren't placed into the instruction stream directly, sched_emit_

[commit, spu] Fix aliasing violation in SPU libgcc

Hello, the SPU __multi3 and __udivmodti4 routines were using code that violates the aliasing rules for type-punning between TImode and vector types. This now causes errors when building those files. The patch below replaces this code by type-punning via unions. Tested on spu-elf, committed to m

[commit, spu] Fix internal error due to spu_emit_epilogue

Hello, spu_expand_epilogue used to emit a barrier after the return jump; this seems to confuse the CFG layer after some recent changes, causing internal errors. Since other backends don't emit a barrier here either, I've removed this from the SPU backend as well. Tested on spu-elf, committed to m

Re: [PATCH] make Ada runtime function building use build_function_type_list

On Wed, Apr 20, 2011 at 10:08:21AM -0700, Nathan Froyd wrote: > This patch changes most of the uses of build_function_type in the Ada to > use build_function_type_list. There are a handful of > build_function_type calls left; replacing those will have to wait until > we get a build_function_type_{

Re: IVOPT improvement patch

On Thu, Jul 29, 2010 at 7:14 AM, H.J. Lu wrote: > On Wed, Jul 28, 2010 at 8:50 PM, H.J. Lu wrote: >> On Wed, Jul 28, 2010 at 6:07 PM, H.J. Lu wrote: >>> On Tue, Jul 27, 2010 at 1:20 PM, Xinliang David Li >>> wrote: On Tue, Jul 27, 2010 at 12:57 PM, Pat Haugen wrote: >> >> Thanks

Re: [RFC] Context sensitive inline analysis

Hi, I don't really have testcase for the HP nor AIX ICE, however I can reproduce same ICE when I hack x86 to not use ctors/dtors. This patch fixes it - the problem is that ipa-prop ignore newly added functions (the global ctor built) while ipa-inline not and ipa-inline does use ipa-prop for its

Re: [PATCH Commitred] Fix PR target/48767

"Joseph S. Myers" wrote: > Yes, that testcase looks like what I had in mind, but you don't need the > dg-* directives (the defaults in gcc.c-torture/compile should be fine). > >> BTW, is it valid C? > > I think this should be considered the same as passing a type such as > "short" that can nev

[PATCH] Fix PR48772

This fixes PR48772, we can't update stmts with virtual defs during VN elimination as that may destroy SSA names we use in the VN lattice. So we have to defer it, similar to removing stmts. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-04-27 Richard Guenth

Re: [PATCH Commitred] Fix PR target/48767

On Wed, 27 Apr 2011, Kaz Kojima wrote: > "Joseph S. Myers" wrote: > > I think you should add a testcase to gcc.c-torture/compile, unless there > > is already one that this patch fixes. > > Ah, indeed. How about the attached testcase? Yes, that testcase looks like what I had in mind, but you d

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

On Tue, 26 Apr 2011, Cary Coutant wrote: > @@ -1856,6 +1871,8 @@ set_debug_level (enum debug_info_type type, int > extended, const char *arg, >else > opts->x_debug_info_level = (enum debug_info_levels) argval; > } > + > + generate_debug_line_table = debug_info_level >= DINFO_L

Re: Don't use linux.h for non-Linux targets

On Wed, 27 Apr 2011, Thomas Schwinge wrote: > One (minor) additional idea: for clarity, shouldn't it now be possible to > remove any #undef GNU_USER_TARGET_OS_CPP_BUILTINS, #undef > GNU_USER_DYNAMIC_LINKER, #undef GNU_USER_LINK_EMULATION, etc. from > *gnu.h? I don't expect you to test this; I'll

Re: [Patch,AVR]: Solve PR42210

Richard Henderson schrieb: > Why are you adding "optimize" to all these insns? None of them will > be matched unless combine is run, which implies optimization. Here is a patch without optimize in the insn conditions. The optimize condition is still present in the insv expander because I do not

Commit: FRV: Remove duplicate register classes

Hi Guys, I am applying the patch below to remove the duplicate register classes from the FRV backend. This was causing problems with the IRA initialization code. Cheers Nick gcc/ChangeLog 2011-04-27 Nick Clifton * config/frv/frv.h (enum reg_class): Delete EVEN_ACC_REGS,

[PATCH][2/n] Alias housekeeping

This fixes TARGET_MEM_REF handling in indirect_refs_may_alias_p, unifying handling of it and MEM_REFs where possible. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-04-27 Richard Guenther * tree-ssa-alias.c (indirect_refs_may_alias_p): Fix

Commit: Fix MCore REGNO_REG_CLASS macro

Hi Guys, I am applying the patch below to fix the REGNO_REG_CLASS macro for the MCore backend so that it will not try to index beyond the end of the regno_reg_class array. This was causing a segmentation fault whilst building the MCore toolchain. Cheers Nick gcc/ChangeLog 2011-04-27

[C++ Patch] PR 48771

Hi, today I (re-)tested the below. As you can see, wrt the audit trail discussion, I also removed altogether valid_type_in_constexpr_fundecl_p, I can restore it if you want, just let me know. Tested x86_64-linux. Ok for mainline? (not sure if it qualifies for the branch or we want to reconsi

  1   2   >