Re: [google] Patch to support calling multi-versioned functions via new GCC builtin. (issue4440078)

2011-05-07 Thread Richard Guenther
On Fri, May 6, 2011 at 7:57 PM, Xinliang David Li wrote: >>> I want propose a more general solution. >>> >>> 1) Generic Annotation Support for gcc IR -- it is used attach to >>> application/optimization specific annotation to gimple statements and >>> annotations can be passed around across passes

Re: Minor type merging optimization

2011-05-07 Thread Richard Guenther
On Fri, May 6, 2011 at 10:24 PM, Jan Hubicka wrote: > Hi, > while looking at type merging code I noticed that type pairs can be managed > to be ordered by their UIDs.  This save some of hashing overhead in one of > most intensively querried hashes. > > Also gimple_lookup_type_leader is hot functio

[patch] Tidy up global_bindings_p langhook

2011-05-07 Thread Eric Botcazou
Hi, following the removal of pending sizes and the simplification of variable_size this patch changes the return type of the global_bindings_p langhook to bool and eliminates 3 calls (out of 4) present in fold-const.c; removing the 4th will require further investigation. Bootstrapped/regtested

[x32] PATCH: Handle NULL TYPE argument in ix86_promote_function_mode

2011-05-07 Thread H.J. Lu
Hi, TYPE may be NULL in ix86_promote_function_mode. I checked in this patch to handle it. H.J. --- commit 31770e61e70228463b70361c46ff6fa81eb856f8 Author: H.J. Lu Date: Fri May 6 10:02:18 2011 -0700 Handle NULL TYPE argument. diff --git a/gcc/ChangeLog.x32 b/gcc/ChangeLog.x32 index 57b2

Inliner speedup

2011-05-07 Thread Jan Hubicka
Hi, while profiling Mozilla build I noticed that I inadvertly broke the optimization of lazily updating priority queue. This patch restores the behaviour and gets inliner down to 30s. (out of 10 minutes build) bootstrapped/regtested x86_64 & comitted. Honza Index: ChangeLog ===

cleanup i386 prefetch_beneficial_p

2011-05-07 Thread Jan Hubicka
Hi, we should use ix86_tune_indices instead of random predicates in i386.c code Bootstrapped/regtested x86_64-linux, comitted. Honza Index: ChangeLog === --- ChangeLog (revision 173532) +++ ChangeLog (working copy) @@ -1,5 +1,13

Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-05-07 Thread Joern Rennecke
details of those failures, see PR target/47093. pr46500-patch-20110507.gz Description: GNU Zip compressed data

Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-05-07 Thread Joseph S. Myers
The c-opts.c change to include tm.h seems independent of the rest of the patch, and is incorrect since the file already has an explicit tm.h include with a comment saying why it's included. I think it would be a good idea for the java/expr.c include (which I can't approve, Java patches should

Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-05-07 Thread Joseph S. Myers
Similar comments also apply to the fortran/trans-types.c change: send to the fortran list as well as gcc-patches, include a comment listing the target macros involved (BOOL_TYPE_SIZE CHAR_TYPE_SIZE DOUBLE_TYPE_SIZE FLOAT_TYPE_SIZE INT_TYPE_SIZE LIBGCC2_HAS_TF_MODE LONG_DOUBLE_TYPE_SIZE LONG_LON

[Patch, libfortran] Thread safety and simplification of error printing

2011-05-07 Thread Janne Blomqvist
Hi, the error printing functionality (in io/unix.c) st_printf and st_vprintf are not thread-safe as they use a static buffer. However, since these routines are used when something has gone wrong, we shouldn't use malloc() to allocate thread-specific buffers or anything fancy like that. The way the

Re: [google] Patch to support calling multi-versioned functions via new GCC builtin. (issue4440078)

2011-05-07 Thread Xinliang David Li
On Sat, May 7, 2011 at 5:46 AM, Richard Guenther wrote: > On Fri, May 6, 2011 at 7:57 PM, Xinliang David Li wrote: I want propose a more general solution. 1) Generic Annotation Support for gcc IR -- it is used attach to application/optimization specific annotation to gimple st

Re: [patch] Tidy up global_bindings_p langhook

2011-05-07 Thread Richard Guenther
On Sat, May 7, 2011 at 3:30 PM, Eric Botcazou wrote: > Hi, > > following the removal of pending sizes and the simplification of variable_size > this patch changes the return type of the global_bindings_p langhook to bool > and eliminates 3 calls (out of 4) present in fold-const.c; removing the 4th

Re: [google] Patch to support calling multi-versioned functions via new GCC builtin. (issue4440078)

2011-05-07 Thread Richard Guenther
On Sat, May 7, 2011 at 7:07 PM, Xinliang David Li wrote: > On Sat, May 7, 2011 at 5:46 AM, Richard Guenther > wrote: >> On Fri, May 6, 2011 at 7:57 PM, Xinliang David Li wrote: > I want propose a more general solution. > > 1) Generic Annotation Support for gcc IR -- it is used attach

