Re: Gthreads patch to disable static initializer macros

2012-02-05 Thread Jakub Jelinek
On Sun, Feb 05, 2012 at 08:26:22PM +, Jonathan Wakely wrote: > This has been testing on darwin, if Rainer tests it successfully on > Tru64 will the gthr-posix.h change be acceptable? Ok with a suitable ChangeLog entry. > diff --git a/libgcc/gthr-posix.h b/libgcc/gthr-posix.h > index 46054f6..

Re: [Patch] PR 52118 (Description of Wunused-local-typedefs)

2012-02-05 Thread Jason Merrill
OK. Jason

Re: [patch] document which edition of Effective C++ is used for -Weffc++

2012-02-05 Thread Jonathan Wakely
On 6 February 2012 01:04, Gerald Pfeifer wrote: > On Mon, 6 Feb 2012, Jonathan Wakely wrote: >>         PR c++/48680 >>         * doc/invoke.texi (C++ Dialect Options): Use @option markup for >>         -Weffc++ and specify guidelines come from second edition. >> >> Tested with 'make doc html' >> O

Re: [patch] document which edition of Effective C++ is used for -Weffc++

2012-02-05 Thread Gerald Pfeifer
On Mon, 6 Feb 2012, Jonathan Wakely wrote: > PR c++/48680 > * doc/invoke.texi (C++ Dialect Options): Use @option markup for > -Weffc++ and specify guidelines come from second edition. > > Tested with 'make doc html' > OK for trunk? Yes. The first is a bug fix, the second

[patch] document which edition of Effective C++ is used for -Weffc++

2012-02-05 Thread Jonathan Wakely
PR c++/48680 * doc/invoke.texi (C++ Dialect Options): Use @option markup for -Weffc++ and specify guidelines come from second edition. Tested with 'make doc html' OK for trunk? diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi index 6105710..6c61e53 100644 --- a/gcc/do

libstdc++/52104 - fix linker error for non-TLS targets

2012-02-05 Thread Jonathan Wakely
The out-of-line destructor in future.cc uses std::call_once which uses a lambda on TLS targets. Because we compile with -fno-implicit-templates an explicit instantiation is needed, but can't be done because the closure type created by the lambda can't be named. This puts the destructor inline in t

Re: Gthreads patch to disable static initializer macros

2012-02-05 Thread Jonathan Wakely
On 5 February 2012 23:53, Jack Howarth wrote: > > There are no unexpected libstdc++ testsuite regressions on > x86_64-apple-darwin11 with this > patch applied to current gcc trunk. Thanks, Jack. If I can't get approval for the change I'll workaround it in libstdc++, but IMHO it would be cleaner

Re: Gthreads patch to disable static initializer macros

2012-02-05 Thread Jack Howarth
On Sun, Feb 05, 2012 at 08:26:22PM +, Jonathan Wakely wrote: > PRs libstdc++/51296 and libstdc++/51906 are both caused by problems > with the Pthreads static initializer macros such as > PTHREAD_MUTEX_INITIALIZER. > > On Tru64 PTHREAD_MUTEX_INITIALIZER and PTHREAD_COND_INITIALIZER can > only b

New German PO file for 'gcc' (version 4.7-b20120128)

2012-02-05 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the German team of translators. The file is available at: http://translationproject.org/latest/gcc/de.po (This file, 'gcc-4.7-b20120128.de.po', h

Added test case for PR 32373, missed vectorization with equivalence

2012-02-05 Thread Thomas Koenig
Hello world, I added the attached test case from the PR after verifying that all loops mentioned in the PR are now vectorized. The PR is now closed. 2012-02-05 Thomas König PR fortran/32373 * gfortran.dg/vect/vect-8.f90: New test case. ! PR fortran/32373 - eight of these lo

Re: [wwwdocs] Add missing Steering Committee member

2012-02-05 Thread Joseph S. Myers
On Sun, 5 Feb 2012, Gerald Pfeifer wrote: > It has been brought to my attention that we have been missing > a prominent member of the Steering Committee on our web page. Should the list distinguish which members are or are not GNU maintainers of GCC? fencepost:/gd/gnuorg/maintainers lists only

Gthreads patch to disable static initializer macros

2012-02-05 Thread Jonathan Wakely
PRs libstdc++/51296 and libstdc++/51906 are both caused by problems with the Pthreads static initializer macros such as PTHREAD_MUTEX_INITIALIZER. On Tru64 PTHREAD_MUTEX_INITIALIZER and PTHREAD_COND_INITIALIZER can only be used for statically-initialized variables, as allowed by POSIX currently, b

Re: [patch, fortran] Fix PR 48847, bogus "unused parameter" warning

