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

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

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

[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

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

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: [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

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/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.

[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 |

[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 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_

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

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 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, 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: [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] 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: [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

[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] 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

Go patch committed: Add rune type

2011-10-28 Thread Ian Lance Taylor
The Go language has a new type, rune, which is currently an alias for int. In the future it will be an alias for int32. This type represents a Unicode character. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r b1a5c9a0b9ff go/gogo.cc --- a/go/

Patch committed: Fix -fsplit-stack unwind

2011-10-28 Thread Ian Lance Taylor
The CFI code for the -fsplit-stack support was incomplete. It did not correctly unwind throughout the function. This patch fixes that by adding the CFI information at the start. This is probably still not precisely correct at all times, but it is better. In 32-bit mode, the exception cleanup ro

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

2011-10-28 Thread Hans-Peter Nilsson
On Mon, 24 Oct 2011, Richard Henderson wrote: > From: Richard Henderson > + /* Certain vector operations can be implemented with vector permutation. > */ > + if (VECTOR_MODE_P (mode)) > +{ > + enum tree_code tcode = ERROR_MARK; > + rtx sel; > + > + if (binoptab == vec_inte

Re: [Patch, libfortran, 1/3] Simplify handling of special files

2011-10-28 Thread Mikael Morin
On Tuesday 18 October 2011 16:42:45 Janne Blomqvist wrote: > Hi, > > in a few places in libgfortran we have some code for handling special > and/or non-seekable files differently. The problem is that special > files don't all have some nice consistent behavior. E.g. wrt. seeking, > some allow seek

[wwwdocs] Update home page with reference to release announcement

2011-10-28 Thread Gerald Pfeifer
Committed. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.818 diff -u -r1.818 index.html --- index.html 27 Oct 2011 12:07:48 - 1.818 +++ index.html 28 Oct 2011 20:45:33 -0

[PR50869] don't attempt to expand CFA within cselib

2011-10-28 Thread Alexandre Oliva
An assertion check meant to verify that var loc expansions that didn't involve VALUEs (say constants, REGs, etc) didn't push values onto the dependency stack failed in an expansion of the argp reg, because equivalences for it are preserved at cselib table resets, and cselib later tries to expand it

Re: scalar vector shift expansion problem on 64-bit

2011-10-28 Thread Jakub Jelinek
On Fri, Oct 28, 2011 at 11:44:17AM -0700, Richard Henderson wrote: > > A wild guess, though untested, because I don't have a reproducer: > > > > 2011-10-28 Jakub Jelinek > > > > * tree-vect-stmts.c (vectorizable_shift): If op1 is vect_external_def > > and has different type from op0, c

Re: [PATCH i386] PR47698 no CMOV for volatile mem

2011-10-28 Thread Sergey Ostanevich
On Fri, Oct 28, 2011 at 7:25 PM, Sergey Ostanevich wrote: > On Fri, Oct 28, 2011 at 4:52 PM, Richard Guenther wrote: >> On Fri, 28 Oct 2011, Sergey Ostanevich wrote: >> >>> On Fri, Oct 28, 2011 at 12:16 PM, Richard Guenther >>> wrote: >>> > On Thu, 27 Oct 2011, Uros Bizjak wrote: >>> > >>> >> H

Re: [patch][google] Allow static const floats unless -pedantic is passed. (issue 5306071)

2011-10-28 Thread Jeffrey Yasskin
Thanks Diego, Here's a new version of the patch with fixes for your comments. I'll submit it in a couple hours unless I hear objections. On Fri, Oct 28, 2011 at 7:57 AM, Diego Novillo wrote: > On Thu, Oct 27, 2011 at 09:27,   wrote: >> Reviewers: Diego Novillo, >> >> Message: >> This patch is in

Re: Require canonical type comparison for typedefs again.

2011-10-28 Thread H.J. Lu
On Fri, Dec 17, 2010 at 12:11 PM, H.J. Lu wrote: > On Wed, Oct 27, 2010 at 7:24 AM, Dodji Seketeli wrote: >> Hello, >> >> So I forgot to remove the wrong "optimization" on the parms of >> template template parameters. In the patch at the end of this message >> this hunks fixes that: >> > ... >> >

Re: [PATCH, rs6000] Preserve link stack for 476 cpus

2011-10-28 Thread David Edelsohn
On Fri, Oct 28, 2011 at 12:36 PM, Peter Bergner wrote: > So David, do we even want to bother trying to support this on -m64 > given the only cpu that needs this is a 32-bit only cpu?  If so, I > can try and work with Alan to figure out how we can merge the > function descriptors for the thunk rou

Re: [C++ Patch] Avoid uninitialized warning in pt.c

2011-10-28 Thread Jason Merrill
OK. Jason

[C++ Patch] Avoid uninitialized warning in pt.c

2011-10-28 Thread Paolo Carlini
Hi, I think we have to do this, the warning I'm seeing doesn't seem bogus: in principle comp_template_args_with_info may leave bad_old_arg and bad_new_arg uninitialized. Ok? Thanks, Paolo. 2011-10-28 Paolo Carlini * pt.c (unify_pack_expansion): Initialize bad_ol

Re: [PATCH, i386]: Remove lshift_insn and lshift code attributes

2011-10-28 Thread Uros Bizjak
On Fri, Oct 28, 2011 at 8:21 PM, Uros Bizjak wrote: > Hello! > > We can extend existing code attributes.  Also, the patch includes some > stylistic changes in XOP shift patterns. > > No functional changes. Eh, the patch was the old one, added is additional patch with updated ChangeLog: 2011-10-2

Re: [PATCH, rs6000] Preserve link stack for 476 cpus

2011-10-28 Thread Richard Henderson
On 10/28/2011 11:35 AM, Peter Bergner wrote: > On Fri, 2011-10-28 at 09:44 -0700, Richard Henderson wrote: >> Not quite. You can't allow the user to set TARGET_LINK_STACK either, >> for 64-bit. Because it won't work without further fixups. More like >> >> if (TARGET_POWERPC64) >> SET_TARGE

Re: PING 2 : [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

2011-10-28 Thread Mike Stump
On Oct 28, 2011, at 8:41 AM, Iain Sandoe wrote: > This is unreviewed for 2 weeks. Odd, usually the Ada people are fairly responsive. If they want me to weigh in, I approve of the concept behind the work.

Re: [Patch Darwin/PPC] implement out-of-line FPR/GPR saves/restores.

2011-10-28 Thread Mike Stump
On Oct 28, 2011, at 3:40 AM, Iain Sandoe wrote: > To test what you suggested I built some code that dropped down a few stack > levels (with saves of FPR/GPR) and then either aborts or spins on a sleep. Uhm, that's not enough, for async, you need to spawn threads that do the interesting stuff in

[PATCH] Pattern recognize shifts with different rhs1/rhs2 types

2011-10-28 Thread Jakub Jelinek
Hi! This patch implements what I've talked about, with this we can now with -mavx2 as well as -mxop vectorize long long/unsigned long long shifts by int or long long/unsigned long long shifts by long long (where the FE casts it to int first). Already covered by the *vshift-* testcases I've commit

Re: scalar vector shift expansion problem on 64-bit

2011-10-28 Thread Richard Henderson
On 10/28/2011 11:41 AM, Jakub Jelinek wrote: > On Fri, Oct 28, 2011 at 06:50:49PM +0200, Jakub Jelinek wrote: >> On Fri, Oct 28, 2011 at 09:07:31AM -0700, Richard Henderson wrote: >>> I think this is the same problem as Jakub is attacking here: >>> >>> http://gcc.gnu.org/ml/gcc-patches/2011-10/ms

Re: [PATCH, rs6000] Preserve link stack for 476 cpus

2011-10-28 Thread Peter Bergner
On Fri, 2011-10-28 at 09:44 -0700, Richard Henderson wrote: > Not quite. You can't allow the user to set TARGET_LINK_STACK either, > for 64-bit. Because it won't work without further fixups. More like > > if (TARGET_POWERPC64) > SET_TARGET_LINK_STACK (0); > if (TARGET_LINK_STACK == -1)

Re: scalar vector shift expansion problem on 64-bit

2011-10-28 Thread Jakub Jelinek
On Fri, Oct 28, 2011 at 06:50:49PM +0200, Jakub Jelinek wrote: > On Fri, Oct 28, 2011 at 09:07:31AM -0700, Richard Henderson wrote: > > I think this is the same problem as Jakub is attacking here: > > > > http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02503.html > > It has been checked in already

[PATCH, i386]: Remove lshift_insn and lshift code attributes

2011-10-28 Thread Uros Bizjak
Hello! We can extend existing code attributes. Also, the patch includes some stylistic changes in XOP shift patterns. No functional changes. 2011-10-28 Uros Bizjak * config/i386/i386.md (shift_insn): Rename code attribute from shiftrt_insn. Also handle ashift RTX. (

Re: [C++ Patch] PR 50864

2011-10-28 Thread Jason Merrill
OK. Jason

[Patch Ada RFA] make sure that multilibs are built with correct s-oscons.ads

2011-10-28 Thread Iain Sandoe
The sizes of items represented in s-oscons.ads can (and do) change with the multi-lib on targets that support libada as a multi-lib. At present, s-oscons.ads is only built once (in gcc/ada) and sym- linked to rts*/ This is causing a bunch of failures on i686-darwin9 where the m64 multi-li

[pph] Various Merging Fixes (issue5330048)

2011-10-28 Thread Lawrence Crowl
Add namespace merging. This change generalizes pph_out_merge_keys on the global namespace to all namspaces. Preallocate decl_lang_specific for namespaces with streaming in their keys. Stream out namespace members in declaration order. This change mysteriously fixes mysterious bugs. Add initial

[C++ Patch] PR 50864

2011-10-28 Thread Paolo Carlini
Hi, as per the recent discussion. This also changes c++/50870 from ice-on-valid to reject-valid, a *tad* better I think (but I mean to continue working on it, for a while). Tested x86_64-linux. Ok for mainline? Thanks, Paolo. /cp 2011-10-28 Paolo Carlini PR

Re: [RFC PATCH] update to libtool-2.4.2 and regenerate

2011-10-28 Thread Matthias Klose
On 10/28/2011 10:33 AM, Rainer Orth wrote: > Markus Trippelsdorf writes: > >> By popular demand, I've prepared a patch that updates the in-tree >> libtool to version 2.4.2. It is needed for lto-bootstrap with >> -fno-fat-lto-objects and FreeBSD10.x versions. > > I see that your patch doesn't de

Re: Go patch committed: Implement new syscall package

2011-10-28 Thread Rainer Orth
Ian, >> I committed this patch which should fix this problem. Bootstrapped and >> ran Go testsuite on x86_64-unknown-linux-gnu. > > thanks, but this is not enough: > > nawk: syntax error at source line 173 > context is > ([^ >>> ]*)$", <<< cparam) == 0) { > nawk: illegal statement at

Re: [RFC PATCH] update to libtool-2.4.2 and regenerate

2011-10-28 Thread Andreas Tobler
On 28.10.11 01:35, Markus Trippelsdorf wrote: By popular demand, I've prepared a patch that updates the in-tree libtool to version 2.4.2. It is needed for lto-bootstrap with -fno-fat-lto-objects and FreeBSD10.x versions. It's a pretty big update as you can see by the following diffstat. I cannot

Re: [PATCH, rs6000] Preserve link stack for 476 cpus

2011-10-28 Thread Richard Henderson
On 10/28/2011 09:36 AM, Peter Bergner wrote: > Oops, you're write. Had I compiled with -m64 -mcpu=power7 -mtune=476fp, > I would have caught that. I guess (supposing we don't want to support > 64-bit) I should have the following hunks instead, correct? > > + /* If not explicitly specified via o

Re: [PATCH, rs6000] Preserve link stack for 476 cpus

2011-10-28 Thread Peter Bergner
On Fri, 2011-10-28 at 08:20 -0700, Richard Henderson wrote: > On 10/27/2011 06:43 PM, Peter Bergner wrote: > > Ok, here's a patch to implement that, and it passes bootstrap and > > regtesting. Richard, is this what you had in mind? I'll note that > > I disabled rs6000_code_end for TARGET_POWERPC6

Re: [Patch, fortran] [00/66] PR fortran/43829 Inline sum and product (AKA scalarization of reductions)

2011-10-28 Thread Mikael Morin
On Friday 28 October 2011 15:56:36 Jack Howarth wrote: > Mikael, > The complete patch bootstraps current FSF gcc trunk on > x86_64-apple-darwin11 and the resulting gfortran compiler can compile the > Polyhedron 2005 benchmarks using... > > Compile Command : gfortran-fsf-4.7 -O3 -ffast-math -fu

Re: [C++ Patch / RFC] PR 50870

2011-10-28 Thread Paolo Carlini
On 10/28/2011 06:07 PM, Jason Merrill wrote: On 10/27/2011 08:22 PM, Paolo Carlini wrote: I'm trying to figure out where the very different args argument is coming from. Earlier than that things are different: in mainline the same args, as arglist, comes from fixup_template_parm, and earlier

Re: [C++ Patch / RFC] PR 50870

2011-10-28 Thread Jason Merrill
On 10/27/2011 08:22 PM, Paolo Carlini wrote: I'm trying to figure out where the very different args argument is coming from. Earlier than that things are different: in mainline the same args, as arglist, comes from fixup_template_parm, and earlier we have fixup_template_parms which creates the

[PING] Pass address space to REGNO_MODE_CODE_OK_FOR_BASE_P

2011-10-28 Thread Ulrich Weigand
The following patch still needs maintainer review: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01874.html Thanks, Ulrich -- Dr. Ulrich Weigand GNU Toolchain for Linux on System z and Cell BE ulrich.weig...@de.ibm.com

Re: Use of vector instructions in memmov/memset expanding

2011-10-28 Thread Richard Henderson
On 10/28/2011 05:41 AM, Michael Zolotukhin wrote: >> > +/* Target hook. Returns rtx of mode MODE with promoted value VAL, that is >> > + supposed to represent one byte. MODE could be a vector mode. >> > + Example: >> > + 1) VAL = const_int (0xAB), mode = SImode, >> > + the result is const

Re: [RFC PATCH] update to libtool-2.4.2 and regenerate

2011-10-28 Thread Markus Trippelsdorf
On 2011.10.28 at 15:34 +, Joseph S. Myers wrote: > On Fri, 28 Oct 2011, Markus Trippelsdorf wrote: > > > > What about the issues with libtool's notion of sysroots, as mentioned in > > > - have those > > > been resolved, and if so, how

Re: [PATCH i386] PR47698 no CMOV for volatile mem

2011-10-28 Thread Richard Henderson
On 10/28/2011 05:49 AM, Sergey Ostanevich wrote: > - the code for CMOV is under #ifdef in the mddle-end, which is > explicitly marked as "have to be removed" (ifcvt.c:1446) > - I have no clear evidence all platforms that support conditional move > have the same semantics that lead to the PR We're

Re: [PATCH][2/n] LTO option handling/merging rewrite

2011-10-28 Thread Jack Howarth
On Fri, Oct 28, 2011 at 01:43:07PM +0200, Richard Guenther wrote: > > This moves the existing processing of user options from lto1 to > the lto driver (lto-wrapper). It also changes the way we stream > user options from some custom binary format over to simply > streaming the original command-lin

Re: [PATCH][2/n] LTO option handling/merging rewrite

2011-10-28 Thread Joseph S. Myers
On Fri, 28 Oct 2011, Richard Guenther wrote: > + /* Fallthru. */ > + case OPT_fPIC: > + case OPT_fpic: > + case OPT_fpie: > + case OPT_fcommon: > + case OPT_fexceptions: > + append_option (decoded_options, decoded_options_count, foption); > + break; No doubt

PING 2 : [Patch Darwin/PR49992 2/2] remove ranlib special-casing from the darwin port.

2011-10-28 Thread Iain Sandoe
This is unreviewed for 2 weeks. I am sure that this issue will be affecting Ada on Darwin10/11 with the latest toolchains. It might be subtle without LTO - OTOH when LTO is engaged it breaks things completely. On 22 Oct 2011, at 08:37, Iain Sandoe wrote: On 14 Oct 2011, at 10:37, Iain

PING 2 : [Patch Darwin/PR49992 1/2] remove ranlib special-casing from the darwin port.

2011-10-28 Thread Iain Sandoe
Since this is approved by Mike, if there is no further comment by Monday, I plan to apply it. On 22 Oct 2011, at 08:36, Iain Sandoe wrote: On 14 Oct 2011, at 10:36, Iain Sandoe wrote: As per the PR audit trail, there is no reason to retain this special-casing for Darwin. (given that curre

Re: [RFC PATCH] update to libtool-2.4.2 and regenerate

2011-10-28 Thread Joseph S. Myers
On Fri, 28 Oct 2011, Markus Trippelsdorf wrote: > > What about the issues with libtool's notion of sysroots, as mentioned in > > - have those > > been resolved, and if so, how? And what about the directories only > > present in the src

Re: [PATCH i386] PR47698 no CMOV for volatile mem

2011-10-28 Thread Sergey Ostanevich
On Fri, Oct 28, 2011 at 4:52 PM, Richard Guenther wrote: > On Fri, 28 Oct 2011, Sergey Ostanevich wrote: > >> On Fri, Oct 28, 2011 at 12:16 PM, Richard Guenther wrote: >> > On Thu, 27 Oct 2011, Uros Bizjak wrote: >> > >> >> Hello! >> >> >> >> > Here's a patch for PR47698, which is about CMOV shou

Re: [trans-mem] Fix outer transactions to be considered abortable too.

2011-10-28 Thread Richard Henderson
On 10/28/2011 08:08 AM, Torvald Riegel wrote: > Fix outer transactions to be considered abortable too. > > * trans-mem.c (lower_transaction): Also add an "over" laber for outer > transactions. > (expand_transactions): Do not set hasNoAbort for outer transactions. >

Re: [trans-mem] fix C++ transaction_wrap attribute

2011-10-28 Thread Richard Henderson
On 10/28/2011 07:50 AM, Aldy Hernandez wrote: > * c-family/c-common.c (handle_tm_wrap_attribute): Handle decl > argument. > * testsuite/c-c++-common/tm/wrap-2.c: Move... > * testsuite/gcc.dg/tm/wrap-2.c: ...here. > * testsuite/g++.dg/tm/wrap-2.C: New. Ok. r~

Re: [PATCH, rs6000] Preserve link stack for 476 cpus

2011-10-28 Thread Richard Henderson
On 10/27/2011 06:43 PM, Peter Bergner wrote: > Ok, here's a patch to implement that, and it passes bootstrap and > regtesting. Richard, is this what you had in mind? I'll note that > I disabled rs6000_code_end for TARGET_POWERPC64, since I was running > into linker errors when building libgcc. T

[trans-mem] Fix outer transactions to be considered abortable too.

2011-10-28 Thread Torvald Riegel
Atomic transactions marked as outer(-atomic) transactions can abort too if they are calling functions whose type has the may_cancel_outer attribute. Given that outer transactions are probably rare, this patch just assumes that all outer transactions might abort irrespective of whether they are actu

Re: [PATCH][PING] Vectorize conversions directly

2011-10-28 Thread Richard Henderson
On 10/28/2011 01:22 AM, Dmitry Plotnikov wrote: > gcc/ > * tree-cfg.c (verify_gimple_assign_unary): Allow vector conversions. > * optabs.c (supportable_convert_operation): New function. > * optabs.h (supportable_convert_operation): New prototype. > * tree-vect-stmts.c (vectorizable_

Re: [cxx-mem-model][PATCH 0/9] Convert i386 to new atomic optabs.

2011-10-28 Thread Richard Henderson
On 10/28/2011 04:06 AM, Jakub Jelinek wrote: > It just wants a guarantee that the builtin will actually be implemented > in hw. I guess if __sync_fetch_op (new/old) isn't supported but > __sync_compare_and_swap_* is, we could just use the former and let > optabs.c deal with that. But we have to h

[trans-mem] fix C++ transaction_wrap attribute

2011-10-28 Thread Aldy Hernandez
The C++ front-end gives us a DECL for transaction_wrap attribute's argument. The C front-end OTOH gives us an IDENTIFIER_NODE. I have no idea why this change after the merge, but I have fixed the attribute handler to work with both front-ends. Also, for this case, distilled from testsuite/c-

Re: [PATCH][2/n] LTO option handling/merging rewrite

2011-10-28 Thread Diego Novillo
Isn't this the same patch as http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02348.html? Diego.

Re: [PATCH][RFC] Re-write LTO option merging

2011-10-28 Thread Diego Novillo
On 11-10-27 01:46 , Richard Guenther wrote: On Wed, 26 Oct 2011, Richard Guenther wrote: This completely rewrites LTO option merging. At compile (uselessly now at WPA?) time we now stream a COLLECT_GCC_OPTIONS like string as it comes from argv of the compiler binary. Those options are read i

[v3] Trivial formatting changes to a recently added testcase

2011-10-28 Thread Paolo Carlini
HI, committed to mainline. Thanks, Paolo. /// 2011-10-28 Paolo Carlini * testsuite/30_threads/condition_variable_any/50862.cc: Trivial formatting fixes. Index: testsuite/30_threads/condition_variable_any/50862.cc

ping: [RFA:] testsuite infrastructure for options implied by dg-final methods

2011-10-28 Thread Hans-Peter Nilsson
Ping. Subject changed from '[RFA:] fix breakage with "Update testsuite to run with slim LTO"' except it doesn't fix *all* breakage introduced by that patch, only the one I observed and intended to fix. > Date: Fri, 21 Oct 2011 04:29:20 +0200 > From: Hans-Peter Nilsson > > Date: Fri, 21 Oct 2011

Re: [trans-mem] Explicitly go irrevocable even if transaction will always go irrevocable.

2011-10-28 Thread Patrick Marlier
On 10/28/2011 08:53 AM, Aldy Hernandez wrote: If the original test was not wrong, you need to add a new test (and bonus points for finding out why this test is currently failing :)). long g, xxx, yyy; /* { dg-final { scan-tree-dump-times "transforming: .*_ITM_RaWU8 \\(&g\\);" 1 "tmmemopt" } }

Re: [Patch,AVR]: Tweak 8-bit parity expansion

2011-10-28 Thread Denis Chertykov
2011/10/28 Georg-Johann Lay : > This is minor tweak to support 8-bit parity. > > Otherwise, the input operand of 8-bit values will be extended before parity > computation. > > The final representation as libgcc call is not generated in split1 and no more > in expand. Notice that > > - combine is no

Re: [PATCH, rs6000] Update Power7 scheduling

2011-10-28 Thread David Edelsohn
On Thu, Oct 27, 2011 at 6:14 PM, Pat Haugen wrote: > The following patch fixes some issues with the Power7 scheduling > description. The patch is neutral on cpu2006 (was actually hoping to see > some improvements, but it's still the right thing to do since it more > accurately describes the hardwa

Re: [Patch, fortran] [00/66] PR fortran/43829 Inline sum and product (AKA scalarization of reductions)

2011-10-28 Thread Jack Howarth
Mikael, The complete patch bootstraps current FSF gcc trunk on x86_64-apple-darwin11 and the resulting gfortran compiler can compile the Polyhedron 2005 benchmarks using... Compile Command : gfortran-fsf-4.7 -O3 -ffast-math -funroll-loops -flto -fwhole-program %n.f90 -o %n without runtime r

[Patch,AVR]: Tweak 8-bit parity expansion

2011-10-28 Thread Georg-Johann Lay
This is minor tweak to support 8-bit parity. Otherwise, the input operand of 8-bit values will be extended before parity computation. The final representation as libgcc call is not generated in split1 and no more in expand. Notice that - combine is not allowed to propagate hard regs into zero-ex

[google] ThreadSanitizer instrumentation pass (issue5303083)

2011-10-28 Thread Dmitriy Vyukov
The patch is for google/main branch. ThreadSanitizer is a data race detector for C/C++ programs. http://code.google.com/p/data-race-test/wiki/ThreadSanitizer The tool consists of two parts: instrumentation module (this file) and a run-time library. The instrumentation module mainintains shadow cal

Re: [RFC PATCH] update to libtool-2.4.2 and regenerate

2011-10-28 Thread Ian Lance Taylor
Rainer Orth writes: > Markus Trippelsdorf writes: > >> By popular demand, I've prepared a patch that updates the in-tree >> libtool to version 2.4.2. It is needed for lto-bootstrap with >> -fno-fat-lto-objects and FreeBSD10.x versions. > > I see that your patch doesn't deal with libgo/config, w

[PATCH] Fix early inliner inlining uninlinable functions

2011-10-28 Thread Richard Guenther
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 the updated flag to the edge when iterating early inlining. Boo

Re: [trans-mem] Explicitly go irrevocable even if transaction will always go irrevocable.

2011-10-28 Thread Aldy Hernandez
diff --git a/gcc/testsuite/gcc.dg/tm/memopt-1.c b/gcc/testsuite/gcc.dg/tm/memopt-1.c index 06d4f64..9a48dcb 100644 --- a/gcc/testsuite/gcc.dg/tm/memopt-1.c +++ b/gcc/testsuite/gcc.dg/tm/memopt-1.c @@ -2,8 +2,8 @@ /* { dg-options "-fgnu-tm -O -fdump-tree-tmmemopt" } */ long g, xxx, yyy; -exte

Re: [PATCH i386] PR47698 no CMOV for volatile mem

2011-10-28 Thread Richard Guenther
On Fri, 28 Oct 2011, Sergey Ostanevich wrote: > On Fri, Oct 28, 2011 at 12:16 PM, Richard Guenther wrote: > > On Thu, 27 Oct 2011, Uros Bizjak wrote: > > > >> Hello! > >> > >> > Here's a patch for PR47698, which is about CMOV should not be > >> > generated for memory address marked as volatile. >

Re: [PATCH i386] PR47698 no CMOV for volatile mem

2011-10-28 Thread Sergey Ostanevich
On Fri, Oct 28, 2011 at 12:16 PM, Richard Guenther wrote: > On Thu, 27 Oct 2011, Uros Bizjak wrote: > >> Hello! >> >> > Here's a patch for PR47698, which is about CMOV should not be >> > generated for memory address marked as volatile. >> > Successfully bootstrapped and passed make check on >> >

Re: Use of vector instructions in memmov/memset expanding

2011-10-28 Thread Michael Zolotukhin
Hi Jan! Thanks for the review, you could find my answers to some of your remarks below. I'll send a corrected patch soon with answers to the rest of your remarks. > - {{rep_prefix_1_byte, {{-1, rep_prefix_1_byte}}}, > + {{{rep_prefix_1_byte, {{-1, rep_prefix_1_byte}}}, >{rep_prefix_1_byte, {

Re: [Patch Darwin/Ada] work around PR target/50678

2011-10-28 Thread Iain Sandoe
On 18 Oct 2011, at 13:31, Arnaud Charlet wrote: It's broken in all Libc versions that are in the wild (AFAICT from looking at the released sources). We will need to deal with configury/__ENVIRONMENT_MAC_OS_X_VERSION_MIN_REQUIRED__ stuff once there is a fixed Libc. OK, would be good to foll

[v3] Add some basic tests for associative/unordered::count()

2011-10-28 Thread Paolo Carlini
Hi, checked x86_64-linux, committed to mainline. Thanks, Paolo. // 2011-10-28 Paolo Carlini * testsuite/23_containers/unordered_map/operations/count.cc: New. * testsuite/23_containers/multimap/operations/count.cc: Likewise. * testsuite/23_con

[PATCH][2/n] LTO option handling/merging rewrite

2011-10-28 Thread Richard Guenther
This moves the existing processing of user options from lto1 to the lto driver (lto-wrapper). It also changes the way we stream user options from some custom binary format over to simply streaming the original command-line as passed to the compiler by the driver as a COLLECT_GCC_OPTIONS-like stri

Re: [RFC PATCH] Gather vectorization (PR tree-optimization/50789)

2011-10-28 Thread Richard Guenther
On Fri, 28 Oct 2011, Richard Guenther wrote: > On Fri, 28 Oct 2011, Jakub Jelinek wrote: > > > On Fri, Oct 28, 2011 at 12:59:48PM +0200, Richard Guenther wrote: > > > It is also because of re-use of memory via memcpy (yes, some dubious > > > TBAA case from C, but essentially we don't want to brea

Re: [RFC PATCH] Gather vectorization (PR tree-optimization/50789)

2011-10-28 Thread Richard Guenther
On Fri, 28 Oct 2011, Jakub Jelinek wrote: > On Fri, Oct 28, 2011 at 12:59:48PM +0200, Richard Guenther wrote: > > It is also because of re-use of memory via memcpy (yes, some dubious > > TBAA case from C, but essentially we don't want to break that). Thus > > we can't use TBAA on anonymous memory

Re: [RFC PATCH] Gather vectorization (PR tree-optimization/50789)

2011-10-28 Thread Jakub Jelinek
On Fri, Oct 28, 2011 at 12:59:48PM +0200, Richard Guenther wrote: > It is also because of re-use of memory via memcpy (yes, some dubious > TBAA case from C, but essentially we don't want to break that). Thus > we can't use TBAA on anonymous memory. No, IMHO we always use a ref_all mem access in t

Re: Another ggc anti fragmentation patchkit

2011-10-28 Thread Richard Guenther
On Sat, Oct 22, 2011 at 7:54 AM, Andi Kleen wrote: > This version addresses all earlier review comments. Passes bootstrap > and testing on x86-64. Ok? Ok. Thanks, Richard. > -Andi > >

Re: [PATCH] Add gcc-ar/nm/ranlib wrappers for slim LTO v2

2011-10-28 Thread Richard Guenther
On Fri, Oct 21, 2011 at 1:55 AM, Andi Kleen wrote: > From: Andi Kleen > > Slim LTO requires running ar/nm/ranlib with the LTO plugin. The most > convenient way to get this into existing Makefiles is using small > wrappers that pass the plugin. This matches how other compilers > (LLVM, icc) do thi

Re: [Patch]: PR49868: Named address space support for AVR

2011-10-28 Thread Denis Chertykov
2011/10/28 Georg-Johann Lay : > Georg-Johann Lay schrieb: > >> This patch adds named address space support to read data from flash (aka. >> progmem) to target AVR. >> >> The patch has two parts: >> >> The first part is a repost of Ulrich's work from >>    http://gcc.gnu.org/ml/gcc/2011-08/msg00131.

Re: [cxx-mem-model][PATCH 0/9] Convert i386 to new atomic optabs.

2011-10-28 Thread Jakub Jelinek
On Thu, Oct 27, 2011 at 09:07:29PM -0700, Richard Henderson wrote: > Jakub, in the seventh patch, is there any good reason why OMP is > making the decision of whether or not to generate a compare-and-swap > loop? Why shouldn't we simply always generate the __sync_fetch_op > builtin and let optabs.

Re: [Patch]: PR49868: Named address space support for AVR

2011-10-28 Thread Georg-Johann Lay
> The patch requires Ulrich's works which is still in review Now the correct link: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01874.html

Re: [RFC PATCH] Gather vectorization (PR tree-optimization/50789)

2011-10-28 Thread Richard Guenther
On Fri, 28 Oct 2011, Jakub Jelinek wrote: > On Fri, Oct 28, 2011 at 02:01:36PM +0400, Kirill Yukhin wrote: > > this looks really cool. I have a liitle question, since I do not > > understand vectorizer as good. > > > > Say, we have a snippet: > > int *p; > > int idx[N]; > > int arr[M]; > > for (.

  1   2   >