Re: [Patch,Fortran] PR39427/37829 - implement F2003's constructors

2011-11-07 Thread Paul Richard Thomas
Dear Tobias, Please stop sending us the patch! I received it right from the first mailing Cheers Paul On Sun, Nov 6, 2011 at 5:51 PM, Tobias Burnus bur...@net-b.de wrote: Last try: Also gzip the release notes - let's see whether it mailserver accepts that email. Tobias PS: I really

Re: increase call_saved_regs[] in caller-save.c

2011-11-07 Thread Michael Matz
Hi, On Sun, 6 Nov 2011, Jeff Law wrote: On Fri, 2011-11-04 at 12:25 -0600, Jeff Law wrote: The only way I can think of to have two pseudos assigned the same hard reg at the same point in the insn stream is if the two pseudos are known to have the same value. Having the same value is

Re: [Patch] alpha: add support for -fsection-anchors

2011-11-07 Thread Tristan Gingold
On Nov 4, 2011, at 4:21 PM, Richard Henderson wrote: On 11/04/2011 07:52 AM, Tristan Gingold wrote: Richard, as it impacts all alpha targets, do you have any objection ? I suspect not. We might even see what we can do about enabling it by default for all alpha targets… Committed. I let

Re: Implement C1X _Alignas, _Alignof, max_align_t, stdalign.h

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 00:43, Jonathan Wakely wrote: On 7 November 2011 00:37, Joseph S. Myers wrote: On Mon, 7 Nov 2011, Jonathan Wakely wrote: On 6 November 2011 23:53, Joseph S. Myers wrote: As with stdnoreturn.h, the contents of stdalign.h are conditioned out for C++; I'll leave it to

Re: [Patch, Fortran, OOP] PR 50919: Don't use vtable for NON_OVERRIDABLE TBP

2011-11-07 Thread Janus Weil
Hi Paul, The patch actually consists of two parts: 1) The resolve.c part prevents the conversion to a PPC call via the _vptr (for functions and subroutines). This is obviously OK thanks for the review. 2) The class.c parts prevents adding the non-overridable TBP to the vtable. As noted

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

2011-11-07 Thread Bin Cheng
Hi, This patch fixes the bug PR50663, as I reported at: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50663 It records the missing information and add implicit set info to AVIN of the basic block that implicit set is recorded for. The patch has been tested on x86 and arm-none-eabi, and no failure

Re: [Patch, Fortran, OOP] PR 50919: Don't use vtable for NON_OVERRIDABLE TBP

