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

2011-11-11 Thread Eric Botcazou
> This has been filed as radar #10302855, but we need a work-around > until that is resolved (possibly forever on older systems). > > OK for trunk? > (what opinion about 4.6?) Did you apply it to the 4.6 branch? I think that this would be appropriate. > ada: > > PR target/50678 > * i

[PATCH 3/3] rs6000: Rewrite sync patterns for atomic; expand early.

2011-11-11 Thread Richard Henderson
From: Richard Henderson The conversion of the __sync post-reload splitters was half complete. Since there are nearly no restrictions on what may appear between LL and SC, expand all the patterns immediatly. This allows significantly easier code generation for subword atomic operations. --- gcc/

[PATCH 2/3] ppc-linux: Fix call to _Unwind_SetGRPtr

2011-11-11 Thread Richard Henderson
From: Richard Henderson --- libgcc/config/rs6000/linux-unwind.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libgcc/config/rs6000/linux-unwind.h b/libgcc/config/rs6000/linux-unwind.h index 2011632..13bf413 100644 --- a/libgcc/config/rs6000/linux-unwind.h +++ b/libgc

[PATCH 0/3] Conversion to __atomic builtins

2011-11-11 Thread Richard Henderson
Well, most of it. The first patch removes two avoidable warnings in rs6000.md. It seems like we could avoid many more of the remaining, but those are harder; this one was obvious. The second patch is a build error. It has appeared on this list previously, but not yet applied. The third implemen

[PATCH 1/3] rs6000: fix*_trunc insns use nonimmediate_operand

2011-11-11 Thread Richard Henderson
From: Richard Henderson --- gcc/config/rs6000/rs6000.md |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/config/rs6000/rs6000.md b/gcc/config/rs6000/rs6000.md index 331aa79..93b0b6c 100644 --- a/gcc/config/rs6000/rs6000.md +++ b/gcc/config/rs6000/rs6000.md @@ -6787

Re: [PATCH 0/4][CFT] Handle legacy __sync libcalls

2011-11-11 Thread Kaz Kojima
Richard Henderson wrote: > These are the targets that used external __sync calls in gcc 4.6. > I've been intending to test them myself, but since these aren't > bare *-elf targets, it's taking me some time to get the various > cross-environment set up. > > Port maintainers, please test. SH patch

Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-11 Thread Dimitrios Apostolou
Hi David, I couldn't imagine such breakage... If too many platforms break perhaps we should undo the optimisation - see attached patch. Thanks, Dimitris P.S. see also bug #51094 I've attached some more fixes === modified file 'gcc/config/elfos.h' --- gcc/config/elfos.h 2011-10-30 01:45:46

PATCH [1/n] addr32: Properly use Pmode and word_mode

2011-11-11 Thread H.J. Lu
Hi, The current x32 implementation uses LEAs to convert 32bit address to 64bit. However, we can use addr32 prefix to use 32bit address directly. It improves performance by 5% in SPEC CPU 2K/2006. All changes are done in x86 backend, except for a smaill unwind library assert change: http://gcc.g

[PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-11 Thread David Miller
Any ELF target that overrides ASM_GENERATE_INTERNAL_LABEL is at risk of not building any more due to the recent elfos.h changes. Those changes require that the label format generated by ASM_GENERATE_INTERNAL_LABEL and TARGET_ASM_INTERNAL_LABEL are in sync, but that is only being ensured for targe

[PATCH 4/4] sh-linux: Install __sync libfuncs.

2011-11-11 Thread Richard Henderson
Cc: Kaz Kojima --- gcc/config/sh/linux.h |4 gcc/config/sh/sh.c|8 2 files changed, 12 insertions(+), 0 deletions(-) diff --git a/gcc/config/sh/linux.h b/gcc/config/sh/linux.h index edfd99b..7a75341 100644 --- a/gcc/config/sh/linux.h +++ b/gcc/config/sh/linux.h @@ -131,

[PATCH 1/4] arm: Install __sync libfuncs for Linux.

2011-11-11 Thread Richard Henderson
Cc: Richard Earnshaw Cc: Ramana Radhakrishnan --- gcc/config/arm/arm.c |4 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c index 6ef6f62..abf8ce1 100644 --- a/gcc/config/arm/arm.c +++ b/gcc/config/arm/arm.c @@ -1096,6 +1096,10 @

[PATCH 3/4] hppa: Install __sync libfuncs for linux.

2011-11-11 Thread Richard Henderson
Cc: John David Anglin --- gcc/config/pa/pa-linux.h |3 +++ gcc/config/pa/pa.c |3 +++ gcc/config/pa/pa.h |5 + 3 files changed, 11 insertions(+), 0 deletions(-) diff --git a/gcc/config/pa/pa-linux.h b/gcc/config/pa/pa-linux.h index 6c6cf21..addc0e1 100644 --- a/gcc/co

[PATCH 2/4] mips: Install the __sync libfuncs for mips16

2011-11-11 Thread Richard Henderson
Cc: Richard Sandiford --- gcc/config/mips/mips.c |8 ++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/gcc/config/mips/mips.c b/gcc/config/mips/mips.c index ff72e28..75e73bd 100644 --- a/gcc/config/mips/mips.c +++ b/gcc/config/mips/mips.c @@ -11218,9 +11218,13 @@ mips_in

[PATCH 0/4][CFT] Handle legacy __sync libcalls

2011-11-11 Thread Richard Henderson
These are the targets that used external __sync calls in gcc 4.6. I've been intending to test them myself, but since these aren't bare *-elf targets, it's taking me some time to get the various cross-environment set up. Port maintainers, please test. r~ Richard Henderson (4): arm: Install __

Re: [PATCH] Revert sparc vec_init improvements as they cause 64-bit regressions.

2011-11-11 Thread David Miller
From: Eric Botcazou Date: Fri, 11 Nov 2011 11:05:06 +0100 >> One thing that really irks me is how pseudo's can only be subreg'd >> on UNITS_PER_WORD boundaries. That's the real reason this stuff >> doesn't work and it's nearly impossible to subreg 32-bit values >> that end up in float regs on sp

Re: cxx-mem-model merge [6 of 9] - libstdc++-v3

2011-11-11 Thread Torvald Riegel
On Fri, 2011-11-11 at 12:45 -0500, Andrew MacLeod wrote: > On 11/11/2011 12:43 AM, Benjamin Kosnik wrote: > > > >> bkoz: As relates to the existing problem, how is the legacy support > >> invoked in compatibility-atomic-c++0x.cc? That has the old style > >> implementation of atomic_flag with a loc

Re: [PATCH, testsuite]: Do not run simulate-thread on alpha*-*-linux*

2011-11-11 Thread Richard Henderson
On 11/11/2011 09:44 AM, Uros Bizjak wrote: > 2011-11-11 Uros Bizjak > > * lib/gcc-simulate-thread.exp (simulate-thread): Do not run on > alpha*-*-linux* targets. Ok. r~

Re: [Patch testsuite/darwin] fix PR testsuite/51059

2011-11-11 Thread Mike Stump
On Nov 11, 2011, at 12:26 PM, Iain Sandoe wrote: > This probably qualifies as obvious - but having discussed some of the > background with Mike .. > .. there are other ways of solving the problem - although probably rather > heavy-weight for this problem. > > .. So, I'll let him have the say...

Go patch committed: Introduce g variable

2011-11-11 Thread Ian Lance Taylor
As another step toward multiplexing goroutines onto OS threads, this patch adopts the g variable used in the other Go's compiler runtime support library. g is a thread-local global which holds goroutine-specific information, as opposed to the existing thread-local global m which holds thread-speci

[Patch testsuite/darwin] fix PR testsuite/51059

2011-11-11 Thread Iain Sandoe
This probably qualifies as obvious - but having discussed some of the background with Mike .. .. there are other ways of solving the problem - although probably rather heavy-weight for this problem. .. So, I'll let him have the say... OK for trunk? Iain testsuite: PR testsuite/5105

Re: [libitm] Work around missing AVX support

2011-11-11 Thread Iain Sandoe
On 11 Nov 2011, at 14:33, Rainer Orth wrote: Iain Sandoe writes: however, most of the suite fails on darwin9 - with an undefined reference to delete(void*). Could this be the same issue I've been seeing on Tru64 UNIX, i.e. lack of weakdef support? http://gcc.gnu.org/ml/gcc-patch

Re: Selective Scheduling Reviews

2011-11-11 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/11/11 02:24, Andrey Belevantsev wrote: > On 10.11.2011 21:31, Jeff Law wrote: >> -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 >> >> >> [ This should have gone out some time ago... Sorry for the long >> delay ] >> >> I'm pleased to announce t

Re: Revert "PowerPC shrink-wrap support 3 of 3"

2011-11-11 Thread Hans-Peter Nilsson
> From: Hans-Peter Nilsson > Date: Thu, 10 Nov 2011 18:52:39 +0100 > > From: Hans-Peter Nilsson > > Date: Thu, 10 Nov 2011 15:12:54 +0100 > > > > From: Bernd Schmidt > > > Date: Thu, 10 Nov 2011 14:29:04 +0100 > > > > > HP, can you run full tests? > > > > Cross-test to cris-elf in progress.

Re: cxx-mem-model merge [6 of 9] - libstdc++-v3

2011-11-11 Thread Hans-Peter Nilsson
> From: Andrew MacLeod > Date: Fri, 11 Nov 2011 18:45:11 +0100 > On 11/11/2011 12:43 AM, Benjamin Kosnik wrote: > I think there is also an argument for single threaded-ness vs multi > threaded. If there is no atomic support and its single threaded, we > don't really need the lock... and I'm no

PATCH: Assert DWARF register size <= saved reg size

2011-11-11 Thread H.J. Lu
Hi, I am working on 32bit Pmode for x32: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50797 It removes all LEAs, which convert 32bit address to 64bit, and uses 0x67 address prefix instead. I got 5% speed up in SPEC CPU 2K/2006. But assert in _Unwind_SetGRValue: gcc_assert (dwarf_reg_size_table

Re: cxx-mem-model merge [6 of 9] - libstdc++-v3

2011-11-11 Thread Hans-Peter Nilsson
> From: Benjamin Kosnik > Date: Fri, 11 Nov 2011 06:43:29 +0100 > So, all: > > config/cpu/*/atomicity.h And config/cpu/*/atomic_word.h presumably? > Should go. I'll look in to peeling off this cruft sharpish. brgds, H-P

Re: [PATCH] Improve VEC_BASE

2011-11-11 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/11/11 00:53, Jakub Jelinek wrote: > > I've actually committed it yesterday after discussion with Richi on > IRC. No problem. > While his patch optimizes it, it doesn't do so for -O0 Funny, I almost make this argument for accepting your patch,

Re: [patch tree-optimization 1/2]: Branch-cost optimizations

2011-11-11 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/07/11 13:28, Kai Tietz wrote: > Sure. A more general question, which was raised by Richi here. > For BC optimization it is of course interesting to know real > instruction-costs and not just guessings. The current code in > fold-const guess

Re: [Patch] Move Objective-C runtime flags to modern options system.

2011-11-11 Thread Mike Stump
On Nov 11, 2011, at 12:25 AM, Iain Sandoe wrote: > FWIW your example doesn't reproduce the problem because it contains no > objective c exceptions code. Ah, but it can be seen to contradict what you said. It also found a bug. > However, OK - I see your point (I also see where the problem came f

Re: [Patch ObjC/NeXT] use correct personality routine for Objective-C/NeXT/ABI0/1

2011-11-11 Thread Mike Stump
On Nov 11, 2011, at 2:32 AM, Iain Sandoe wrote: > This corrects a mistake I made when splitting the runtime code up - which > causes the GNU eh personality routine to be specified for NeXT ABI 0&1. > OK for trunk/4.6? Ok.

Implement openmp atomic load/store with __atomic builtins

2011-11-11 Thread Richard Henderson
With the __sync builtins, we weren't able to implement the raw loads and stores efficiently. But with the __atomic builtins we can. Tested on x86_64-linux and committed. r~ * gimple-pretty-print.c (dump_gimple_omp_atomic_load): Dump needed. (dump_gimple_omp_atomic_store): Likewi

Re: FW: [PATCH][Cilkplus] Patch to fix unused variable

2011-11-11 Thread H.J. Lu
On Fri, Nov 11, 2011 at 8:21 AM, Iyer, Balaji V wrote: > Forgot to mention.. this is the 4th and final patch in this sequence. > > Thanks, > > Balaji V. Iyer. > > From: Iyer, Balaji V > Sent: Friday, November 11, 2011 11:16 AM > To: gcc-patches@gcc.gnu.org

[Gcc.amd] [Patch 002] Document bdver1 in changes.html for GCC4.6

2011-11-11 Thread venkataramanan.kumar
> Subject: Re: [Gcc.amd] [Patch 001] [x86 backend] Define march/mtune for > upcoming AMD Bulldozer procesor. > > > Hello! > > > > > This patch defines -march=bdver1 and -mtune=bdver1 flag for the upcoming > > > AMD Bulldozer processor. > Hi, > it seems that bdver/btver is not mentioned in invoke.t

[Gcc.amd] [Patch 001] Document bdver1/btver1 in invoke.texi

2011-11-11 Thread venkataramanan.kumar
> Subject: Re: [Gcc.amd] [Patch 001] [x86 backend] Define march/mtune for > upcoming AMD Bulldozer procesor. > > > Hello! > > > > > This patch defines -march=bdver1 and -mtune=bdver1 flag for the upcoming > > > AMD Bulldozer processor. > Hi, > it seems that bdver/btver is not mentioned in invoke.t

[PATCH] Fix tree-stdarg after "Mark objects death@end of scope" changes

2011-11-11 Thread Jakub Jelinek
Hi! This fixes stdarg-2.c failures on i?86-linux, bootstrapped/regtested on i686-linux, will commit as obvious tonight. 2011-11-11 Jakub Jelinek PR tree-optimization/51091 * tree-stdarg.c (execute_optimize_stdarg): Ignore TREE_CLOBBER_P rhs also in the va_list_simple_p

Re: cxx-mem-model merge [6 of 9] - libstdc++-v3

2011-11-11 Thread Andrew MacLeod
On 11/11/2011 12:43 AM, Benjamin Kosnik wrote: bkoz: As relates to the existing problem, how is the legacy support invoked in compatibility-atomic-c++0x.cc? That has the old style implementation of atomic_flag with a lock, which would allow this target to compile... which is another option pe

[PATCH, testsuite]: Do not run simulate-thread on alpha*-*-linux*

2011-11-11 Thread Uros Bizjak
Hello! For some reason, single-stepping executable between ldl_l and stl_c insns in gdb [1] breaks LL/SC chaining, so atomic operations never finish. This calls for gdb bugreport. Also taking into account that dejagnu timeout didn't trigger for unattended testsuite run and considering huge amount

Re: [Patch, Fortran] PR 51073: fix for zero-sized coarray arrays

2011-11-11 Thread Steve Kargl
On Fri, Nov 11, 2011 at 03:24:45PM +0100, Tobias Burnus wrote: > > The patch does the same we do for nonstatic variables: It allocates a > single byte in this case; as it is done in the front-end and as it is a > compile-time constant, there is no performance problem ;-) What about memory press

Re: [PATCH] Don't ICE on SLP calls if the same call is used in multiple SLP instances (PR tree-optimization/51058)

2011-11-11 Thread Ira Rosen
On 11 November 2011 19:06, Jakub Jelinek wrote: > On Fri, Nov 11, 2011 at 06:57:58PM +0200, Ira Rosen wrote: >> On 11 November 2011 17:32, Jakub Jelinek wrote: >> > 2011-11-11 Jakub Jelinek >> > >> >PR tree-optimization/51058 >> >* tree-vect-slp.c (vect_remove_slp_scalar_calls)

Re: cxx-mem-model merge [6 of 9] - libstdc++-v3

2011-11-11 Thread Benjamin Kosnik
> > I just realized I may be feeding you an inconsistent > > configuration, see the atomicity stuff in > > libstdc++-v3/config/cpu/cris. Is that just obsolete and unused > > now or what do I need to add for that to work? > > > > You don't need to do anything there. I think that atomicity stuff >

Re: Mark objects death@end of scope

2011-11-11 Thread Michael Matz
Hi, On Fri, 11 Nov 2011, Ulrich Weigand wrote: > I haven't fully debugged it yet, but it seems to be related to the > linked list of unwind contexts that are maintained by the SjLj logic. > During unwinding, those are pulled off the list one by one; it seems the > routines that do that don't

Re: Mark objects death@end of scope

2011-11-11 Thread Ulrich Weigand
Michael Matz wrote: > On Fri, 11 Nov 2011, Ulrich Weigand wrote: > > > One reason why this happens is that the unwind*.c files are specifically > > built with -fexception. I think this is for the benefit of the DWARF > > unwinder, to ensure CFI records are available for those routines. > > Exc

Re: [PATCH] Don't ICE on SLP calls if the same call is used in multiple SLP instances (PR tree-optimization/51058)

2011-11-11 Thread Jakub Jelinek
On Fri, Nov 11, 2011 at 06:06:18PM +0100, Jakub Jelinek wrote: > > Please add > > ! { dg-final { cleanup-tree-dump "vect" } } > > > > OK otherwise. > > This is not a /vect/ testcase, but fortran torture. I guess > if you really want I could move it over to gfortran.dg/vect/ instead, > then the !

Re: [PATCH] Don't ICE on SLP calls if the same call is used in multiple SLP instances (PR tree-optimization/51058)

2011-11-11 Thread Jakub Jelinek
On Fri, Nov 11, 2011 at 06:57:58PM +0200, Ira Rosen wrote: > On 11 November 2011 17:32, Jakub Jelinek wrote: > > 2011-11-11  Jakub Jelinek   > > > >        PR tree-optimization/51058 > >        * tree-vect-slp.c (vect_remove_slp_scalar_calls): New function. > >        (vect_schedule_slp): Call it.

Re: [PATCH] Don't ICE on SLP calls if the same call is used in multiple SLP instances (PR tree-optimization/51058)

2011-11-11 Thread Ira Rosen
On 11 November 2011 17:32, Jakub Jelinek wrote: > Hi! Hi, > > Removing the scalar call in vectorizable_call for SLP vectorization > is too early, when another SLP instance refers to the same scalar call, > we'll ICE because that stmt doesn't have bb anymore or gsi_for_stmt > doesn't succeed for

Re: [PATCH] Fold VEC_PERM_EXPR/VEC_INTERLEAVE*EXPR/VEC_EXTRACT*EXPR with VECTOR_CST/CONSTRUCTOR arguments (PR tree-optimization/51074, take 2)

2011-11-11 Thread Richard Henderson
On 11/11/2011 08:41 AM, Jakub Jelinek wrote: > On Fri, Nov 11, 2011 at 08:36:36AM -0800, Richard Henderson wrote: >> Ok, except >> >>> +elts[i] >>> + = fold_convert (TREE_TYPE (TREE_TYPE (arg)), integer_zero_node); >> >> build_int_cst. > > That would work for integer modes only, but here

Re: Mark objects death@end of scope

2011-11-11 Thread Michael Matz
Hi, On Fri, 11 Nov 2011, Ulrich Weigand wrote: > One reason why this happens is that the unwind*.c files are specifically > built with -fexception. I think this is for the benefit of the DWARF > unwinder, to ensure CFI records are available for those routines. Except for the routines that sta

Re: SPU build broken (Re: CFT: [build] Move libgcc2 to toplevel libgcc)

2011-11-11 Thread Ulrich Weigand
Rainer Orth wrote: > "Ulrich Weigand" writes: > > Shouldn't the variable still be called LIB2FUNCS_EXCLUDE after the > > move to libgcc? LIB2ADD seems to expect full file names ... > > Of course, the change is bogus. I can only (half) explain this by the > change from LIB2FUNCS_STATIC_EXTRA to

[PATCH, alpha]: Restore bootstrap, broken due to config/elfos.h renames.

2011-11-11 Thread Uros Bizjak
Hello! 2011-11-11 Uros Bizjak * config/alpha/elf.h (ELF_ASCII_ESCAPES): Rename from ESCAPES. (ELF_STRING_LIMIT): Rename from STRING_LIMIT. Tested on alphaev68-pc-linux-gnu, committed to mainline SVN. Uros. Index: elf.h =

Re: [PATCH] Fold VEC_PERM_EXPR/VEC_INTERLEAVE*EXPR/VEC_EXTRACT*EXPR with VECTOR_CST/CONSTRUCTOR arguments (PR tree-optimization/51074, take 2)

2011-11-11 Thread Jakub Jelinek
On Fri, Nov 11, 2011 at 08:36:36AM -0800, Richard Henderson wrote: > Ok, except > > > +elts[i] > > + = fold_convert (TREE_TYPE (TREE_TYPE (arg)), integer_zero_node); > > build_int_cst. That would work for integer modes only, but here the type can be REAL_TYPE too. I think fold_convert

Re: SPU build broken (Re: CFT: [build] Move libgcc2 to toplevel libgcc)

2011-11-11 Thread Rainer Orth
Paolo Bonzini writes: > On 11/11/2011 05:23 PM, Rainer Orth wrote: >> The trivial patch allowed a x86_64-unknown-linux-gnu x spu-elf cross to >> finish the libgcc build, and at least the set of objects built before my >> patch series is identical to the set built now. >> >> Ok for mainline? > > O

Re: [PATCH] Fold VEC_PERM_EXPR/VEC_INTERLEAVE*EXPR/VEC_EXTRACT*EXPR with VECTOR_CST/CONSTRUCTOR arguments (PR tree-optimization/51074, take 2)

2011-11-11 Thread Richard Henderson
On 11/11/2011 07:24 AM, Jakub Jelinek wrote: > PR tree-optimization/51074 > * fold-const.c (vec_cst_ctor_to_array, fold_vec_perm): New functions. > (fold_binary_loc): Handle VEC_EXTRACT_EVEN_EXPR, > VEC_EXTRACT_ODD_EXPR, VEC_INTERLEAVE_HIGH_EXPR and > VEC_INTERLEAVE_LO

Re: [PATCH RFA] rtl-optimization/PR50663, conditional propagation missed in cprop.c pass

2011-11-11 Thread Eric Botcazou
> 2011-11-07 Bin Cheng > > PR rtl-optimization/50663 > * cprop.c (bb_implicit): New global variable. > (insert_set_in_table): Add additional parameter, record implicit set > info. > (hash_scan_set): Add additional parameter. > (compute_hash_table_work): And >

Fix various minor issues in cprop.c

2011-11-11 Thread Eric Botcazou
While reviewing PR rtl-opt/50663, I ran into some minor issues in cprop.c that can easily be addressed: - a few outdated comments, - non-obvious naming of variables (pavloc, absaltered), - reversed naming (transp instead of kill in compute_local_properties), - inconsistent protoytype for c

[Patch, libfortran] PR 51090 Check getenv result before proceeding

2011-11-11 Thread Janne Blomqvist
Hi, committed the patch below as obvious. 2011-11-11 Janne Blomqvist PR libfortran/51090 * runtime/main.c (find_addr2line): NULL check before proceeding. Index: main.c === --- main.c (revision 181287) +++ ma

Re: [PATCH] Optimize in RTL vector AND { -1, -1, ... }, IOR { -1, -1, ... } and XOR { -1, -1, ... } (take 2)

2011-11-11 Thread Bernd Schmidt
On 10/31/11 14:24, Henderson, Stuart wrote: >> 2011-09-26 Jakub Jelinek >> >> * rtl.h (const_tiny_rtx): Change into array of 4 x MAX_MACHINE_MODE >> from 3 x MAX_MACHINE_MODE. >> (CONSTM1_RTX): Define. >> * emit-rtl.c (const_tiny_rtx): Change into array of 4 x >> MAX_MAC

Re: SPU build broken (Re: CFT: [build] Move libgcc2 to toplevel libgcc)

2011-11-11 Thread Paolo Bonzini
On 11/11/2011 05:23 PM, Rainer Orth wrote: The trivial patch allowed a x86_64-unknown-linux-gnu x spu-elf cross to finish the libgcc build, and at least the set of objects built before my patch series is identical to the set built now. Ok for mainline? Ok. Have you checked for other occurrenc

Re: SPU build broken (Re: CFT: [build] Move libgcc2 to toplevel libgcc)

2011-11-11 Thread Rainer Orth
"Ulrich Weigand" writes: > Rainer Orth wrote: > >> diff --git a/gcc/config/spu/t-spu-elf b/gcc/config/spu/t-spu-elf > >> -# We exclude those because the libgcc2.c default versions do not support >> -# the SPU single-precision format (round towards zero). We provide our >> -# own versions below a

Re: Continue strict-volatile-bitfields fixes

2011-11-11 Thread Bernd Schmidt
On 11/11/11 16:30, Joey Ye wrote: > -fstrict-volatile-bitfields doesn't work incorrectly in some cases > when storing into a volatile bit-field. > > Bernd provided a fix here about 1 year ago: > http://gcc.gnu.org/ml/gcc-patches/2010-12/msg00217.html. > But it is pending to trunk. Here are my hu

FW: [PATCH][Cilkplus] Patch to fix unused variable

2011-11-11 Thread Iyer, Balaji V
Forgot to mention.. this is the 4th and final patch in this sequence. Thanks, Balaji V. Iyer. From: Iyer, Balaji V Sent: Friday, November 11, 2011 11:16 AM To: gcc-patches@gcc.gnu.org Subject: [PATCH][Cilkplus] Patch to fix unused variable Hello Everyone,

[PATCH][Cilkplus] Patch to fix unused variable

2011-11-11 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch affecting both C and C++ compilers. This patch will fix an unused variable warning in collect2.c by enclosing the variable inside #ifdef TARGET_AIX_VERSION. Thanks, Balaji V. Iyer.diff --git a/gcc/ChangeLog.cilk b/gcc/ChangeLog.cilk inde

[PATCH][Cilkplus] Patch to remove CILKPLUS IMPLEMENTED macro

2011-11-11 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch, affecting both C and C++ compilers. This patch will remove the CILKPLUS_IMPLEMENTED macro and all the #ifdef and #ifndef that uses it. This is patch #3. Thanks, Balaji V. Iyer.diff --git a/gcc/ChangeLog.cilk b/gcc/ChangeLog.cilk ind

[PATCH][Cilkplus] Patch to finish const tree

2011-11-11 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch, affecting the C and C++ compilers. This patch will fix a warning about converting a const void * to tree. This patch is PATCH #2. Thanks, Balaji V. Iyer.diff --git a/gcc/ChangeLog.cilk b/gcc/ChangeLog.cilk index 285f059..511d8f7

Re: PR c++/30195

2011-11-11 Thread Jason Merrill
On 11/11/2011 04:42 AM, Dodji Seketeli wrote: Fabien Chêne a écrit: Are the other debugging backends not interested at all in USING_DECLs ? The way debug info is generated for USING_DECLs is that handle_using_decl (via cp_emit_debug_info_for_using) asks the backend to generate debug info for

[PATCH][Cilkplus] Patch to fix Finish Call Expr

2011-11-11 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilk Plus branch, mainly affecting the G++ compiler. This patch will add the extra parameter (enum call_context) into finish_call_expr that was added in the weekly merge. I am planning to send approximately 4 patches today, and so please com

Re: [PATCH] Emit vzeroupper even from gen_return and gen_simple_return

2011-11-11 Thread Uros Bizjak
On Fri, Nov 11, 2011 at 4:33 PM, Jakub Jelinek wrote: > The avx-vzeroupper-14.c testcase now fails, because normal epilogue isn't > emitted and before simple_return or return we forgot to emit the vzeroupper > insn.  Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, > ok for tr

Re: Mark objects death@end of scope

2011-11-11 Thread Ulrich Weigand
Michael Matz wrote: > * gimplify.c (gimplify_bind_expr): Add clobbers for all variables > that go out of scope and live in memory. This seems to have completely broken SPU exception handling (note that SPU is currently completely broken anyway due to the libgcc move). What happens is

[PATCH] Emit vzeroupper even from gen_return and gen_simple_return

2011-11-11 Thread Jakub Jelinek
Hi! The avx-vzeroupper-14.c testcase now fails, because normal epilogue isn't emitted and before simple_return or return we forgot to emit the vzeroupper insn. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2011-11-11 Jakub Jelinek * config/i386/i3

[PATCH] Don't ICE on SLP calls if the same call is used in multiple SLP instances (PR tree-optimization/51058)

2011-11-11 Thread Jakub Jelinek
Hi! Removing the scalar call in vectorizable_call for SLP vectorization is too early, when another SLP instance refers to the same scalar call, we'll ICE because that stmt doesn't have bb anymore or gsi_for_stmt doesn't succeed for it. Fixed by postponing replacement of calls with zeroing of lhs

[PATCH] Fold VEC_PERM_EXPR/VEC_INTERLEAVE*EXPR/VEC_EXTRACT*EXPR with VECTOR_CST/CONSTRUCTOR arguments (PR tree-optimization/51074, take 2)

2011-11-11 Thread Jakub Jelinek
Hi! On Thu, Nov 10, 2011 at 12:00:53PM -0800, Richard Henderson wrote: > VEC_PERM_EXPR is explicitly modulo. Don't fail, mask. Here is an updated patch. In addition to the creation of subroutines this performs the permutation folding using an unsigned char array for selector and folds VEC_PERM_

PATCH, PR 50605] Handle MEM_REFs in is_gimple_ip_invariant_address

