Re: [PATCH] PR c++/51145 - Alias template in elaborated-type-specifier

2011-11-18 Thread Gabriel Dos Reis
Just a terminology note: it should be "template alias" and not the other way around.

Re: Minor contrib.texi update

2011-11-18 Thread Gerald Pfeifer
On Tue, 15 Nov 2011, Jeff Law wrote: > Attached wrong file as Andrey pointed out earlier... Here's the right > one... Oh how I wish thunderbird would show the attachment inline... Ok. Just mind the long line and it's Rawhide (uppercase), I think. Gerald

Re: [wwwdocs] add news items for TM work

2011-11-18 Thread Gerald Pfeifer
On Fri, 18 Nov 2011, Aldy Hernandez wrote: Reading this I was wondering which platforms. On the other page it says x86 and alpha; should we just use this here, or were you thinking to keep this more marketing speak? ;-) *blush*, yes... marketing speak. Seeing that people are contributing other

Re: Memset/memcpy patch

2011-11-18 Thread Jan Hubicka
> Given that x86 memset/memcpy is still broken, I think we should revert > it for now. Well, looking into the code, the SSE alignment issues needs work - the alignment test merely tests whether some alignmnet is known not whether 16 byte alignment is known that is the cause of failures in 32bit bo

Re: [trunk] RFS: translate built-in include paths for sysroot (issue5394041)

2011-11-18 Thread 沈涵
Hi, Joseph, thanks! ChangeLog entries added to the issue description. ChangeLog * Makefile.in (GPLUSPLUS_INCLUDE_DIR_ADD_SYSROOT): add a macro definition to compile command. * cppdefault.c (GPLUSPLUS_INCLUDE_DIR_ADD_SYSROOT): replace hard coded "add_sysroot" field

[Patch/RFC] PR 51216

2011-11-18 Thread Paolo Carlini
Hi, is this the right place to handle these TREE_CODEs and avoid the ICEs on invalid or should we do something else/earlier? (tested x86_64-linux) Thanks, Paolo. /// Index: testsuite/g++.dg/cpp0x/pr51216.C === --

Fwd: Re: Links to release criteria?

2011-11-18 Thread Diego Novillo
Re-posting to gcc-patches. Diego. Original Message Subject: Re: Links to release criteria? Date: Fri, 18 Nov 2011 15:05:24 -0800 From: Diego Novillo To: Gerald Pfeifer CC: Richard Guenther , g...@gcc.gnu.org On Fri, Nov 18, 2011 at 07:54, Gerald Pfeifer wrote: On Thu, 17

[Patch, Fortran] PR50640, PR51207 - fix select_type_12

