[PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​p.

2011-05-08 Thread Revital Eres
Hello, (sorry for multiple copies of this email) This small fix was inserted to skip DEBUG_INSNs while recognizing doloop pattern in loop-doloop.c file. It's a fix for the already approved do-loop patch (not in mainline yet, http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in

[PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloo​​​​p.

2011-05-08 Thread Revital1 Eres
Hello, (sorry for multiple copies of this email) This small fix was inserted to skip DEBUG_INSNs while recognizing doloop pattern in loop-doloop.c file. It's a fix for the already approved do-loop patch (not in mainline yet, http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in

Re: [PATCH, SMS 3/3] Skip DEBUG_INSN in loop-doloop.c

2011-05-08 Thread Zdenek Dvorak
Hi, This small fix was inserted to skip DEBUG_INSNs while recognizing doloop pattern in loop-doloop.c file. It's a fix for the already approved do-loop patch (not in mainline yet, http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01718.html) in loop-doloop.c The patch was tested together with

Re: [SPARC] Remove unused macros

2011-05-08 Thread Eric Botcazou
This patch remove some unused macros from sparc.h. The RTX_OK_FOR_OFFSET_P and RTX_OK_FOR_OLO10_P macros is used only in sparc_legitimate_address_p function and moved to sparc.c. Thanks for spotting this. OK to install? * config/sparc/sparc.h (REG_OK_FOR_INDEX_P,

Re: [rs6000] Fix thinko in output_profile_hook

2011-05-08 Thread Eric Botcazou
2011-03-24  Eric Botcazou  ebotca...@adacore.com * config/rs6000/rs6000.c (output_profile_hook): Fix thinko. Applied as obvious. -- Eric Botcazou

Re: [AVR] Couple of tweaks

2011-05-08 Thread Eric Botcazou
Have you run the regression test suite for the AVR for this patch? The compiler doesn't even build without the print_operand_address hunk... I've installed this hunk as obvious to make some progress. -- Eric Botcazou

C++ Patch for PR c++/48859

2011-05-08 Thread Fabien Chêne
Hello, Here the problem is that we were recursing even if there was a user defined constructor. In fact, the check was only done at the top, not in nested fields. If there is a user defined constructor, uninitialized const or reference members are diagnosed elsewhere. Bootstraped and tested on

Re: Ping: Make 128 bits the default vector size for NEON

2011-05-08 Thread Ira Rosen
On 6 May 2011 13:29, Richard Earnshaw rearn...@arm.com wrote: On Thu, 2011-04-21 at 09:02 +0300, Ira Rosen wrote: http://gcc.gnu.org/ml/gcc-patches/2011-03/msg02172.html The last version: ChangeLog:      * doc/invoke.texi (preferred-vector-size): Document.      * params.h

Re: [Patch, libfortran] Thread safety and simplification of error printing

2011-05-08 Thread Janne Blomqvist
On Sat, May 7, 2011 at 19:35, Janne Blomqvist blomqvist.ja...@gmail.com wrote: Hi, the error printing functionality (in io/unix.c) st_printf and st_vprintf are not thread-safe as they use a static buffer. However, since these routines are used when something has gone wrong, we shouldn't use

Re: Ping: Make 128 bits the default vector size for NEON

2011-05-08 Thread Gerald Pfeifer
On Sun, 8 May 2011, Ira Rosen wrote: How about ARM specific flag similar to -mprefer-avx128 (not tested)? If this goes in, please also update gcc-4.7/changes.html. Thanks, Gerald

Re: Ping: Make 128 bits the default vector size for NEON

2011-05-08 Thread Ira Rosen
On 8 May 2011 15:02, Gerald Pfeifer ger...@pfeifer.com wrote: On Sun, 8 May 2011, Ira Rosen wrote: How about ARM specific flag similar to -mprefer-avx128 (not tested)? If this goes in, please also update gcc-4.7/changes.html. Do you mean that the new flag should be documented? This patch

[wwwdocs] Remove redundant bgcolor= from projects/tree-ssa/tree-browser.html

2011-05-08 Thread Gerald Pfeifer
This has been redundant for 8+ years, and in the course of simplifying things globally I noticed and yanked this now. Gerald Index: projects/tree-ssa/tree-browser.html === RCS file:

Re: GAS GCC FAQ query

2011-05-08 Thread Gerald Pfeifer
On Fri, 6 May 2011, Jonathan Wakely wrote: I would propose to clarify as: To ensure that GCC finds the GNU assembler (or the GNU linker), I see no harm in that change, Gerald, what do you think? Agreed. Things would have been different twenty years ago, but these days using linker is a lot

Re: GAS GCC FAQ query

2011-05-08 Thread Richard Kenner
Agreed. Things would have been different twenty years ago, but these days using linker is a lot more natural and common (as a grep in gcc/doc confirms, too). Even 20 years ago, I think linker would have been the more natural word. I remember linker from my IBM days in the early 80's.

Re: [Patch, libfortran] Thread safety and simplification of error printing

2011-05-08 Thread N.M. Maclaren
On May 8 2011, Janne Blomqvist wrote: the error printing functionality (in io/unix.c) st_printf and st_vprintf are not thread-safe as they use a static buffer. ... While this patch makes error printing thread-safe, it's no longer async-signal-safe as the stderr lock might lead to a deadlock.

Re: [PATCH] Typed DWARF stack

2011-05-08 Thread H.J. Lu
On Tue, May 3, 2011 at 6:33 AM, H.J. Lu hjl.to...@gmail.com wrote: On Sat, Apr 16, 2011 at 1:11 AM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Mar 25, 2011 at 12:32:37PM +0100, Jakub Jelinek wrote: As I said in my GCC Summit talk, currently we just give up on any floating

[C++ patch, committed as obvious] PR 48816

2011-05-08 Thread Paolo Carlini
Hi, committed to mainline. Paolo. // 2011-05-08 Paolo Carlini paolo.carl...@oracle.com PR c++/48816 * cxx-pretty-print.c (pp_cxx_template_declaration): Remove effectively unused variable. Index: cxx-pretty-print.c

Re: Propagate BB predicates in ipa-inline-analysis

2011-05-08 Thread H.J. Lu
On Fri, Apr 29, 2011 at 2:18 PM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this more or less complettes the infrastructure for predicates by adding logic propagating predicates across CFG.  I also added switch statement handling and __builtin_constant_p construct, so we understand functions using

Re: [Patch, libfortran] Thread safety and simplification of error printing

2011-05-08 Thread N.M. Maclaren
Sorry - I should have clarified that ANYTHING that can't be used independently in multiple threads and at multiple levels in the same thread counts as a resource, and that includes stderr. Regards, Nick Maclaren.

[C++ Patch] PRs 48737 48744

2011-05-08 Thread Paolo Carlini
Hi, as far as I can see this is just another case where we want to pass down more consistently the complain argument in order to avoid hard errors in sfinae contexts. In particular, we don't want hard errors from reshape_init itself (in order to fix 48737) and we want digest_init_r to

Re: [Patch, libfortran] Thread safety and simplification of error printing

2011-05-08 Thread Janne Blomqvist
On Sun, May 8, 2011 at 14:40, Janne Blomqvist blomqvist.ja...@gmail.com wrote: On Sat, May 7, 2011 at 19:35, Janne Blomqvist blomqvist.ja...@gmail.com wrote: Hi, the error printing functionality (in io/unix.c) st_printf and st_vprintf are not thread-safe as they use a static buffer.

Re: [Patch, libfortran] Thread safety and simplification of error printing

2011-05-08 Thread Janne Blomqvist
On Sun, May 8, 2011 at 16:42, N.M. Maclaren n...@cam.ac.uk wrote: On May 8 2011, Janne Blomqvist wrote: the error printing functionality (in io/unix.c) st_printf and st_vprintf are not thread-safe as they use a static buffer. ... While this patch makes error printing thread-safe, it's no

[PATCH, IRA]: Fix PR 48927 - Issues with enable attribute and IRA register preferences

2011-05-08 Thread Uros Bizjak
Hello! Attached patch fixes changed register allocation where enabled attribute is used. The core of the problem was with IRA, where IRA does not look at enabled attribute when scanning through alternatives string to perform various tasks (including register allocation preferences). Attached

Re: [Patch, libfortran] Thread safety and simplification of error printing

2011-05-08 Thread N.M. Maclaren
On May 8 2011, Janne Blomqvist wrote: It's theoretically insoluble, given the constraints you are working under. Sorry. It is possible to do reasonably well, but there will always be likely scenarios where all you can do is to say Aargh! I give up. Well, I realize perfection is impossible,

Re: Ping: Make 128 bits the default vector size for NEON

2011-05-08 Thread Gerald Pfeifer
On Sun, 8 May 2011, Ira Rosen wrote: If this goes in, please also update gcc-4.7/changes.html. Do you mean that the new flag should be documented? Yes, as we're adding new flags, it's (nearly?) always a good idea to document them as part of the release notes. This patch

Re: [wwwdocs] Use regular h2 markup for java/projects.html

2011-05-08 Thread Gerald Pfeifer
On Sat, 30 Apr 2011, Steven Bosscher wrote: 2011-04-26  Gerald Pfeifer  ger...@pfeifer.com        * projects.html: Use regular h2 markup for section headers        instead of fake tables. The Compiler improvements section is 10 years behind on GCC's development (tree-ssa!). The recently

[PATCH] for Re: C99 and C++0x status pages

2011-05-08 Thread Gerald Pfeifer
On Tue, 31 Aug 2010, Andre Majorel wrote: Yesterday, I spent an hour looking for the C99 and C++0x status pages in http://gcc.gnu.org/, http://gcc.gnu.org/c99status.html http://gcc.gnu.org/projects/cxx0x.html Apparently, the shortest path to the latter is Releases - GCC

[PATCH] allow setting LDFLAGS_FOR_TARGET in top-level configuration.

2011-05-08 Thread 關振德
Hi, The x86 Android toolchain needs setting LDFLAGS_FOR_TARGET to build. This patch does that. The patch was tested by bootstrapping natively on x86_64 linux. Do I also need to submit this to binutils as well? -Doug

Re: [PATCH] allow setting LDFLAGS_FOR_TARGET in top-level configuration.

2011-05-08 Thread H.J. Lu
On Sun, May 8, 2011 at 7:31 PM, Doug Kwan (關振德) dougk...@google.com wrote: Hi,    The x86 Android toolchain needs setting LDFLAGS_FOR_TARGET to build.  This patch does that.  The patch was tested by bootstrapping natively on x86_64 linux.  Do I also need to submit this to binutils as well?

Re: [PATCH] allow setting LDFLAGS_FOR_TARGET in top-level configuration.

2011-05-08 Thread 關振德
Sorry, forgot the patch and the ChangeLog 2011-05-08 Doug Kwan dougk...@google.com * configure.ac: Propagate LDFLAGS_FOR_TARGET. * configure: Regenerated. * Makefile.tpl (LDFLAGS_FOR_TARGET): Use LDFLAGS_FOR_TARGET value from configure. * Makefile.in:

Re: [C++ Patch] PRs 48737 48744

2011-05-08 Thread Jason Merrill
On 05/08/2011 12:51 PM, Paolo Carlini wrote: @@ -5203,7 +5203,7 @@ reshape_init_r (tree type, reshape_iter *d, bool f { ++d-cur; gcc_assert (BRACE_ENCLOSED_INITIALIZER_P (init)); - return reshape_init (type, init); + return

Re: Ping: Make 128 bits the default vector size for NEON

2011-05-08 Thread Ira Rosen
Gerald Pfeifer ger...@pfeifer.com wrote on 09/05/2011 01:53:35 AM: On Sun, 8 May 2011, Ira Rosen wrote: If this goes in, please also update gcc-4.7/changes.html. Do you mean that the new flag should be documented? Yes, as we're adding new flags, it's (nearly?) always a good idea to