2011-11-11 Thread Martin Jambor
Hi, the problem in PR 50605 is that is_gimple_ip_invariant returns false for &MEM[(struct tRecorderImp *)&recorder + 8B] where &reorder is an IP gimple invariant. This patch fixes that by copying the code that handles MEM_REFs from is_gimple_invariant_address (and only changing decl_address_i

Re: [libitm] Work around missing AVX support

2011-11-11 Thread Rainer Orth
Iain Sandoe writes: > however, most of the suite fails on darwin9 - with an undefined reference > to delete(void*). Could this be the same issue I've been seeing on Tru64 UNIX, i.e. lack of weakdef support? http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01426.html At least my weakdef.c t

[Patch, Fortran] PR 51073: fix for zero-sized coarray arrays

2011-11-11 Thread Tobias Burnus
Dear all, attached one patches for issues found by Joel when testing gfortran on RTEMS. Coarrays with -fcoarray=lib: For zero-sized static ("save") coarray arrays, we should allocate a single byte rather than 0 bytes as "ptr = malloc (0)" might return either NULL or a unique pointer. If it

Re: [libitm] Work around missing AVX support

2011-11-11 Thread Iain Sandoe
An update .. in case anyone is following... On 11 Nov 2011, at 00:21, Richard Henderson wrote: On 11/10/2011 03:29 PM, Iain Sandoe wrote: The m64 build fails because of the -Wl,-undefined -Wl,dynamic_lookup FAOD, Is there some reason that this library needs to resolve symbols from some exter

Re: Mark objects death at end of scope

2011-11-11 Thread Michael Matz
Hi, On Thu, 10 Nov 2011, Steve Ellcey wrote: > This patch (r181172) has broken my bootstrap of IA64 Linux and I am > trying to figure out what to do about it. > > The failure happens while building libunwind (I did not configure with > --with-system-libunwind): > > /ctires/gcc/nightly/build-ia6

Re: [PATCH] [Annotalysis] Fix ICE caused by ipa-sra optimization.

2011-11-11 Thread Martin Jambor
Hi, On Fri, Nov 04, 2011 at 08:01:41AM -0700, Delesley Hutchins wrote: > Thanks for the suggestion. Unfortunately, knowing the original > declaration doesn't help me; I also need to know the original > arguments that were passed at the call site, before those arguments > were removed by ipa-sra.

[Patch ObjC/NeXT] use correct personality routine for Objective-C/NeXT/ABI0/1

2011-11-11 Thread Iain Sandoe
This corrects a mistake I made when splitting the runtime code up - which causes the GNU eh personality routine to be specified for NeXT ABI 0&1. This causes a linkage error if "-fexceptions" is specified for NeXT @ m32 (although there's no functional effect, since there is no ZCE impleme

Re: [PATCH] PR target/50038 fix: redundant zero extensions removal

2011-11-11 Thread Eric Botcazou
> I have already signed copyright agreement with the FSF. Will I need > the separate one for this particular commit? No, if your contributions are already covered by a copyright agreement with the FSF, nothing more needs to be done. -- Eric Botcazou

Re: [PATCH] Revert sparc vec_init improvements as they cause 64-bit regressions.

2011-11-11 Thread Eric Botcazou
> Eric, I tried my best to get the new code working properly on 64-bit > and I just couldn't figure out a reasonably way to do so. Same here, this looks really tricky. On the one hand we could give in a little and tolerate inferior code quality in 64-bit mode, but on the other hand this is a bi

Re: [PATCH] PR target/50038 fix: redundant zero extensions removal

2011-11-11 Thread Ilya Enkovich
Hello Eric, 2011/11/11 Eric Botcazou : >> Great! I'll be back with patch covering all non functional changes. >> Will it be OK to have everything in one patch (including current >> functional changes) or I should split it? > > Let's also rename the file while we are at it.  I'd suggest Redundant

Re: PR c++/30195

2011-11-11 Thread Dodji Seketeli
Fabien Chêne a écrit: > Are the other debugging backends not interested at all in USING_DECLs ? The way debug info is generated for USING_DECLs is that handle_using_decl (via cp_emit_debug_info_for_using) asks the backend to generate debug info for the DECLs the USING_DECL resolves to, basically

Re: [PATCH] PR target/50038 fix: redundant zero extensions removal

2011-11-11 Thread Eric Botcazou
> Great! I'll be back with patch covering all non functional changes. > Will it be OK to have everything in one patch (including current > functional changes) or I should split it? Let's also rename the file while we are at it. I'd suggest Redundant Extension Elimination for the name of the pass

Re: Selective Scheduling Reviews

2011-11-11 Thread Andrey Belevantsev
On 10.11.2011 21:31, Jeff Law wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 [ This should have gone out some time ago... Sorry for the long delay ] I'm pleased to announce that the GCC steering committee has approved the nomination of Andrey Belevantsev, Alexander Monakov, and Dmitry M

[PATCH] Revert sparc vec_init improvements as they cause 64-bit regressions.

2011-11-11 Thread David Miller
Eric, I tried my best to get the new code working properly on 64-bit and I just couldn't figure out a reasonably way to do so. Therefore I simply reverted the changes. I'll come back to this at some point in the future. One thing that really irks me is how pseudo's can only be subreg'd on UNITS

Re: PR c++/30195

2011-11-11 Thread Fabien Chêne
2011/11/10 Dodji Seketeli : > Fabien Chêne a écrit: > >> Index: gcc/dbxout.c >> === >> --- gcc/dbxout.c      (revision 178088) >> +++ gcc/dbxout.c      (working copy) >> @@ -1518,6 +1518,8 @@ dbxout_type_fields (tree type) >>        i

Re: [Patch] Move Objective-C runtime flags to modern options system.

2011-11-11 Thread Iain Sandoe
On 11 Nov 2011, at 00:30, Mike Stump wrote: On Nov 10, 2011, at 9:40 AM, Iain Sandoe wrote: Thanks for catching that --- brainstorm on my part ... the code under discussion should have been #ifndef OBCPLUS There is no prohibition against C having exceptions, so, doesn't matter if you turn