2011-11-18 Thread Tobias Burnus
The patch in class.c for PR50640 makes the default initializer ("__def_init_)TREE_READONLY. I also set attr.subroutine for __copy_, which is not needed but its lacking irritated me once when stepping through trans*.c. The patch in trans-decl.c/trans.h fixes a 4.7 regressions (P1), which cause

Re: PING: [PATCH, ARM, iWMMXt][4/5]: WMMX machine description

2011-11-18 Thread Ramana Radhakrishnan
Hi Xinyu, This doesn't apply cleanly currently on trunk and the reject appears to come from iwmmxt.md and I've not yet investigated why. Can you have a look ? cheers Ramana On 26 September 2011 04:22, Xinyu Qi wrote: > Ping. > > http://gcc.gnu.org/ml/gcc-patches/2011-09/msg00279.html > >      

Re: [RFA/ARM][Patch 01/02]: Thumb2 epilogue in RTL

2011-11-18 Thread Ramana Radhakrishnan
On 10 November 2011 18:07, Sameera Deshpande wrote: > Please find attached the reworked patch. OK but for a very small bit . I'll note that we need to add support for the iwMMXt registers but the attached patch (untested) should be what is needed to support the iwMMxt registers and this should b

Re: [PATCH] Remove dead labels to increase superblock scope

2011-11-18 Thread Tom de Vries
On 11/18/2011 10:29 PM, Eric Botcazou wrote: >> For the test-case of PR50764, a dead label is introduced by >> fixup_reorder_chain in cfg_layout_finalize, called from >> pass_reorder_blocks. > > I presume that there is no reasonable way of preventing fixup_reorder_chain > from introducing it or o

Re: [RFA/ARM][Patch]: Fix NEG_POOL_RANGE

2011-11-18 Thread Ramana Radhakrishnan
On 17 November 2011 15:16, Sameera Deshpande wrote: > Hi! > > Please find attached the patch updating NEG_POOL_RANGE from 1008 to > 1020 -(8 + ). This is OK - can you add a comment around the neg_pool_range attribute in arm.md stating that the limit should essentially be - (8 + ?. cheers Raman

Re: PING: [PATCH, ARM, iWMMXt][3/5]: built in define and expand

2011-11-18 Thread Ramana Radhakrishnan
On 20 October 2011 08:39, Xinyu Qi wrote: > Ping > > http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01103.html > >        * config/arm/arm.c (enum arm_builtins): Revise built-in fcode. >        (builtin_description bdesc_2arg): Revise built in declaration. >        (builtin_description bdesc_1arg):

Fix gcc.dg/cpp/assert4.c for non-GNU Linux systems

2011-11-18 Thread Joseph S. Myers
The testcase gcc.dg/cpp/assert4.c, which tests that #system(linux) is true if and only if __gnu_linux__ is defined, wasn't updated for the changes associated with Android support that arranged for __linux__ to be defined for all systems with the Linux kernel but __gnu_linux__ only to be defined

Re: [Patch Darwin] be more friendly to gdb

2011-11-18 Thread Mike Stump
On Nov 17, 2011, at 8:24 AM, Iain Sandoe wrote: > As things stand, Darwin's 'as' cannot produce dwarf debug. > OK for trunk? Ok.

Re: [Patch] make it possible for the target to rename ".tm_clone_table"

2011-11-18 Thread Joseph S. Myers
On Fri, 18 Nov 2011, Iain Sandoe wrote: > > > Index: gcc/defaults.h > > > === > > > --- gcc/defaults.h(revision 181476) > > > +++ gcc/defaults.h(working copy) > > > @@ -392,6 +392,14 @@ see the files COPYING3 and COPYI

Re: [PATCH] postreload: Invalidate reg_state info at barrier/volatile insns

2011-11-18 Thread Eric Botcazou
> With the patch from http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49452 > the regstate information is resetted at the unspec_volatile instead of > invalidated! You're right. This wouldn't have happened if the code had been properly commented. > 2011-11-14 Andreas Krebbel > > * postrelo

Re: Ping! Re: [RFA/ARM][Patch 02/02]: ARM epilogues in RTL

2011-11-18 Thread Ramana Radhakrishnan
On 5 October 2011 17:04, Sameera Deshpande wrote: > Ping! > > http://gcc.gnu.org/ml/gcc-patches/2011-09/msg01855.html This should now be rebased given your other changes to the Thumb2 epilogues patch or is there a newer version of this patch . This doesn't apply cleanly on top of the other patch

Re: [PATCH] Remove dead labels to increase superblock scope

2011-11-18 Thread Eric Botcazou
> For the test-case of PR50764, a dead label is introduced by > fixup_reorder_chain in cfg_layout_finalize, called from > pass_reorder_blocks. I presume that there is no reasonable way of preventing fixup_reorder_chain from introducing it or of teaching fixup_reorder_chain to remove it? -- Eric

PR other/51174: handle architectures with no DECL_COMDAT_GROUP

2011-11-18 Thread Aldy Hernandez
Hi Richard. David reports that on AIX, you can have DECL_COMDAT defined, but no DEC_COMDAT_GROUP. Consequently tm_mangle() gets called incorrectly with a null. I suggested making tm_mangle() handle nulls, but David preferred handling it in the caller. I tested this patch on x86-64 Linux, a

Re: [PATCH] Remove dead labels to increase superblock scope

2011-11-18 Thread Tom de Vries
On 11/18/2011 09:16 PM, Andrew Pinski wrote: > On Fri, Nov 18, 2011 at 5:11 AM, Tom de Vries wrote: >> Hi Eric, >> >> this patch fixes an inefficiency problem I observed while working on PR50764. >> >> For the test-case of PR50764, a dead label is introduced by >> fixup_reorder_chain >> in cfg_la

Re: [C++ Patch] PR c++/51188

2011-11-18 Thread Jason Merrill
On 11/18/2011 02:53 PM, Fabien Chêne wrote: I have just committed. Unfortunately, I have mentioned the wrong PR number in the ChangeLog. Consequently, Bugzilla isn't quite happy, shall I adjust the previous commit with the good PR number (51188) ? Yes, do adjust the ChangeLog. I do this period

Re: [PATCH] Remove dead labels to increase superblock scope

2011-11-18 Thread Andrew Pinski
On Fri, Nov 18, 2011 at 5:11 AM, Tom de Vries wrote: > Hi Eric, > > this patch fixes an inefficiency problem I observed while working on PR50764. > > For the test-case of PR50764, a dead label is introduced by > fixup_reorder_chain > in cfg_layout_finalize, called from pass_reorder_blocks. > The

Re: [C++ Patch] PR c++/51188

2011-11-18 Thread Fabien Chêne
2011/11/18 Jason Merrill : > On 11/18/2011 05:18 AM, Fabien Chêne wrote: >> >> +template<  size_t, size_t>  struct AlignedBuffer; >> +template<  size_t size>  struct AlignedBuffer >> +<  size, 8>  { >> +}; > > This testcase will break on targets where alignof(void*) != 8; just define > the base tem

Re: [PATCH] PR c++/51145 - Alias template in elaborated-type-specifier

2011-11-18 Thread Jason Merrill
See my comment in the PR. Jason

[PATCH 2/2] m68k: Implement CAS and TAS patterns.

2011-11-18 Thread Richard Henderson
--- gcc/config/m68k/m68k.md |9 + gcc/config/m68k/sync.md | 80 +++ 2 files changed, 89 insertions(+), 0 deletions(-) create mode 100644 gcc/config/m68k/sync.md diff --git a/gcc/config/m68k/m68k.md b/gcc/config/m68k/m68k.md index 672ef0d..e4b

[PATCH 0/2] Add atomic support to m68k

2011-11-18 Thread Richard Henderson
The first patch adds support for the m68k-linux syscall. The second patch adds native support for the m680[2346]0 CAS instruction, and the m68000/Coldfire TAS instruction. Both tested only via cross-compile. Please test... r~ Richard Henderson (2): m68k-linux: Implement atomic operations

[PATCH 1/2] m68k-linux: Implement atomic operations via syscall.

2011-11-18 Thread Richard Henderson
--- gcc/config/m68k/linux.h |4 + gcc/config/m68k/m68k.c|8 ++ libgcc/config.host|6 +- libgcc/config/m68k/linux-atomic.c | 211 + libgcc/config/m68k/t-linux|1 + 5 files changed, 227 insertions(+),

[PATCH] PR c++/51145 - Alias template in elaborated-type-specifier

2011-11-18 Thread Dodji Seketeli
Hello, As reminded in the audit trail of this PR, the identifier of an elaborate specifier should never be a simple-template-id that resolves to an alias template specialization, but the current implementation fails to enforce that. I reused check_elaborated_type_specifier in cp_parser_elaborated

Re: PATCH: PR target/33944: streaming 64-bit integer stores

2011-11-18 Thread Uros Bizjak
On Fri, Nov 18, 2011 at 6:58 PM, H.J. Lu wrote: > _mm_stream_si64 intrinsic is missing.  It is implemented in VC++: > > http://msdn.microsoft.com/en-us/library/bb531466.aspx > > We are adding it to Intel64 SDM.  Here is a patch to implement it in GCC. > I added UNSPEC_MOVNTI so that it won't get

Re: [Patch] make it possible for the target to rename ".tm_clone_table"

2011-11-18 Thread Rainer Orth
Iain Sandoe writes: > well ... what about this instead then? That one should at least be no worse than the current situation :-) > Index: gcc/defaults.h > === > --- gcc/defaults.h(revision 181476) > +++ gcc/defaults.h(worki

Re: [Patch] make it possible for the target to rename ".tm_clone_table"

2011-11-18 Thread Iain Sandoe
On 18 Nov 2011, at 18:12, Rainer Orth wrote: Iain Sandoe writes: This, together with the unconditional use in varasm.c, will lead to a bootstrap failure on Tru64 UNIX, which lacks named sections completely. right, it was worrying me what a target without named sections does - ... I can

Re: PR 44707 testcase failure

2011-11-18 Thread Ulrich Weigand
Mike Stump wrote: > On Nov 18, 2011, at 9:04 AM, "Ulrich Weigand" wrote: > > Huh, it seems the test in this form assumes the assembler supports /* ... > > */ > > comments. But this isn't really necessary for the test to work; I think the > > best fix would be to just remove the comment and simp

Re: [Patch] make it possible for the target to rename ".tm_clone_table"

2011-11-18 Thread Rainer Orth
Iain Sandoe writes: >> This, together with the unconditional use in varasm.c, will lead to a >> bootstrap failure on Tru64 UNIX, which lacks named sections completely. > > right, it was worrying me what a target without named sections does - > ... I can easily remove the #if defined (TARGET_ASM_N

Re: [Patch] make it possible for the target to rename ".tm_clone_table"

2011-11-18 Thread Iain Sandoe
On 18 Nov 2011, at 17:56, Rainer Orth wrote: Iain Sandoe writes: Index: gcc/defaults.h === --- gcc/defaults.h (revision 181476) +++ gcc/defaults.h (working copy) @@ -392,6 +392,14 @@ see the files COPYING3 and COPYING.

PATCH: PR target/33944: streaming 64-bit integer stores

2011-11-18 Thread H.J. Lu
Hi, _mm_stream_si64 intrinsic is missing. It is implemented in VC++: http://msdn.microsoft.com/en-us/library/bb531466.aspx We are adding it to Intel64 SDM. Here is a patch to implement it in GCC. I added UNSPEC_MOVNTI so that it won't get confused with (define_insn "sse_movntdi" [(set (matc

Re: [Patch] make it possible for the target to rename ".tm_clone_table"

2011-11-18 Thread Rainer Orth
Iain Sandoe writes: > Index: gcc/defaults.h > === > --- gcc/defaults.h(revision 181476) > +++ gcc/defaults.h(working copy) > @@ -392,6 +392,14 @@ see the files COPYING3 and COPYING.RUNTIME respect > #endif > #endif > > +/

[Patch] make it possible for the target to rename ".tm_clone_table"

2011-11-18 Thread Iain Sandoe
At least Darwin needs a different name for the tm_clone_table section. I'm wondering what a target without named sections does for this - is there some reason it needs to be in a separate section (from data)... ... perhaps what I'm missing here will help understand why the clone tests fail

[PATCH, PR 50744] Prevent overflows in IPA-CP

2011-11-18 Thread Martin Jambor
Hi, PR 50744 is an issue with an integer overflow when we propagate the estimated size and time effects from callees to callers. Because such paths in the parameter value graph can be arbitrarily long, we simply need to introduce an artificial cap on these values. This is what the patch below do

Re: [patch, ia64, libgcc] Patch to fix libunwind build on IA64

2011-11-18 Thread Steve Ellcey
On Fri, 2011-11-18 at 18:26 +0100, Michael Matz wrote: > Hi, > > On Fri, 18 Nov 2011, Steve Ellcey wrote: > > > This patch fixes the IA64 bootstrap when building libunwind. When using > > -fexceptions libunwind will have a reference to __gcc_personality_v0 and > > the bootstrap fails, using -fno

Re: powerpc compare_and_swap fails

2011-11-18 Thread Richard Henderson
On 11/17/2011 03:58 PM, Alan Modra wrote: > Thanks! This, along with my mutex.h rewrite plus adding a > memory_barrier insn gets me down to just one libgomp failure. The memory_barrier thing should be fixed now as well... r~

Re: RFC: powerpc libgomp locking

2011-11-18 Thread Richard Henderson
On 11/17/2011 03:47 PM, Alan Modra wrote: > BTW, we're left with dead stores into oldval's stack slot. Is it > legal for the first two parameters of __atomic_compare_exchange to > alias? I'm guessing that the stores might disappear if we tell gcc > that they can't alias. No they cannot alias. M

Re: [patch, ia64, libgcc] Patch to fix libunwind build on IA64

2011-11-18 Thread Rainer Orth
Steve Ellcey writes: > This patch fixes the IA64 bootstrap when building libunwind. When using > -fexceptions libunwind will have a reference to __gcc_personality_v0 and > the bootstrap fails, using -fno-exceptions fixes this. This problem > started with r181172, Michael Matz's fix to mark obje

Re: [Patch RFA libitm] Deal with __USER_LABEL_PREFIX__ in the asm

2011-11-18 Thread Richard Henderson
On 11/18/2011 04:00 AM, Iain Sandoe wrote: > libitm: > > * config/x86/sjlj.S (CONCAT1, CONCAT2, SYM): Respond to > __USER_LABEL_PREFIX__ for targets that use it. > TYPE, SIZE, HIDDEN): New macros to assist on non-elf targets. > (_ITM_beginTransaction): Use SYM, TYPE, SIZE macros to

Re: [Patch libitm/testsuite] MAP_ANON

2011-11-18 Thread Richard Henderson
On 11/18/2011 04:27 AM, Iain Sandoe wrote: > libitm: > > * testsuite/libitm.c/memcpy-1.c: Allow for MAP_ANON spelling. > * testsuite/libitm.c/memset-1.c: Likewise. Ok. r~

Re: PR 44707 testcase failure

2011-11-18 Thread Mike Stump
On Nov 18, 2011, at 9:04 AM, "Ulrich Weigand" wrote: > David Edelsohn wrote: >> I noticed that gcc.c-torture/compile/pr44707.c fails on AIX because it >> generates invalid assembly language: >> >> # 12 "/farm/dje/src/src/gcc/testsuite/gcc.c-torture/compile/pr44707.c" 1 >>/* 0(6) 0(7) 0(8)

Re: [patch, ia64, libgcc] Patch to fix libunwind build on IA64

2011-11-18 Thread Michael Matz
Hi, On Fri, 18 Nov 2011, Steve Ellcey wrote: > This patch fixes the IA64 bootstrap when building libunwind. When using > -fexceptions libunwind will have a reference to __gcc_personality_v0 and > the bootstrap fails, using -fno-exceptions fixes this. This problem > started with r181172, Michael

[patch, ia64, libgcc] Patch to fix libunwind build on IA64

2011-11-18 Thread Steve Ellcey
This patch fixes the IA64 bootstrap when building libunwind. When using -fexceptions libunwind will have a reference to __gcc_personality_v0 and the bootstrap fails, using -fno-exceptions fixes this. This problem started with r181172, Michael Matz's fix to mark objects death at end of scope. Tes

Re: PR 44707 testcase failure

2011-11-18 Thread Ulrich Weigand
David Edelsohn wrote: > I noticed that gcc.c-torture/compile/pr44707.c fails on AIX because it > generates invalid assembly language: > > # 12 "/farm/dje/src/src/gcc/testsuite/gcc.c-torture/compile/pr44707.c" 1 > /* 0(6) 0(7) 0(8) 0(9) 0(10) */ > # 0 "" 2 > > I'm confused why > > __

Re: Memset/memcpy patch

2011-11-18 Thread H.J. Lu
On Fri, Nov 18, 2011 at 6:18 AM, Jan Hubicka wrote: >> I found another bug in current implementation. A patch for it doesn't >> cure i686-linux- bootstrap, but fixes fails on some tests (see >> attached). >> >> The problem was that we tried to add runtime tests for alignment even >> if both SRC an

Re: [trunk] RFS: translate built-in include paths for sysroot (issue5394041)

2011-11-18 Thread Joseph S. Myers
On Tue, 15 Nov 2011, Han Shen wrote: > 2011-11-15 Han Shen > > * gcc/Makefile.in: > * gcc/configure: > * gcc/cppdefault.c: You need to include the full ChangeLog entries with your patch. Note that paths in ChangeLogs are relative to the directory with the ChangeLog, so n

Re: [Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Iain Sandoe
On 18 Nov 2011, at 16:17, Jack Howarth wrote: For gcc trunk at r181476 with 181473-libitm-config-weakref.txt, 181473-libitm-tests-mapanon.txt and 181473-libitm-user-label-prefix.txt as well as the regenerations performed in libitm, I see the following failures on x86_64-apple-darwin11 unde

Re: [wwwdocs] add news items for TM work

2011-11-18 Thread Aldy Hernandez
+ The project was partially funded by + thehttp://www.velox-project.eu/";>Velox project. This + feature is experimental and is available for C and C++ on selected + platforms. Reading this I was wondering which platforms. On the other page it says x86 and alpha; should we just use this here

Re: Fix libgcc for FreeBSD 10

2011-11-18 Thread Rainer Orth
Gerald Pfeifer writes: > The snippet below was written when a new major release of FreeBSD > would happen every couple of years, not as frequently as recently > the case. FreeBSD 1.x and 2.x, and a.out support, have been obsolete > for what must be a decade or so, yet the regular expressions now

Re: [Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Jack Howarth
Using /sw/share/dejagnu/baseboards/unix.exp as board description file for target. Using /sw/share/dejagnu/config/unix.exp as generic interface file for target. Using /sw/src/fink.build/gcc47-4.7.0-1/gcc-4.7-2018/libitm/testsuite/config/default.exp as tool-and-target-specific interface file

Re: [libitm] Support Solaris 8 and 9/x86 with Sun as

2011-11-18 Thread Rainer Orth
Richard Henderson writes: > On 11/16/2011 08:38 AM, Rainer Orth wrote: >> * config/generic/asmcfi.h: Fix comment. >> * config/x86/sjlj.S (_ITM_beginTransaction): Provide PIC code >> sequence without .hidden support. >> (GTM_longjmp) [__ELF__]: Only use .hidden if >> HAVE_

[v3] libstdc++/51209

2011-11-18 Thread Paolo Carlini
Hi, tested x86_64-linux, committed to mainline. Paolo. / 2011-11-18 Harti Brandt PR libstdc++/51209 * include/bits/hashtable.h (_Hashtable<>::_M_find_node): Return nullptr when no node is found. * include/tr1/hashtable.h (_Hashtable<>::_M_find

Re: [Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Iain Sandoe
Hi Rainer, On 18 Nov 2011, at 14:49, Rainer Orth wrote: I hope it also works for Rainer and that I've got the right designator for the Tru64 cross-case. I guess it does, given that the testcase is based on my own one :-) so long as my wrapping it in m4 hasn't broken the intent .. ;-

Re: [Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Jack Howarth
On Fri, Nov 18, 2011 at 02:43:58PM +, Iain Sandoe wrote: > > On 18 Nov 2011, at 14:40, Jack Howarth wrote: > >> On Fri, Nov 18, 2011 at 01:44:40PM +, Iain Sandoe wrote: >>> Hi, >>> >>> one of the three remaining libitm issues for Darwin is to supply the >>> dummy funcs fro the weakrefs (as

Re: [C++ Patch] PR 51150

2011-11-18 Thread Jason Merrill
OK. Jason

Re: [C++ Patch] PR 51150

2011-11-18 Thread Paolo Carlini
On 11/18/2011 03:19 PM, Paolo Carlini wrote: On 11/18/2011 03:11 PM, Jason Merrill wrote: On 11/18/2011 08:57 AM, Paolo Carlini wrote: On 11/18/2011 02:53 PM, Jason Merrill wrote: I think let's go straight to cp_build_unary_op from tsubst_copy_and_build in this case; it can't be overloaded, an

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

2011-11-18 Thread Denis Chertykov
2011/11/17 Georg-Johann Lay : > Denis Chertykov wrote: > >> Let's wait for >> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01874.html >> Denis. > > This are yet more intrinsic named address spaces: > > * __pgm1, ... __pgm5 are 16-bit address spaces that refer to >  the n-th 64k chunk of flash. Coun

Re: [Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Rainer Orth
Hi Iain, > one of the three remaining libitm issues for Darwin is to supply the dummy > funcs fro the weakrefs (as Rainer has done for Tru64) for the versions of > Darwin that need them. > > Since we now have the situation where there are several targets which might > need dummy functions for the

Re: [PATCH, debug] Emit basic block markers in .debug_line section

2011-11-18 Thread Tom Tromey
> "Roberto" == Roberto Agostino Vitillo writes: Roberto> I ended up not using it and it's clear to me that its practical Roberto> uses may be questionable ("fast" single stepping in a Roberto> debugger?) but I thought it might be worth to share it. Thanks. I think that if there is no use fo

Re: [Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Iain Sandoe
On 18 Nov 2011, at 14:40, Jack Howarth wrote: On Fri, Nov 18, 2011 at 01:44:40PM +, Iain Sandoe wrote: Hi, one of the three remaining libitm issues for Darwin is to supply the dummy funcs fro the weakrefs (as Rainer has done for Tru64) for the versions of Darwin that need them. Since we

Re: [Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Jack Howarth
On Fri, Nov 18, 2011 at 01:44:40PM +, Iain Sandoe wrote: > Hi, > > one of the three remaining libitm issues for Darwin is to supply the > dummy funcs fro the weakrefs (as Rainer has done for Tru64) for the > versions of Darwin that need them. > > Since we now have the situation where there

Re: [PATCH] Incorrect volatile bitfield load-extend

2011-11-18 Thread Richard Earnshaw
On 10/11/11 03:57, Joey Ye wrote: > Trunk gcc mis-handles following volatile bitfield case on ARM target: > > $ cat a.c > extern void check(int); > typedef struct { > volatile unsigned short a:8, b:8; > } BitStruct; > BitStruct bits = {1, 2}; > int main () > { > check(bits.a); > return 0; >

Re: [wwwdocs] add news items for TM work

2011-11-18 Thread Gerald Pfeifer
Hej Aldy, thanks for doing this. It is mostly okay, I just have a minor suggestion or two. On Tue, 15 Nov 2011, Aldy Hernandez wrote: > + The project was partially funded by > + the http://www.velox-project.eu/";>Velox project. This > + feature is experimental and is available for C and C++ on

[Patch libitm/testsuite] MAP_ANON

2011-11-18 Thread Iain Sandoe
For some reason, Darwin spells "MAP_ANONYMOUS" and "MAP_ANON" which causes the memcpy/set tests to fail. fixed as below OK for trunk? Iain libitm: * testsuite/libitm.c/memcpy-1.c: Allow for MAP_ANON spelling. * testsuite/libitm.c/memset-1.c: Likewise. Index: libitm/testsuite/

Re: [C++ Patch] PR 51150

2011-11-18 Thread Paolo Carlini
On 11/18/2011 03:11 PM, Jason Merrill wrote: On 11/18/2011 08:57 AM, Paolo Carlini wrote: On 11/18/2011 02:53 PM, Jason Merrill wrote: I think let's go straight to cp_build_unary_op from tsubst_copy_and_build in this case; it can't be overloaded, and we don't want to mess with build_non_depende

Re: Memset/memcpy patch

2011-11-18 Thread Jan Hubicka
> I found another bug in current implementation. A patch for it doesn't > cure i686-linux- bootstrap, but fixes fails on some tests (see > attached). > > The problem was that we tried to add runtime tests for alignment even > if both SRC and DST had unknown alignment - in this case it could be > i

Re: [C++ Patch] PR 51150

2011-11-18 Thread Jason Merrill
On 11/18/2011 08:57 AM, Paolo Carlini wrote: On 11/18/2011 02:53 PM, Jason Merrill wrote: I think let's go straight to cp_build_unary_op from tsubst_copy_and_build in this case; it can't be overloaded, and we don't want to mess with build_non_dependent_expr. Ok. In the meanwhile I actually test

[Patch RFA libitm] Deal with __USER_LABEL_PREFIX__ in the asm

2011-11-18 Thread Iain Sandoe
Darwin needs to account for __USER_LABEL_PREFIX__ in the asm. I used some of the arm code in libgcc as an example ... allows the lib to build OK on x86-darwin ... (with working hidden symbols etc.) comments/OK for trunk (assuming a test on linux is successful, of course)? Iain libitm:

Re: [C++ Patch] PR 51150

2011-11-18 Thread Paolo Carlini
On 11/18/2011 02:53 PM, Jason Merrill wrote: I think let's go straight to cp_build_unary_op from tsubst_copy_and_build in this case; it can't be overloaded, and we don't want to mess with build_non_dependent_expr. Ok. In the meanwhile I actually tested the attached, and it works. Is it what you

Re: [C++ Patch] PR 51150

2011-11-18 Thread Jason Merrill
I think let's go straight to cp_build_unary_op from tsubst_copy_and_build in this case; it can't be overloaded, and we don't want to mess with build_non_dependent_expr. Jason

[Patch RFA libitm] config-based assessment of weakref capability

2011-11-18 Thread Iain Sandoe
Hi, one of the three remaining libitm issues for Darwin is to supply the dummy funcs fro the weakrefs (as Rainer has done for Tru64) for the versions of Darwin that need them. Since we now have the situation where there are several targets which might need dummy functions for the weak dec

Re: [C++ Patch] PR 51150

2011-11-18 Thread Paolo Carlini
On 11/18/2011 02:29 PM, Paolo Carlini wrote: On 11/18/2011 02:27 PM, Paolo Carlini wrote: On 11/18/2011 02:16 PM, Jason Merrill wrote: On 11/17/2011 06:52 PM, Paolo Carlini wrote: The problem is, in all these tests, value is always a FIX_TRUNC_EXPR, which, AFAICS, is always left untouched. Aha

Re: [C++ Patch] PR 51150

2011-11-18 Thread Paolo Carlini
On 11/18/2011 02:27 PM, Paolo Carlini wrote: On 11/18/2011 02:16 PM, Jason Merrill wrote: On 11/17/2011 06:52 PM, Paolo Carlini wrote: The problem is, in all these tests, value is always a FIX_TRUNC_EXPR, which, AFAICS, is always left untouched. Aha, I guess tsubst_copy_and_build needs to handl

Re: [C++ Patch] PR 51150

2011-11-18 Thread Paolo Carlini
On 11/18/2011 02:16 PM, Jason Merrill wrote: On 11/17/2011 06:52 PM, Paolo Carlini wrote: The problem is, in all these tests, value is always a FIX_TRUNC_EXPR, which, AFAICS, is always left untouched. Aha, I guess tsubst_copy_and_build needs to handle FIX_TRUNC_EXPR better. I see. Maybe better

Re: [C++ Patch] PR 51150

2011-11-18 Thread Jason Merrill
On 11/17/2011 06:52 PM, Paolo Carlini wrote: The problem is, in all these tests, value is always a FIX_TRUNC_EXPR, which, AFAICS, is always left untouched. Aha, I guess tsubst_copy_and_build needs to handle FIX_TRUNC_EXPR better. Jason

[PATCH] Remove dead labels to increase superblock scope

2011-11-18 Thread Tom de Vries
Hi Eric, this patch fixes an inefficiency problem I observed while working on PR50764. For the test-case of PR50764, a dead label is introduced by fixup_reorder_chain in cfg_layout_finalize, called from pass_reorder_blocks. The dead label is removed by a cleanup_cfg in pass_duplicate_computed_got

Re: [PATCH] PR c++/51191 - ICE on alias of alias template instantiation

2011-11-18 Thread Jason Merrill
On 11/17/2011 06:32 PM, Dodji Seketeli wrote: I am bootstrapping and testing the (IMHO better) patch below on x86_64-unknown-linux-gnu against trunk. OK. Jason

Re: [C++ Patch] PR c++/51188

2011-11-18 Thread Jason Merrill
On 11/18/2011 05:18 AM, Fabien Chêne wrote: +template< size_t, size_t> struct AlignedBuffer; +template< size_t size> struct AlignedBuffer +< size, 8> { +}; This testcase will break on targets where alignof(void*) != 8; just define the base template and remove the partial specialization.

[v3] Declare std::hash specializations noexcept

2011-11-18 Thread Paolo Carlini
Hi, this makes sure the various hash specializations are declared noexcept. Francois, please let me know if you are seeing something wrong / missing as part of your work... Tested x86_64-linux, committed. Thanks, Paolo. // 2011-11-18 Paolo Carlini * src/hash-

Re: Memset/memcpy patch

2011-11-18 Thread Michael Zolotukhin
I found another bug in current implementation. A patch for it doesn't cure i686-linux- bootstrap, but fixes fails on some tests (see attached). The problem was that we tried to add runtime tests for alignment even if both SRC and DST had unknown alignment - in this case it could be impossible to m

Re: [wwwdocs] Buildstat update for 4.6

2011-11-18 Thread Gerald Pfeifer
On Fri, 4 Nov 2011, Tom G. Christensen wrote: > I missed them since my script that parses the archive did not expect > the extra text after the 'testsuite' part of the subject. > > Here's an updated patch. Thanks, applied. Gerald

Re: [patch tree-optimization]: Fix for PR/49806

2011-11-18 Thread Kai Tietz
PING 2011/8/9 Kai Tietz : > Hello, > > actual the remaining issue about this PR is that vrp > constructs by range analyzis for bitwise and/or expressions > type-casted results, like ((type) A) op ((type) B), or ((type) A) op CST. > So I've added to simplify_bit_ops_using_rnages the transformation

[C++ Patch] PR c++/51188

2011-11-18 Thread Fabien Chêne
Hi, As discussed in the audit trail, here is a patch that fixes this PR and all its duplicates. Tested x86_64-unknown-linux-gnu without regressions. OK to commit ? gcc/testsuite/ChangeLog 2011-11-18 Fabien Chêne PR c++/51188 * g++.dg/lookup/using46.C: New. * g++.dg/lo

Re: [PATCH] Mark back edges in insert_backedge_copies

2011-11-18 Thread Richard Guenther
On Thu, Nov 17, 2011 at 10:55 PM, William J. Schmidt wrote: > Greetings, > > While investigating http://gcc.gnu.org/bugzilla/show_bug.cgi?id=39976, I > found that tree-outof-ssa.c:insert_backedge_copies tests whether edges > are back edges, but the back edges haven't been marked to facilitate > th

Re: [PATCH, tree-opt]: Fix PR5118, ICE: tree check: expected tree that contains ‘typed’ structure, have ‘block’ in fold_checksum_tree

2011-11-18 Thread Richard Guenther
On Thu, Nov 17, 2011 at 8:47 PM, Uros Bizjak wrote: > Hello! > > Currently, bootstrap crashes with --enable-stage1-checking=all with: > > ../../../libgcc/libgcc2.c:553:3: internal compiler error: tree check: expected > tree that contains ‘typed’ structure, have ‘block’ in fold_checksum_tree, at >

Re: [rs6000] Fix PR 50906, eh_frame and other woes

2011-11-18 Thread Olivier Hainque
On Nov 17, 2011, at 20:46 , David Edelsohn wrote: > Coincidentally, IBM's XL compiler is encountering a similar issue and > proposing a similar solution of having the stack pointer update > conflict with all memory accesses. Interesting timing :-) > I think all these corner cases > confirm tha