2011-11-07 Thread Janne Blomqvist
On Mon, Nov 7, 2011 at 10:50, Janus Weil ja...@gcc.gnu.org wrote: 2) The class.c parts prevents adding the non-overridable TBP to the vtable. As noted by Tobias, the second part breaks the ABI, so we might consider deferring it until other ABI-breaking features will be implemented (cf.

[vms,committed]: Consolidate vms config files

2011-11-07 Thread Tristan Gingold
Hi, this patch extract common definitions from alpha/vms.h and ia64/vms.h into vms/vms.h to reduce code duplication. It replaces alpha/vms64.h and ia64/vms64.h by vms/vms.h for the same purpose. Applied on trunk. Tristan. 2011-11-07 Tristan Gingold ging...@adacore.com *

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Jonathan Wakely
I'm testing my suggestion on a netbsd machine, I'd be grateful for darwin testing once I've committed it, which I expect to be later today.

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread niXman
I am currently working on a patch for OpenBSD.I wrote a some autoconf tests for mingw/*BSD. 2011/11/7 Jonathan Wakely jwakely@gmail.com: I'm testing my suggestion on a netbsd machine, I'd be grateful for darwin testing once I've committed it, which I expect to be later today.

Re: [v3] define string::pop_back()

2011-11-07 Thread Eric Botcazou
* include/bits/basic_string.h (basic_string::at): Move adjacent to other overload. (basic_string::pop_back): Define. * include/debug/string (__gnu_debug::basic_string::pop_back): Likewise. * include/ext/vstring.h (__versa_string::pop_back): Likewise. *

Re: [v3] define string::pop_back()

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 09:17, Eric Botcazou wrote:         * include/bits/basic_string.h (basic_string::at): Move adjacent to other overload.         (basic_string::pop_back): Define.         * include/debug/string (__gnu_debug::basic_string::pop_back): Likewise. * include/ext/vstring.h

Re: [Patch, Fortran, OOP] PR 50919: Don't use vtable for NON_OVERRIDABLE TBP

2011-11-07 Thread Tobias Burnus
On 11/07/2011 09:50 AM, Janus Weil wrote: As noted by Tobias, the second part breaks the ABI ... On the other hand, one could argue that the OOP ABI is still quite young and hasn't really stabilized yet (it was broken already from 4.5 to 4.6), so we might as well break it again. I know that

Re: [v3] define string::pop_back()

2011-11-07 Thread Eric Botcazou
Thanks, I'll try to work out the right adjustment for the solaris symbols file but can't test it. The 2 new symbols are duplicated, because they are matched by a regexp: ##_ZNSs[0-58-9][g-z]* (glob) _ZNSs4nposE; _ZNSs4rendEv; _ZNSs4swapERSs; _ZNSs8pop_backEv;

Re: [patch] 19/n: trans-mem: middle end/misc patches (LAST PATCH)

2011-11-07 Thread Richard Guenther
On Sun, Nov 6, 2011 at 7:53 PM, Aldy Hernandez al...@redhat.com wrote: [rth, more comments for you below] On 11/04/11 04:14, Richard Guenther wrote:    new_version = cgraph_create_node (new_decl); -   new_version-analyzed = true; +   new_version-analyzed = old_version-analyzed; Hm?  

Re: [PR50878, PATCH] Fix for verify_dominators in -ftree-tail-merge

2011-11-07 Thread Tom de Vries
On 10/31/2011 09:19 PM, Tom de Vries wrote: On 10/30/2011 10:54 AM, Richard Guenther wrote: On Sun, Oct 30, 2011 at 9:27 AM, Tom de Vries tom_devr...@mentor.com wrote: On 10/30/2011 09:20 AM, Tom de Vries wrote: Richard, I have a fix for PR50878. Sorry, with patch this time. Ok for now,

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

2011-11-07 Thread Sameera Deshpande
Hi Ramana, Please find attached reworked patch. The patch is tested with check-gcc, check-gdb and bootstrap with no regression. On Fri, 2011-10-21 at 13:43 +0100, Ramana Radhakrishnan wrote: Why are you differentiating on stack_only ? Does it really matter ? The patterns pop_multi* generate

Re: [RFA/ARM][Patch 02/05]: LDRD generation instead of POP in A15 Thumb2 epilogue.

2011-11-07 Thread Sameera Deshpande
I don't believe REG_FRAME_RELATED_EXPR does the right thing for anything besides prologues. You need to emit REG_CFA_RESTORE for the pop inside an epilogue. Richard, here is updated patch that uses REG_CFA_RESTORE instead of REG_FRAME_RELATED_EXPR. The patch is tested with check-gcc,

Re: [RFA/ARM][Patch 03/05]: STRD generation instead of PUSH in A15 Thumb2 prologue.

2011-11-07 Thread Sameera Deshpande
Hi Ramana, Please find attached reworked patch. The patch is tested with check-gcc, check-gdb and bootstrap with no regression. Ok? - Thanks and regards, Sameera D.diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c index 05c9368..334a25f 100644 --- a/gcc/config/arm/arm.c +++

Re: [patch] 19/n: trans-mem: middle end/misc patches (LAST PATCH)

2011-11-07 Thread Richard Guenther
On Mon, Nov 7, 2011 at 6:55 AM, Aldy Hernandez al...@redhat.com wrote: False.  You get the equivalent of bootstrap comparison mismatches. If we actually used tm during the bootstrap. The simplest thing to do is to change the hash this table uses. E.g. use the DECL_UID right from the start,

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

2011-11-07 Thread Olivier Hainque
On Nov 4, 2011, at 2:07 PM, Alan Modra wrote: Lots of bugs here. Most of them in TARGET_SPE_ABI code, but some also for other ABIs. ... Another bug we're running into here is an unwarranted move of the sp restore prior to register fetches despite an attempt at preventing that with a

RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 [ Working virtually from Hawaii tonight... :-) ] So while working on warnings for NULL pointer dereferences and seeing how many explicit implicit NULL pointer dereferences are left in the statement stream due to various VEC other oddities, I

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

2011-11-07 Thread Paul Brook
+strcpy (pattern, \fldmfdd\\t\); +strcat (pattern, +reg_names[REGNO (SET_DEST (XVECEXP (operands[0], 0, 0)))]); +strcat (pattern, \!, {\); +strcat (pattern, table[(REGNO (XEXP (XVECEXP (operands[0], 0, 1), 0)) + -

Re: vector garbaged collected while still in use

2011-11-07 Thread Richard Guenther
On Mon, Nov 7, 2011 at 8:53 AM, Xinliang David Li davi...@google.com wrote: I have seen compiler build error (segmentation fault) in libstdc++-v3. It turns out that a vector allocated in gc memory is GCed before the vector is released. The gc call is from a call to synethesize_method from

Re: Implement C1X _Alignas, _Alignof, max_align_t, stdalign.h

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 08:47, Jonathan Wakely wrote: On 7 November 2011 00:43, Jonathan Wakely wrote: On 7 November 2011 00:37, Joseph S. Myers wrote: On Mon, 7 Nov 2011, Jonathan Wakely wrote: On 6 November 2011 23:53, Joseph S. Myers wrote: As with stdnoreturn.h, the contents of stdalign.h

Re: RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Jakub Jelinek
On Mon, Nov 07, 2011 at 02:55:02AM -0700, Jeff Law wrote: [ Working virtually from Hawaii tonight... :-) ] ;) You might legitimately wonder how often this triggers. A GCC 4.6.0 checking-enabled compiler sees a .64% codesize improvement from this optimization. That's an awful lot of

Re: [commited] Add custom RTEMS lm32 multilibs

2011-11-07 Thread Rainer Orth
Hi Ralf, Index: gcc/config.gcc === --- gcc/config.gcc(revision 180973) +++ gcc/config.gcc(revision 180974) @@ -1539,6 +1539,7 @@ tm_file=dbxelf.h elfos.h ${tm_file} lm32/rtems.h rtems.h newlib-stdint.h

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

2011-11-07 Thread Iain Sandoe
On 7 Nov 2011, at 09:53, Olivier Hainque wrote: On Nov 4, 2011, at 2:07 PM, Alan Modra wrote: Lots of bugs here. Most of them in TARGET_SPE_ABI code, but some also for other ABIs. ... Another bug we're running into here is an unwarranted move of the sp restore prior to register fetches

Re: RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Richard Guenther
On Mon, Nov 7, 2011 at 11:16 AM, Richard Guenther richard.guent...@gmail.com wrote: On Mon, Nov 7, 2011 at 11:07 AM, Jakub Jelinek ja...@redhat.com wrote: On Mon, Nov 07, 2011 at 02:55:02AM -0700, Jeff Law wrote: [ Working virtually from Hawaii tonight...  :-) ] ;) You might legitimately

[vms,committed]: Partial support of VMS pragma

2011-11-07 Thread Tristan Gingold
Hi, the native c compiler for VMS, DEC-C, supports a few pragma that are extensively used in system header files. This patch add partial support for a few of them. Committed on trunk. Tristan. 2011-11-07 Tristan Gingold ging...@adacore.com * config.gcc (*-*-*vms*): Set

Re: Many testsuite failures on x86_64 due recent fix about f16cintrin.h header

2011-11-07 Thread Uros Bizjak
Hello! Attached patch fixes all remaining i386 testsuite failures. The header did say that we can include it also through immintrin.h ... 2011-11-07 Uros Bizjak ubiz...@gmail.com * config/i386/f16cintrin: Remove extra _X86INTRIN_H_INCLUDED check. Tested on x86_64-pc-linux-gnu

Re: [vms,committed]: Partial support of VMS pragma

2011-11-07 Thread Eric Botcazou
2011-11-07 Tristan Gingold ging...@adacore.com * config.gcc (*-*-*vms*): Set c_target_objs, cxx_target_objs. * config/vms/t-vms: Add vms-c.o rule. * config/vms/vms-c.c: New file. Note that there is #ifdef 0-ed code in the new file. -- Eric Botcazou

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

2011-11-07 Thread Kai Tietz
2011/8/1 Richard Guenther richard.guent...@gmail.com: On Fri, Jul 29, 2011 at 2:07 PM, Kai Tietz kti...@redhat.com wrote: Hello, this patch fixes regression of bug report PR middle-end/49806, which was caused due unhandled type-cast patterns reasoned by boolification of compares and

Re: [v3] define string::pop_back()

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 09:40, Eric Botcazou wrote: Thanks, I'll try to work out the right adjustment for the solaris symbols file but can't test it. The 2 new symbols are duplicated, because they are matched by a regexp:    ##_ZNSs[0-58-9][g-z]* (glob)    _ZNSs4nposE;    _ZNSs4rendEv;    

Re: [vms,committed]: Partial support of VMS pragma

2011-11-07 Thread Tristan Gingold
On Nov 7, 2011, at 11:28 AM, Eric Botcazou wrote: 2011-11-07 Tristan Gingold ging...@adacore.com * config.gcc (*-*-*vms*): Set c_target_objs, cxx_target_objs. * config/vms/t-vms: Add vms-c.o rule. * config/vms/vms-c.c: New file. Note that there is #ifdef 0-ed code in

Re: [v3] define string::pop_back()

2011-11-07 Thread Eric Botcazou
Won't that fail to match string::max_size? No, if it doesn't appear in the list, then it isn't matched by the regexp. For GNU I added a regex for that explicitly: +_ZNKSs8max_size*; +_ZNKSbIwSt11char_traitsIwESaIwEE8max_size*; What is GNU here? The Solaris version file is a

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

2011-11-07 Thread Alan Modra
On Mon, Nov 07, 2011 at 10:53:51AM +0100, Olivier Hainque wrote: Another bug we're running into here is an unwarranted move of the sp restore prior to register fetches despite an attempt at preventing that with a stack_tie instruction (VxWorks target).

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

2011-11-07 Thread Olivier Hainque
On Nov 7, 2011, at 12:02 PM, Alan Modra wrote: The failure can still be exposed on mainline with a minor adjustment to the C testcase quoted in the msg. Even after revision 181056? I'll double check but I'm pretty sure that this change cannot help the specific problem at hand. VxWorks is

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

2011-11-07 Thread Georg-Johann Lay
Georg-Johann Lay wrote: Denis Chertykov wrote: 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

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

2011-11-07 Thread Hans-Peter Nilsson
From: Andrew MacLeod amacl...@redhat.com Date: Mon, 7 Nov 2011 05:25:15 +0100 On 11/06/2011 07:38 PM, Hans-Peter Nilsson wrote: This (formally a change in the range 181027:181034) got me three libstdc++ regressions for cris-elf, which has no atomic support whatsoever (well, not the

[Patch - vms]: Add support of pragma __extern_prefix

2011-11-07 Thread Tristan Gingold
Hi, DEC-C for vms has '#pragma __extern_prefix' which is not unlike '#pragma extern_prefix' supported by DEC-C for Tru64. This patch adds supports for the VMS version (which can save and restore the current prefix). It reuses most of the current infrastructure, but to do so

Re: [v3] define string::pop_back()

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 10:57, Eric Botcazou wrote: Won't that fail to match string::max_size? No, if it doesn't appear in the list, then it isn't matched by the regexp. For GNU I added a regex for that explicitly: +    _ZNKSs8max_size*; +    _ZNKSbIwSt11char_traitsIwESaIwEE8max_size*; What

[PATCH, ARM][0/6] LDRD/STRD generation - introduction

2011-11-07 Thread Greta Yorsh
The following sequence of patches enables generation of LDRD/STRD instructions for Cortex-A15 with -O2 and for all Cortex-A CPUs with -Os when profitable. This almost always improves code size and is expected to improve performance on Cortex-A15. [0/6] LDRD/STRD generation - introduction (this

Problems with trans-mem branch on Solaris/x86 etc.

2011-11-07 Thread Rainer Orth
This weekend, I tried the trans-mem branch as of r180975 on i386-pc-solaris2.11. Doing so, I ran into a couple of problems. I'm reporting them here, both because bugzilla currently lacks a libitm component and in the hope that they can be fixed before the merge. * At first I tried a bootstrap

Re: [commited] Add custom RTEMS lm32 multilibs

2011-11-07 Thread Ralf Corsepius
On 11/07/2011 11:17 AM, Rainer Orth wrote: Hi Ralf, Index: gcc/config.gcc === --- gcc/config.gcc (revision 180973) +++ gcc/config.gcc (revision 180974) @@ -1539,6 +1539,7 @@ tm_file=dbxelf.h elfos.h ${tm_file}

[PATCH,ARM][3/6] Merge LDR/STR into LDRD/STRD with -Os

2011-11-07 Thread Greta Yorsh
This patch enables generation of LDRD/STRD for configurations in which LDM/STM would be preferable (e.g., Cortex-A15 with -Os or Cortex-A9 with -O2), but cannot be generated. There are several situations in which LDM/STM cannot be generated, but LDRD/STRD can be generated, for example memory

[PATCH,ARM][4/6] Improve peepholes for generating LDM with commutative operators

2011-11-07 Thread Greta Yorsh
This patch improves existing peephole optimizations. These peephole optimizations merge individual LDRs into LDM in the case that the order of registers in LDR instructions is not ascending, but the loaded values can be reordered because they are use of the loaded values is commutative. There are

[PATCH,ARM][5/6] Generate LDRD/STRD for internal memcpy

2011-11-07 Thread Greta Yorsh
This patch generates LDRD/STRD sequences for movmemqi and movmemdi when source and destination buffers are aligned. It is disabled when optimizing for code size, becasue LDM/STM is preferable in such cases. It is enabled for performance for CPUs that set prefer_ldrd_strd tune flag. This flag is

[PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Richard Guenther
This tries to find a way to prepend explicitly set command-line options by those implicitly set by the frontend (-fexceptions in this case). Unfortunately we don't seem to have a good way to extract this information easily, so for -fexceptions I hope all frontends set that during

Re: [PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Iain Sandoe
On 7 Nov 2011, at 12:17, Richard Guenther wrote: This tries to find a way to prepend explicitly set command-line options by those implicitly set by the frontend (-fexceptions in this case). Unfortunately we don't seem to have a good way to extract this information easily, so for

Re: [PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Eric Botcazou
This tries to find a way to prepend explicitly set command-line options by those implicitly set by the frontend (-fexceptions in this case). Unfortunately we don't seem to have a good way to extract this information easily, so for -fexceptions I hope all frontends set that during

Re: [PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Richard Guenther
On Mon, 7 Nov 2011, Iain Sandoe wrote: On 7 Nov 2011, at 12:17, Richard Guenther wrote: This tries to find a way to prepend explicitly set command-line options by those implicitly set by the frontend (-fexceptions in this case). Unfortunately we don't seem to have a good way to

Re: [PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Iain Sandoe
On 7 Nov 2011, at 12:40, Richard Guenther wrote: On Mon, 7 Nov 2011, Iain Sandoe wrote: It would also be nice to preserve the Objective-C flavor (GNU/ NeXT), since we have to make a guess for this in darwin.c when in lto. How is the default selected (that's not obvious to me).

Re: [PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Richard Guenther
On Mon, 7 Nov 2011, Eric Botcazou wrote: This tries to find a way to prepend explicitly set command-line options by those implicitly set by the frontend (-fexceptions in this case). Unfortunately we don't seem to have a good way to extract this information easily, so for -fexceptions I

Re: [Patch, Fortran, OOP] PR 50919: Don't use vtable for NON_OVERRIDABLE TBP

2011-11-07 Thread Janus Weil
Thus, I believe it is rather difficult to run into the ABI issue in the real world (due to technical but also due to usage reasons), but it is possible. Hence, I wanted to point out that there is an ABI issue, but I also don't oppose the committal as it is difficult to encounter the issue -

Re: [PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Richard Guenther
On Mon, 7 Nov 2011, Iain Sandoe wrote: On 7 Nov 2011, at 12:40, Richard Guenther wrote: On Mon, 7 Nov 2011, Iain Sandoe wrote: It would also be nice to preserve the Objective-C flavor (GNU/NeXT), since we have to make a guess for this in darwin.c when in lto. How is the

[PATCH, testsuite]: Do not cache check_effective_target_sync_{long_long,int_128} results

2011-11-07 Thread Uros Bizjak
Hello! Availability of long long and int_128 sync instructions can vary on x86, due to -m32, so don't cache the result. The problem can be triggered with: make -k check RUNTESTFLAGS=--target_board=unix\{,-m32\} FAIL: gcc.dg/atomic-compare-exchange-5.c (test for excess errors) WARNING:

Re: Massive breakage with your libgcc patches

2011-11-07 Thread Bernd Schmidt
On 11/03/11 20:20, Rainer Orth wrote: * config/c6x/t-elf (LIB2ADDEH): Set. * config/c6x/t-c6x-elf: Remove. It builds now, but parts of libgcc are missing. There's no sign of muldf3, for examples. Bernd

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

2011-11-07 Thread Denis Chertykov
Let's wait for http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01874.html Denis.

Re: [PATCH] Fix PR50999, serialize frontend specific flags (-fexceptions)

2011-11-07 Thread Richard Guenther
On Mon, 7 Nov 2011, Richard Guenther wrote: On Mon, 7 Nov 2011, Eric Botcazou wrote: This tries to find a way to prepend explicitly set command-line options by those implicitly set by the frontend (-fexceptions in this case). Unfortunately we don't seem to have a good way to extract

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Jonathan Wakely
This provides a working thread::hardware_concurrency on platforms that support pthread_num_processors_np or the hw.ncpu sysctl, but by testing for the features in configure rather than hardcoding OS macro tests in thread.cc * acinclude.m4 (GLIBCXX_CHECK_SC_NPROC_ONLN): Define.

Re: RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Tom Tromey
Jeff == Jeff Law l...@redhat.com writes: Jeff First, it's perfectly fine to have a NULL pointer dereference in a Jeff program as long as that code is never executed. Once the code is Jeff executed, we've entered the realm of undefined behavior. Jeff Thus in a conforming program we can safely

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Iain Sandoe
On 7 Nov 2011, at 13:45, Jonathan Wakely wrote: This provides a working thread::hardware_concurrency on platforms that support pthread_num_processors_np or the hw.ncpu sysctl, but by testing for the features in configure rather than hardcoding OS macro tests in thread.cc if the system

[trans-mem] Fix instantiation of transaction expressions.

2011-11-07 Thread Torvald Riegel
I stumbled upon this when working on noexcept. When txn expressions were processed in tsubst_expr(), the previous code incorrectly created empty transaction statements because no statements got added to the statement list used by (begin|finish)_transaction_stmt(). Also, return __transaction_atomic

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 14:10, Iain Sandoe wrote: On 7 Nov 2011, at 13:45, Jonathan Wakely wrote: This provides a working thread::hardware_concurrency on platforms that support pthread_num_processors_np or the hw.ncpu sysctl, but by testing for the features in configure rather than hardcoding OS

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 14:16, Jonathan Wakely wrote: On 7 November 2011 14:10, Iain Sandoe wrote: On 7 Nov 2011, at 13:45, Jonathan Wakely wrote: This provides a working thread::hardware_concurrency on platforms that support pthread_num_processors_np or the hw.ncpu sysctl, but by testing for

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Iain Sandoe
On 7 Nov 2011, at 14:16, Jonathan Wakely wrote: On 7 November 2011 14:10, Iain Sandoe wrote: On 7 Nov 2011, at 13:45, Jonathan Wakely wrote: This provides a working thread::hardware_concurrency on platforms that support pthread_num_processors_np or the hw.ncpu sysctl, but by testing for

[patch]: support of VMS module include files

2011-11-07 Thread Tristan Gingold
Hi, on VMS systems, the header files are in fact members of text library files. Obviously, we don't want to support such a weirdness in GCC, and we require that the user unpack the header libraries. However, it is still a good idea to keep each header library in a subdirectory. There are 2

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

2011-11-07 Thread Andrew MacLeod
On 11/07/2011 06:17 AM, Hans-Peter Nilsson wrote: Sorry, no it didn't help. No additional regressions either though, but the undef'd reference changed to two: Excess errors: /tmp/atreg1/gccobj/cris-elf/libstdc++-v3/include/bits/atomic_base.h:273: undefined reference to

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 14:23, Iain Sandoe wrote: On 7 Nov 2011, at 14:16, Jonathan Wakely wrote: On 7 November 2011 14:10, Iain Sandoe wrote: On 7 Nov 2011, at 13:45, Jonathan Wakely wrote: This provides a working thread::hardware_concurrency on platforms that support

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

2011-11-07 Thread Richard Guenther
On Sun, Nov 6, 2011 at 11:12 PM, Kai Tietz kti...@redhat.com wrote: Hello, By this patch branch-cost optimization is moved from tree AST to cfgexpand from gimple to RTL.  By this we are able to do better optimization on conditionals simliar for all targets and do the final transition for

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Iain Sandoe
On 7 Nov 2011, at 14:23, Iain Sandoe wrote: On 7 Nov 2011, at 14:16, Jonathan Wakely wrote: On 7 November 2011 14:10, Iain Sandoe wrote: On 7 Nov 2011, at 13:45, Jonathan Wakely wrote: This provides a working thread::hardware_concurrency on platforms that support

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

2011-11-07 Thread Richard Guenther
On Sun, Nov 6, 2011 at 11:17 PM, Kai Tietz kti...@redhat.com wrote: Hello, the second patch extends the tree-ssa-ifcombine pass so, that it chains up simple if-and/or-if patterns via associative bitwise-and/or operations.  This allows for example optimization for cases like: if (c == 0)

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

2011-11-07 Thread Andrew MacLeod
On 11/06/2011 07:38 PM, Hans-Peter Nilsson wrote: This (formally a change in the range 181027:181034) got me three libstdc++ regressions for cris-elf, which has no atomic support whatsoever (well, not the version represented in cris-elf), so something is amiss at the bottom of the default

Re: cxx-mem-model merge gcc part1 patch 3/8

2011-11-07 Thread Tom de Vries
On 11/06/2011 04:04 PM, Andrew MacLeod wrote: (get_atomic_op_for_code): New. Return an opcode table entry. I'm getting these errors with a MIPS cross build: ... /scratch/vries/b4/ref-mips-ml-11-11-07/src/gcc-mainline/gcc/optabs.c: In function 'get_atomic_op_for_code':

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Jonathan Wakely
On 7 November 2011 14:40, Iain Sandoe wrote: On 7 Nov 2011, at 14:23, Iain Sandoe wrote: On 7 Nov 2011, at 14:16, Jonathan Wakely wrote: On 7 November 2011 14:10, Iain Sandoe wrote: On 7 Nov 2011, at 13:45, Jonathan Wakely wrote: This provides a working thread::hardware_concurrency on

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

2011-11-07 Thread David Edelsohn
On Mon, Nov 7, 2011 at 4:53 AM, Olivier Hainque hain...@adacore.com wrote: There are lots of subtle inter-section dependencies and redundancies in emit_epilogue, which has grown pretty difficult to understand IMHO. I can see two tracks to improve things in this area: - Concentrate on the

Re: implementation of std::thread::hardware_concurrency()

2011-11-07 Thread Iain Sandoe
On 7 Nov 2011, at 14:52, Jonathan Wakely wrote: On 7 November 2011 14:40, Iain Sandoe wrote: so there's a reason to use the systlbyname (and use hw.logicalcpu or similar, maybe). [unless that's just a buggy sysconf] Well if that's how they want to play it then I'm not even going to think

Re: Many testsuite failures on x86_64 due recent fix about f16cintrin.h header

2011-11-07 Thread Quentin Neill
On Mon, Nov 7, 2011 at 4:25 AM, Uros Bizjak ubiz...@gmail.com wrote: Hello! Attached patch fixes all remaining i386 testsuite failures. The header did say that we can include it also through immintrin.h ... 2011-11-07  Uros Bizjak  ubiz...@gmail.com        * config/i386/f16cintrin: Remove

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

2011-11-07 Thread Hans-Peter Nilsson
From: Andrew MacLeod amacl...@redhat.com Date: Mon, 7 Nov 2011 15:44:47 +0100 Actually, this target has no lock free support whatsoever? ie, no compare_and_swap instruction, nor an implementation of sync_lock_test_and_set and sync_lock_release? In CRIS versions where such support would

[PATCH, testsuite]: Fix FAIL: gcc.target/i386/pr49781-1.c scan-assembler-not for atom

2011-11-07 Thread Uros Bizjak
Hello! 2011-11-07 Uros Bizjak ubiz...@gmail.com * gcc.target/i386/pr49781-1.c (dg-options): Add -mtune=generic. Tested on x86_64-pc-linux-gnu, committed to mainline SVN. Uros. Index: gcc.target/i386/pr49781-1.c === ---

[Patch, fortran] Documentation fix for MCLOCK{,8}, TIME{,8}

2011-11-07 Thread Janne Blomqvist
Hi, The functions clock() and time() are part of the C standard library, they are not Unix specific. Committed the patch below as obvious. 2011-11-07 Janne Blomqvist j...@gcc.gnu.org * intrinsic.texi (MCLOCK, MCLOCK8, TIME, TIME8): Functions clock and time are part of the C

Re: [PATCH] Gather vectorization (PR tree-optimization/50789, take 2)

2011-11-07 Thread Uros Bizjak
On Fri, Nov 4, 2011 at 2:22 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Nov 04, 2011 at 12:21:49PM +0100, Richard Guenther wrote: Ok.  I guess it's ok to use builtins for now - I didn't think of the memory reference issue ;) Based on IRC discussion I'm posting an updated patch (both

Re: RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/07/11 06:53, Tom Tromey wrote: Jeff == Jeff Law l...@redhat.com writes: Jeff First, it's perfectly fine to have a NULL pointer dereference in a Jeff program as long as that code is never executed. Once the code is Jeff executed, we've

Re: RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Richard Guenther
On Mon, Nov 7, 2011 at 4:43 PM, Jeff Law l...@redhat.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 11/07/11 06:53, Tom Tromey wrote: Jeff == Jeff Law l...@redhat.com writes: Jeff First, it's perfectly fine to have a NULL pointer dereference in a Jeff program as long as that

Re: [patch] 19/n: trans-mem: middle end/misc patches (LAST PATCH)

2011-11-07 Thread Aldy Hernandez
This won't work - DECL_UIDs are not stable -g vs. -g0 - only their _order_ is stable - thus you won't get comparison fails with code generated dependent on DECL_UID order, but you will if you depend on the DECL_UID value (which you do by using it as a hash). And we will still generate

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

2011-11-07 Thread Kai Tietz
2011/11/7 Richard Guenther richard.guent...@gmail.com: On Sun, Nov 6, 2011 at 11:12 PM, Kai Tietz kti...@redhat.com wrote: Hello, By this patch branch-cost optimization is moved from tree AST to cfgexpand from gimple to RTL.  By this we are able to do better optimization on conditionals

Re: RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Tom Tromey
Jeff == Jeff Law l...@redhat.com writes: Jeff So, presumably there's no way to know we're throwing to Jeff NullPointerException from the exception information attached to the Jeff statement or BB? If not I could disable if the statement with the Jeff memory op throws anywhere. It's not ideal,

Re: [C++ PATCH] PR c++/45114 - Support alias templates

2011-11-07 Thread Dodji Seketeli
On 11/05/2011 07:36 PM, Dodji Seketeli wrote: +#define TYPE_DECL_NAMES_ALIAS_TEMPLATE_P(NODE) \ This doesn't seem to be needed anymore. Removed, thanks. +dump_alias_template_specialization (tree t, int flags) +{ + gcc_assert (alias_template_specialization_p

[Patch,AVR]: Clean-up load non-integer constants

2011-11-07 Thread Georg-Johann Lay
This is a patch that cleans up the code for loading CONST or SYMBOL_REF into a register. This code is also needed to be able to load symbols into PSImode registers. Up to now, this is not needed because there are no 24-bit symbols yet, but it is needed for 24-bit address spaces. Fiddling with

Patch to fix optabs.c when SWITCHABLE_TARGETS is defined.

2011-11-07 Thread Andrew MacLeod
On 11/07/2011 09:43 AM, Tom de Vries wrote: On 11/06/2011 04:04 PM, Andrew MacLeod wrote: (get_atomic_op_for_code): New. Return an opcode table entry. I'm getting these errors with a MIPS cross build: ... /scratch/vries/b4/ref-mips-ml-11-11-07/src/gcc-mainline/gcc/optabs.c: In

Re: C++/c-common PATCH for c++/35688 (wrong visibility of template instantiation)

2011-11-07 Thread Joseph S. Myers
On Sun, 6 Nov 2011, Jason Merrill wrote: The function constrain_visibility_for_template tries to set the visibility of a template instantiation properly by giving it the minimum visibility of the template itself and the template arguments. But this PR points out that we were failing to do

Re: Mark objects death at end of scope

2011-11-07 Thread Michael Matz
Hi, On Thu, 3 Nov 2011, Richard Guenther wrote: Otherwise the patch looks ok, but given the Ada issue let's wait until that is sorted out in some way. That also gives others the chance to comment on the patch. So, this is what I came up with. As discussed I didn't do the uintptr_t case

Re: [patch] 19/n: trans-mem: middle end/misc patches (LAST PATCH)

2011-11-07 Thread Richard Guenther
On Mon, Nov 7, 2011 at 4:52 PM, Aldy Hernandez al...@redhat.com wrote This won't work - DECL_UIDs are not stable -g vs. -g0 - only their _order_ is stable - thus you won't get comparison fails with code generated dependent on DECL_UID order, but you will if you depend on the DECL_UID value

Re: [patch] 19/n: trans-mem: middle end/misc patches (LAST PATCH)

2011-11-07 Thread Aldy Hernandez
I can certainly move this to expand_call_stmt() if you prefer. Do you have an objection to the RTL walk? This isn't my code, but I'm open to suggestions on an alternative to implement. It just catched my eye... moving it to expand_call_stmt would be nice indeed, but I was suggesting to add

Re: [PATCH] Add an intermediate coverage format for gcov

2011-11-07 Thread Mike Stump
On Nov 6, 2011, at 11:04 PM, Sharad Singhai wrote: Sorry about the delay. I have updated the patch to output demangled names under a new option (-m) and added a test case. I like it, now you just need a gcov type person to like it. :-)

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

2011-11-07 Thread Kai Tietz
2011/11/7 Richard Guenther richard.guent...@gmail.com: On Sun, Nov 6, 2011 at 11:17 PM, Kai Tietz kti...@redhat.com wrote: Hello, the second patch extends the tree-ssa-ifcombine pass so, that it chains up simple if-and/or-if patterns via associative bitwise-and/or operations.   This allows

Re: [patch] 19/n: trans-mem: middle end/misc patches (LAST PATCH)

2011-11-07 Thread Richard Henderson
On 11/07/2011 01:44 AM, Richard Guenther wrote: It just catched my eye... moving it to expand_call_stmt would be nice indeed, but I was suggesting to add that note where we produce the CALL rtx, not sure if that's reasonably straight-forward (I suppose there was a reason to go with the hack

Re: RFA: New pass to delete unexecutable paths in the CFG

2011-11-07 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I also wonder if instead of a new pass control-dependent DCE can be taught of this? It could. I'm not sure it really buys us anything except maybe being able to reuse the pdom control-dependency analysis. It actually more naturally belongs in

Re: [patch] 19/n: trans-mem: middle end/misc patches (LAST PATCH)

2011-11-07 Thread Richard Henderson
On 11/07/2011 08:04 AM, Aldy Hernandez wrote: @@ -1812,6 +1844,8 @@ expand_call_stmt (gimple stmt) bool builtin_p; size_t i; + mark_transaction_restart_calls (stmt); + if (gimple_call_internal_p (stmt)) { expand_internal_call (stmt); You're calling it too early,

  1   2   3   >