Re: Updated^2: RFA: Fix middle-end/46500 (void * encapsulated)

2011-05-07 Thread Joern Rennecke
NTER_SIZE SHORT_TYPE_SIZE SIZE_TYPE). Instead of including SIZE_TYPE in this list, I've removed the #if 0 code that did (not) use it. I've also added 2011 to the Copyright years in touched files that didn't already have it. pr46500-patch-20110507-2.gz Description: GNU Zip compressed data

Re: copy_tree_r and STATEMENT_LIST (was Re: C++ PATCHes relating to c++/48834, c++/40975 (array new))

2011-05-07 Thread Eric Botcazou
> It seems pretty straightforward to me that a function named copy_tree_r > should copy everything that isn't always shared (like decls). It > already copies SAVE_EXPR, after all; how is copying STATEMENT_LIST going > to cause trouble in a context where copying SAVE_EXPR isn't? OK, this can make

[Patch, libfortran] PR 48915 Fix incorrect return code with -fdump-core, error handling changes

2011-05-07 Thread Janne Blomqvist
Hi, this simple bug led me to rethink normal and error termination and thus the patch changes some other related stuff as well. So the original problem was that when -fdump-core was enabled, STOP with a numeric stop code made the program terminate with an incorrect return code, since the core dump

Put libgcc config headers in separate libgcc_tm.h

2011-05-07 Thread Joseph S. Myers
This patch creates a separate libgcc_tm_file config.gcc variable and associated generated header libgcc_tm.h to avoid listing files in ../../libgcc/config/ in tm_file and so that macros moved to libgcc headers can be poisoned on the host because those headers are no longer included on the host. Th

gcc-patches@gcc.gnu.org

2011-05-07 Thread Jan Hubicka
Hi, while I plan to deffer the heuristics tunning once rest of IPA infrastructure updates is on the place, I noticed that there are several obvious nonseces in the badness computation. This patch fix that. The changes are: 1) inline_call increases overall program size when inlining increase si

Re: [patch] fix typos and grammar in -fuse-linker-plugin docs

2011-05-07 Thread Gerald Pfeifer
On Fri, 6 May 2011, Jonathan Wakely wrote: > 2011-05-06 Jonathan Wakely > > * doc/invoke.texi (-fuse-linker-plugin): Improve grammar. > > I was going to commit a smaller version of this patch as obvious (just > the second of the three hunks in the patch) but I spotted a few other > imp

Re: [patch] fix typos and grammar in -fuse-linker-plugin docs

2011-05-07 Thread Jonathan Wakely
On 7 May 2011 22:33, Gerald Pfeifer wrote: > On Fri, 6 May 2011, Jonathan Wakely wrote: >> 2011-05-06  Jonathan Wakely   >> >>         * doc/invoke.texi (-fuse-linker-plugin): Improve grammar. >> >> I was going to commit a smaller version of this patch as obvious (just >> the second of the three h

[PATCH] C++0x, virt-specifier (final/override) support for member functions (parser and analysis both)

2011-05-07 Thread Ville Voutilainen
Tested on Linux/X86-32. Final on class is not yet supported, that's probably what I'll be working on next. Changelog and patch follow. Please holler if anything's wrong, as far as I can see this works as it's supposed to. 2011-05-08 Ville Voutilainen Implement final/override for memb

Re: [PATCH] C++0x, virt-specifier (final/override) support for member functions (parser and analysis both)

2011-05-07 Thread Jason Merrill
On 05/07/2011 07:48 PM, Ville Voutilainen wrote: +static tree set_virt_specifiers (tree decl, cp_virt_specifiers specifiers) The name of the function needs to be at the beginning of the line. + else + virt_specifier = VIRT_SPEC_UNSPECIFIED; + + if (!virt_specifier) +

Re: [PATCH] C++0x, virt-specifier (final/override) support for member functions (parser and analysis both)

2011-05-07 Thread Jason Merrill
I might as well fix these issues myself, actually. Jason

[PATCH, SMS 1/3] Support closing_branch_deps (second try)

2011-05-07 Thread Revital Eres
Hello, The attached patch includes enhancements for SMS to support targets that their doloop part is not decoupled from the rest of the loop's instructions, as SMS currently requires. In this case, the branch can not be placed wherever we want (as is currently done) due to the fact it must honor d

[PATCH, SMS 2/3] Skip DEBUG_INSNs while recognizing doloop

2011-05-07 Thread Revital Eres
Hello, The attached patch adds code to skip DEBUG_INSNs while recognizing doloop pattern. The patch was tested together with the rest of the patches in this series and on top of the patch to support do-loop for ARM (not yet in mainline, but approved http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01