2012-02-05 Thread Thomas Koenig
Hi Steve, You need a space after TREE_NO_WARNING. I think that the patch can be applied now given its simplicity. Committed as rev. 183916 with the space. Thanks a lot for the review! Thomas

Re: [Patch, fortran] PR52102 - [OOP] Wrong result with ALLOCATE of CLASS components with array constructor SOURCE-expr

2012-02-05 Thread Paul Richard Thomas
Committed as revision 183915 after green light from Tobias on #gfortran. 2012-02-05 Paul Thomas * trans-array.c (gfc_array_allocate): Zero memory for all class array allocations. * trans-stmt.c (gfc_trans_allocate): Ditto for class scalars. PR fortran/52102

Re: Documenting the MIPS changes in 4.7

2012-02-05 Thread Andrew Pinski
On Sun, Feb 5, 2012 at 7:13 AM, Richard Sandiford wrote: > I've committed this patch to describe the MIPS changes in GCC 4.7. > Corrections, comments, and help with wordsmithing are all welcome. > > Thanks, > Richard > > > Index: htdocs/gcc-4.7/changes.html > ==

[v3] libstdc++/51956 improve pretty printers for shared_ptr and weak_ptr

2012-02-05 Thread Jonathan Wakely
PR libstdc++/51956 * python/libstdcxx/v6/printers.py (StdPointerPrinter): Rename to... (SharedPointerPrinter): This. Also show weak count. * testsuite/libstdc++-prettyprinters/shared_ptr.cc: New. tested x86_64-linux, committed to trunk. diff --git a/libstdc++-v3/pyt

Re: [patch, fortran] Fix PR 48847, bogus "unused parameter" warning

2012-02-05 Thread Steve Kargl
On Sun, Feb 05, 2012 at 11:12:01AM +0100, Thomas Koenig wrote: > Hello world, > > the attached, rather obvious patch fixed a bogus "unused parameter" > warning with procedure dummy arguments and warns about these if > they occur. > > Regression-tested. OK for trunk? Or rather wait until 4.8? >

Re: [PATCH 4.8, i386]: Enable post-reload compare optimization pass (PR28685)

2012-02-05 Thread Uros Bizjak
On Sun, Feb 5, 2012 at 6:27 PM, Steven Bosscher wrote: >>        PR target/28685 >>        * config/i386/i386.c (TARGET_FLAGS_REGNUM): New. >> > > Hmm, how is this (apparently new in 2011) TARGET_FLAGS_REGNUM > different from the older targetm.fixed_condition_code_regs? This is how backend enabl

Re: [wwwdocs] Add missing Steering Committee member

2012-02-05 Thread Gerald Pfeifer
On Sun, 5 Feb 2012, Paolo Carlini wrote: >> +Richard Stallmann (Free Software Foundation) > s/Stallmann/Stallman Oops, of course! Too much German recently. :-o Thanks, Paolo. Fixed right away. Gerald

Re: [PATCH 4.8, i386]: Enable post-reload compare optimization pass (PR28685)

2012-02-05 Thread Steven Bosscher
On Sun, Feb 5, 2012 at 4:27 PM, Uros Bizjak wrote: > 2012-02-05  Uros Bizjak   > >        PR target/28685 >        * config/i386/i386.c (TARGET_FLAGS_REGNUM): New. > Hmm, how is this (apparently new in 2011) TARGET_FLAGS_REGNUM different from the older targetm.fixed_condition_code_regs? Ciao! St

Re: [wwwdocs] Add missing Steering Committee member

2012-02-05 Thread Paolo Carlini
On 02/05/2012 06:15 PM, Gerald Pfeifer wrote: It has been brought to my attention that we have been missing a prominent member of the Steering Committee on our web page. Fixed thusly. Gerald Index: steering.html === RCS file: /cvs

[wwwdocs] Add missing Steering Committee member

2012-02-05 Thread Gerald Pfeifer
It has been brought to my attention that we have been missing a prominent member of the Steering Committee on our web page. Fixed thusly. Gerald Index: steering.html === RCS file: /cvs/gcc/wwwdocs/htdocs/steering.html,v retrieving r

[PATCH 4.8, i386]: Enable post-reload compare optimization pass (PR28685)

2012-02-05 Thread Uros Bizjak
Hello! Attached patch enables post-reload compare optimization pass for x86 targets. As shown in following test, patch removes redundant compare: --cut here-- int test(int a, int b) { int lt = a < b; int eq = a == b; if (lt) return 1; return eq; } --cut here-- gcc -O2 on x86_64, a c

Documenting the MIPS changes in 4.7

