Re: [PATCH] Add capability to run several iterations of early optimizations

2011-10-28 Thread Matt
On Fri, 28 Oct 2011, Richard Guenther wrote: I discussed the idea of iterating early optimizations shortly with Honza. I was trying to step back a bit and look at what we try to do right now, which is, optimize functions in topological order (thus, try to make sure all callees are already early

[trans-mem] Handle aborts of outer transactions.

2011-10-28 Thread Torvald Riegel
This patch adds handling of aborts of outer transactions in libitm. Not much to say otherwise; I could have added support for restarting nested transactions but we don't have a use case for that right now, and testing this properly would also require some work. OK for branch? commit 6f1b4636157f92

Re: [PATCH 3/6] Implement interleave via permutation.

2011-10-28 Thread Richard Henderson
On 10/28/2011 02:54 PM, Hans-Peter Nilsson wrote: > Also VEC_UNPACK_HI_EXPR, VEC_UNPACK_LO_EXPR, and > VEC_PACK_TRUNC_EXPR to mention some. VEC_PACK_TRUNC_EXPR requires extra mode changes. I was intending to get to this one too, but have gotten side-tracked with other work this week. VEC_UNPAC

Re: [PATCH] Add capability to run several iterations of early optimizations

2011-10-28 Thread Maxim Kuvyrkov
On 28/10/2011, at 11:43 PM, Richard Guenther wrote: > On Fri, Oct 28, 2011 at 1:05 AM, Maxim Kuvyrkov > wrote: >> >> OK for trunk? > > diff --git a/gcc/cgraph.c b/gcc/cgraph.c > index f056d3d..4738b28 100644 > --- a/gcc/cgraph.c > +++ b/gcc/cgraph.c > @@ -2416,7 +2416,7 @@ cgraph_add_new_funct

Re: [trans-mem] Handle aborts of outer transactions.

2011-10-28 Thread Richard Henderson
On 10/28/2011 03:10 PM, Torvald Riegel wrote: > Handle aborts of outer transactions. > > * beginend.cc (GTM::gtm_thread::rollback): Add aborting flag and > handle aborts. > (_ITM_abortTransaction): Handle aborts of outer transactions. > * libitm_i.h: Same. > *

Re: [Patch, libfortran, 2/3] Introduce a ssize member function

2011-10-28 Thread Mikael Morin
On Tuesday 18 October 2011 16:51:15 Janne Blomqvist wrote: > Hi, > > the attached patch introduces a new "member function" ssize in order > to determine the size of a unit. The benefit is that with unbuffered > IO, when getting the size of a file we can replace 3 lseek() calls > with one fstat() c

Re: [PATCH 3/6] Implement interleave via permutation.

2011-10-28 Thread Hans-Peter Nilsson
On Fri, 28 Oct 2011, Richard Henderson wrote: > VEC_UNPACK_HI_EXPR and VEC_UNPACK_LO_EXPR need even more -- at > least for signed -- a source of sign-extended data. Either a LT > comparison that produces -1 (common, but not universal) or an > ASHIFTRT with a constant. We're definitely in the land

Re: [Patch, libfortran, 3/3] Update file position lazily

2011-10-28 Thread Mikael Morin
On Tuesday 18 October 2011 17:11:24 Janne Blomqvist wrote: > Also, I think I've found a small standards conformance bug. From F2008 > (N1830) 9.10.2.23 (page 256): "... ASIS if the connection was opened > without changing its position." and "If the file has been repositioned > since the connection,

Re: [PATCH] Add capability to run several iterations of early optimizations

2011-10-28 Thread Matt
On Sat, 29 Oct 2011, Maxim Kuvyrkov wrote: I like this variant a lot better than the last one - still it lacks any analysis-based justification for iteration (see my reply to Matt on what I discussed with Honza). Yes, having a way to tell whether a function have significantly changed would be

[trans-mem 1/3] Rework block IPA_TM block scanning.

2011-10-28 Thread Richard Henderson
There was a monsterous thinko in bb_in_TM_region, both in its allocation and initialization. Split that into a per-cg_data element transaction_blocks_normal. Initialize it with the proper CFG installed. * trans-mem.c (struct tm_ipa_cg_data): Add transaction_blocks_normal. (bb_in_

[trans-mem 2/3] Update to handle same_body_aliases.

2011-10-28 Thread Richard Henderson
* trans-mem.c (ipa_tm_mayenterirr_function): Recurse for aliases. (ipa_tm_mark_needed_node): Split out of callback_mark_needed. (ipa_tm_create_version_alias): Rename from callback_mark_needed. Handle comdat groups properly. (ipa_tm_create_version): Use ipa_tm

[trans-mem 3/3] Allow for the original of a tm_clone pair to be optimized away.

2011-10-28 Thread Richard Henderson
* trans-mem.c (finish_tm_clone_pairs_1): Never create cgraph_nodes here; allow them to vanish out from under. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/branches/transactional-memory@180637 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog.tm |3 +++ gcc/varasm.c |

Re: [C++ Patch] PR 50864

2011-10-28 Thread Paolo Carlini
On 10/28/2011 08:19 PM, Jason Merrill wrote: OK. Jason Humpf, looks I did something wrong when testing this and now I see the new testcase failing. I'm going to look a bit into this, and otherwise in an hour or so will simply revert the whole thing. Sorry, Paolo.

Patch committed: Use GNU/Linux in comment

2011-10-28 Thread Ian Lance Taylor
This patch changes "Linux" to "GNU/Linux" in a comment. Bootstrapped and ran libiberty testsuite on x86_64-unknown-linux-gnu. Committed to mainline as libiberty maintainer. Ian Index: setproctitle.c === --- setproctitle.c (revision

Re: [C++ Patch] PR 50864

2011-10-28 Thread Paolo Carlini
On 10/29/2011 01:53 AM, Paolo Carlini wrote: On 10/28/2011 08:19 PM, Jason Merrill wrote: OK. Jason Humpf, looks I did something wrong when testing this and now I see the new testcase failing. I'm going to look a bit into this, and otherwise in an hour or so will simply revert the whole thing

Re: [PATCH] Fix early inliner inlining uninlinable functions

2011-10-28 Thread Maxim Kuvyrkov
On 29/10/2011, at 1:57 AM, Richard Guenther wrote: > > We fail to keep the cannot-inline flag up-to-date when turning > indirect to direct calls. The following patch arranges to do > this during statement folding (which should always be called > when that happens). It also makes sure to copy th

Re: Patch committed: Use GNU/Linux in comment

2011-10-28 Thread Joseph S. Myers
On Fri, 28 Oct 2011, Ian Lance Taylor wrote: > This patch changes "Linux" to "GNU/Linux" in a comment. Bootstrapped > and ran libiberty testsuite on x86_64-unknown-linux-gnu. Committed to > mainline as libiberty maintainer. prctl is a Linux-kernel-specific syscall. The comment is describing wh

[C++ Patch] PR 50901

2011-10-28 Thread Paolo Carlini
Hi, this is a rather nasty 4.6/4.7 Regression, ice-on-valid, and the fix seems simple. Anyway, tested (for real ;) on x86_64-linux. Ok mainline and branch? Thanks, Paolo. / /cp 2011-10-28 Paolo Carlini PR c++/50901 * call.c (build_new_op_1): Handle ABS_EXP

PING: [C++-11 PATCH] Trailing comma in enum

2011-10-28 Thread Magnus Fromreide
Could someone please review this? On Sun, 2011-10-09 at 16:27 +0200, Magnus Fromreide wrote: > Hi. > > As I understand it C++-11 allows trailing commas in enum definitions. > Thus I think the following little patch should be included. > > /MF 2011-10-09 Magnus Fromreide * gcc/cp/parser

Re: Patch committed: Use GNU/Linux in comment

2011-10-28 Thread Ian Lance Taylor
"Joseph S. Myers" writes: > On Fri, 28 Oct 2011, Ian Lance Taylor wrote: > >> This patch changes "Linux" to "GNU/Linux" in a comment. Bootstrapped >> and ran libiberty testsuite on x86_64-unknown-linux-gnu. Committed to >> mainline as libiberty maintainer. > > prctl is a Linux-kernel-specific s

Build broken due to "[PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2"

2011-10-28 Thread Hans-Peter Nilsson
> From: Andi Kleen > Date: Fri, 21 Oct 2011 01:55:48 +0200 > 2011-10-19 Andi Kleen > > * Makefile.in (MOSTLYCLEANFILES): Add gcc-ar/nm/ranlib. > (native): Add gcc-ar, gcc-nm, gcc-ranlib. > (AR_LIBS, gcc-ar, gcc-ar.o, gcc-ranlib, gcc-ranlib.o, > gcc-nm, gcc-nm.o, gcc

<    1   2