2012-02-05 Thread Richard Sandiford
I've committed this patch to describe the MIPS changes in GCC 4.7. Corrections, comments, and help with wordsmithing are all welcome. Thanks, Richard Index: htdocs/gcc-4.7/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/c

[MIPS, committed] Fix lengths of MIPS16 call patterns

2012-02-05 Thread Richard Sandiford
gcc.dg/fixed-point/convert.c and libgomp.fortran/vla3.f90 were failing on MIPS16 for mips64-linux-gnu with PLTs enabled because we were underestimating the length of a direct JAL. Tested on mips64-linux-gnu and various ELF targets. Applied. Richard gcc/ * config/mips/mips.md (sibcall_in

Re: [PATCH] MIPS16 TLS support for GCC

2012-02-05 Thread Richard Sandiford
Richard Sandiford writes: > "Maciej W. Rozycki" writes: >>> If there's still some concern that __mips16_rdhwr might not have >>> the right ABI, then maybe we should simply emit a link-once function >>> in each object that needs it. We could then switch to another function >>> (and another API) w

[MIPS, committed] Expect MIPS16 to use memcpy for short strings

2012-02-05 Thread Richard Sandiford
Although I changed non-MIPS16 code to allow larger inline copies: http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00053.html MIPS16 code is still for now expected to use memcpy. This patch adjusts the testsuite accordingly. Tested on mips64-linux-gnu and various ELF targets. Applied. Richard

[MIPS, committed] XFAIL gcc.dg/pr48774.c for MIPS REL targets

2012-02-05 Thread Richard Sandiford
gcc.dg/pr48774.c fails on MIPS REL targets because of an orphaned HI16 relocation; see PR 52125 for details. This isn't a regression, so I'm just going to XFAIL it for now. Tested on mips64-linux-gnu and various *-elf targets. Applied. Richard gcc/testsuite/ PR target/52125 *

[MIPS, committed] Tidy atomic-related failures for mips-elf

2012-02-05 Thread Richard Sandiford
Plain MIPS I testing on mips-elf has quite a few atomic-related failures, due to the lack of LL and SC. Fixed as below. Tested on mips-elf, mips64-linux-gnu, and a host of others. Applied. Richard gcc/testsuite/ * lib/target-supports.exp (check_effective_target_mips_llsc): New.

Re: [v3] adjust weak_ptr testcase

2012-02-05 Thread Jonathan Wakely
On 23 December 2011 17:33, Jonathan Wakely wrote: > This modifies the test to PASS when the expected type of exception is > caught, instead of being XFAIL due to uncaught exception. > > Tested x86_64-linux, committed to trunk. > >        * testsuite/tr1/2_general_utilities/shared_ptr/cons/ >      

Re: [patch] should not define bool, true or false as macros for C++

2012-02-05 Thread Jonathan Wakely
On 4 February 2012 23:35, Gerald Pfeifer wrote: > For what it's worth, I strongly suggest that you only define those when > __cpluplus is pre-C++11. > > There is simply too much software out there which will run into this Really? Why would any C++ code assume "bool" is defined as a macro? It's bee

[patch] [4.8] Support pattern recognition in SLP

2012-02-05 Thread Ira Rosen
Hi, This patch adds a support of pattern recognition in basic block SLP. Bootstrapped and tested on powerpc64-suse-linux. Ira ChangeLog: * tree-vectorizer.h (vect_pattern_recog): Add new argument. * tree-vect-loop.c (vect_analyze_loop_2): Update call to vect_pattern_re

[v3] libstdc++/52119

2012-02-05 Thread Paolo Carlini
Hi, the library bits seem straightforward enough to go in now. Of course the substantive issue seems C++ front end diagnostics, post 4.7.0 material. Sanity checked x86_64-linux. Thanks, Paolo. /// 2012-02-05 Jeffrey Yasskin Paolo Carlini PR libstdc++

New Swedish PO file for 'gcc' (version 4.7-b20120128)

2012-02-05 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-4.7-b20120128.sv.po',

[patch, fortran] Fix PR 48847, bogus "unused parameter" warning

2012-02-05 Thread Thomas Koenig
Hello world, the attached, rather obvious patch fixed a bogus "unused parameter" warning with procedure dummy arguments and warns about these if they occur. Regression-tested. OK for trunk? Or rather wait until 4.8? Thomas 2012-02-05 Thomas König PR fortran/48847

[Patch, Fortran, committed] PR51972 - Extended checking of gfortran.dg/class_48.f90

2012-02-05 Thread Tobias Burnus
The patch for PR 51972 fixed some issues with intrinsic assignment, which involved polymorphic components. However, I couldn't check the correctness in some cases as I run ICEs. Those were fixed by Mikael's recent patch (cf. PR 51754 et alia). The committed (Rev. ) and attached patch adds the