Re: libstdc++.exp patch

2011-04-26 Thread Rainer Orth
François Dumont francois.cppd...@free.fr writes: Attached patch applied. 2011-04-21 François Dumont francois.cppd...@free.fr * testsuite/lib/libstdc++.exp (check_v3_target_time): Discard unused compilation result thanks to /dev/null. * testsuite/lib/libstdc++.exp

Re: [doc] Adjust reference to ACATS in sourcebuild.texi

2011-04-26 Thread Rainer Orth
Gerald Pfeifer ger...@pfeifer.com writes: Installed on mainline; if someone wants me to, happy to apply to older branches as well. Gerald 2011-04-24 Gerald Pfeifer ger...@pfeifer.com * doc/sourcebuild.texi (Ada Tests): Adjust reference to ACATS testsuite and make it version

[debug] PR42288 emit empty .debug_aranges section

2011-04-26 Thread Mark Wielaard
Hi, This PR was put on hold for a while since gdb didn't immediately need it. But the consensus seemed that it was a good idea to always output the address ranges (.debug_arange section) information if a CU (.debug_info section) was emitted. Because the consumer has no way to tell the difference

Re: [debug] PR42288 emit empty .debug_aranges section

2011-04-26 Thread Rainer Orth
Mark Wielaard m...@redhat.com writes: This PR was put on hold for a while since gdb didn't immediately need it. But the consensus seemed that it was a good idea to always output the address ranges (.debug_arange section) information if a CU (.debug_info section) was emitted. Because the

Re: [debug] PR42288 emit empty .debug_aranges section

2011-04-26 Thread Mark Wielaard
Hi Rainer, On Tue, 2011-04-26 at 11:22 +0200, Rainer Orth wrote: Mark Wielaard m...@redhat.com writes: This PR was put on hold for a while since gdb didn't immediately need it. But the consensus seemed that it was a good idea to always output the address ranges (.debug_arange section)

Re: [PATCH][ARM] Fix -Werror build failure

2011-04-26 Thread Ramana Radhakrishnan
OK? OK. Ramana

Re: libstdc++.exp patch

2011-04-26 Thread Paolo Carlini
On 04/26/2011 10:59 AM, Rainer Orth wrote: Please separate the proc names by commas in the ChangeLog. Indeed. I took care of this. Paolo.

Re: [PATCH] Canonicalize compares in combine [2/3] Modifications to try_combine()

2011-04-26 Thread Chung-Lin Tang
On 2011/4/26 02:08 AM, Jeff Law wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/22/11 09:21, Chung-Lin Tang wrote: This patch is the main bulk of this submission. It modifies the compare combining part of try_combine(), adding a call of CANONICALIZE_COMPARISON into the entire

Re: [Patch,AVR]: Solve PR42210

2011-04-26 Thread Georg-Johann Lay
Richard Henderson schrieb: On 04/21/2011 05:31 AM, Georg-Johann Lay wrote: +;; Some combiner patterns dealing with bits. +;; See PR42210 + +;; Move bit $3.$4 into bit $0.$4 +(define_insn *movbitqi.1-6.a ... +(define_insn *movbitqi.1-6.b ... +(define_insn *movbitqi.0 ... +(define_insn

Re: [RFC] Context sensitive inline analysis

2011-04-26 Thread Jan Hubicka
Honza, This patch causes a bootstrap failure when building libstdc++ on AIX: In file included from /farm/dje/src/src/libstdc++-v3/include/precompiled/stdc++.h:94:0: /tmp/20110423/powerpc-ibm-aix5.3.0.0/libstdc++-v3/include/valarray:1163:1: internal compiler error: vector VEC(tree,base)

Re: [google] Port self-assign warning to google/main branch (issue4442075)

2011-04-26 Thread Diego Novillo
On Fri, Apr 22, 2011 at 16:08, Le-Chun Wu l...@google.com wrote: 2011-04-22  Le-Chun Wu  l...@google.com gcc/c-family/ChangeLog:        * c-common.c (check_for_self_assign): New function.        * c-common.h: New function declaration.        * c.opt: New option. gcc/ChangeLog:        *

Re: [google]Pass --save-temps to the assembler (issue4436049)

2011-04-26 Thread Diego Novillo
On Mon, Apr 25, 2011 at 15:00, Easwaran Raman era...@google.com wrote: 2011-04-25  Easwaran Raman  era...@google.com        * gcc/gcc.c (asm_options): Pass --save-temps to assembler        when invoked with -save-temps= option. OK. Diego.

Re: [PATCH] Fix PR48753

2011-04-26 Thread H.J. Lu
On Tue, Apr 26, 2011 at 3:02 AM, Richard Guenther rguent...@suse.de wrote: Value-profiling now creates proper integer constants. Installed. Richard. 2011-04-26  Richard Guenther  rguent...@suse.de        PR testsuite/48753        * gcc.dg/tree-prof/val-prof-2.c: Adjust. Index:

Re: [PATCH] Fix PR48753

2011-04-26 Thread Jakub Jelinek
On Tue, Apr 26, 2011 at 12:02:38PM +0200, Richard Guenther wrote: Value-profiling now creates proper integer constants. This still fails on 32-bit HWI hosts, like i686-linux. Fixed thusly, regtested on x86_64-linux and i686-linux, committed to trunk: 2011-04-26 Jakub Jelinek

Re: [Patch,AVR]: Solve PR42210

2011-04-26 Thread Richard Henderson
On 04/26/2011 04:51 AM, Georg-Johann Lay wrote: As the SFRs are volatile, insv expander skips them and a combine pattern must care of them. Omitting the complicated bit-plethora patterns I still see long, slow shift loops for HI. So the patch still includes them. Why don't we focus on doing

Re: [Patch,AVR]: Solve PR42210

2011-04-26 Thread Georg-Johann Lay
Richard Henderson schrieb: On 04/26/2011 04:51 AM, Georg-Johann Lay wrote: As the SFRs are volatile, insv expander skips them and a combine pattern must care of them. Omitting the complicated bit-plethora patterns I still see long, slow shift loops for HI. So the patch still includes them.

Ping^3 Re: Target header etc. cleanup patch

2011-04-26 Thread Joseph S. Myers
Ping^3. Parts of this patch http://gcc.gnu.org/ml/gcc-patches/2011-04/msg00130.html are still pending review: the parts for targets fr30, m32r, mn10300, rx, v850 (target maintainers CC:ed). This version applies cleanly to current trunk. 2011-04-26 Joseph Myers jos...@codesourcery.com

Re: Ping^3 Re: Target header etc. cleanup patch

2011-04-26 Thread Nick Clifton
Hi Joseph, 2011-04-26 Joseph Myersjos...@codesourcery.com * config/fr30/fr30-protos.h (Mmode): Don't define. * config/fr30/fr30.h (inhibit_libc): Don't define. * config/m32r/m32r-protos.h (Mmode): Don't define. Expand definition where used. *

Re: Ping^3 Re: Target header etc. cleanup patch

2011-04-26 Thread Joseph S. Myers
On Tue, 26 Apr 2011, Nick Clifton wrote: Similarly it would be nice to standard abbreviations for CUMULATIVE_ARGS, enum rtx_code, enum reg_class and const char *. What do you think ? I suspect I think defining an abbreviation for const char * is actively bad, since const char * is a

Re: [doc] Remove references to mips-tfile on MIPS

2011-04-26 Thread Rainer Orth
Joseph, Not strictly related to this patch, but there are other cleanups possible because of the only-used-on-Tru64 nature of mips-tdump/mips-tfile. In particular, there are seven target macros (all undocumented) used by those programs and nowhere else in GCC: ALIGN_SYMTABLE_OFFSET

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-26 Thread Paolo Bonzini
On 04/26/2011 05:25 PM, Rainer Orth wrote: David, Thanks Rainer, if you cons up a patch to add the .align directive I'll ACK that as well. here's the patch that I've been testing over the weekend. It caused no change for HAVE_AS_SPARC_GOTDATA_OP on anything but Solaris 10 with gas and Sun

Re: Ping^2 Re: Target header etc. cleanup patch

2011-04-26 Thread Christopher Faylor
On Thu, Apr 21, 2011 at 11:56:52PM -0400, Jie Zhang wrote: On 04/21/2011 06:06 AM, Joseph S. Myers wrote: On Thu, 21 Apr 2011, Richard Guenther wrote: On Wed, Apr 20, 2011 at 11:09 PM, Joseph S. Myers jos...@codesourcery.com wrote: Ping^2. This patch

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

2011-04-26 Thread Steve Kargl
On Mon, Apr 25, 2011 at 11:15:35PM +0300, Janne Blomqvist wrote: On Mon, Apr 25, 2011 at 22:45, Steve Kargl s...@troutmask.apl.washington.edu wrote: On Mon, Apr 25, 2011 at 10:26:20PM +0300, Janne Blomqvist wrote: Hmm, I'd prefer if the warning was issued only with -Wsomething which would

Put inliners info about cgraph edges into its own summary structure

2011-04-26 Thread Jan Hubicka
Hi, this patch moves inliner related callgraph edge info (call_stmt_size/time and loop_nest) into inliner own edge summaries, just like we have for ipa-prop. Primary motivation for doing so is that I don't want to add edge predicates to main cgraph edge and handle them in lto-cgraph, but doing

[C PATCH] Avoid nested C_MAYBE_CONST_EXPR in build_binary_op (PR c/48742)

2011-04-26 Thread Jakub Jelinek
Hi! As discussed in the PR, build_binary_op was calling c_wrap_maybe_const which added one C_MAYBE_CONST_EXPR to GT_EXPR operand, then the whole GT_EXPR has been enclosed into another C_MAYBE_CONST_EXPR because int_operands has been true. Fixed thusly, bootstrapped/regtested on x86_64-linux and

Re: [PATCH, C++] Remove unused cp_fold_obj_type_ref

2011-04-26 Thread Rainer Orth
Martin, last year I removed the last use of cp_fold_obj_type_ref (which was a language hook) but forgot to remove the function itself. So I'm about to do it now. I'm currently bootstrapping and testing the following patch. I consider it obvious enough to commit it myself on Tuesday if it

Re: [PATCH, 4.7] Add float/double vector reductions to VSX

2011-04-26 Thread Michael Meissner
On Sat, Apr 23, 2011 at 04:11:50PM -0400, David Edelsohn wrote: Except the patch includes a change to pattern vsx_xxpermdi_mode that is not explained nor mentioned in the ChangeLog. Yes, that vsx_xxpermdi_mode patch was wrong. I believe that snuck in because I had done the work on an older

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-26 Thread David Miller
From: Rainer Orth r...@cebitec.uni-bielefeld.de Date: Tue, 26 Apr 2011 17:25:56 +0200 David, Thanks Rainer, if you cons up a patch to add the .align directive I'll ACK that as well. here's the patch that I've been testing over the weekend. It caused no change for HAVE_AS_SPARC_GOTDATA_OP

C++ PATCH for c++/48530 (hard errors with deleted destructors)

2011-04-26 Thread Jason Merrill
Daniel Krugler has been doing well at ferreting out parts of the compiler that haven't been properly SFINAE'd yet... Tested x86_64-pc-linux-gnu, applied to trunk. commit 93016efbbb141f1bd917c4032b4a94b2b4c85516 Author: Jason Merrill ja...@redhat.com Date: Mon Apr 25 20:07:15 2011 -0400

Re: [C PATCH] Avoid nested C_MAYBE_CONST_EXPR in build_binary_op (PR c/48742)

2011-04-26 Thread Joseph S. Myers
On Tue, 26 Apr 2011, Jakub Jelinek wrote: 2011-04-26 Jakub Jelinek ja...@redhat.com PR c/48742 * c-typeck.c (build_binary_op): Don't wrap arguments if int_operands is true. * gcc.c-torture/compile/pr48742.c: New test. OK. -- Joseph S. Myers

New mirror in Phoenix, US (was: new mirror in Nicosia, Cyprus)

2011-04-26 Thread Gerald Pfeifer
On Mon, 25 Apr 2011, Gerald Pfeifer wrote: Grigory and me mailed off-list a bit, and I just applied the patch below (after checking the mirror now seems to be fully active). Just one question: is this server _really_ located in Cyprus? My traceroutes seem to indicate otherwise. It turns

Re: [PATCH, C++] Remove unused cp_fold_obj_type_ref

2011-04-26 Thread Jason Merrill
On 04/26/2011 02:07 PM, Rainer Orth wrote: the patch was so obvious that it broke bootstrap: I'm applying this patch to fix bootstrap. commit 922f4a593d65ab65025b0af7b59bb9513d394aac Author: jason jason@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Tue Apr 26 21:12:57 2011 + *

Re: [patch, fortran] Make block names unique

2011-04-26 Thread Mikael Morin
On Tuesday 26 April 2011 00:40:40 Thomas Koenig wrote: Am 22.04.2011 22:07, schrieb Thomas Koenig: Hello world, the attached patch makes block names unique, so that -fdump-fortran-original dumps are easier to read. Regression-tested. OK for trunk? Thomas 2011-04-22 Thomas

[Patch, Fortran] PRs 48112/48279 - fix interface related regressions

2011-04-26 Thread Tobias Burnus
This patch fixes three issues, (a) and (b) are 4.6/4.7 regressions. a) PR48112: Due to incomplete resolution, there was an ICE when writing the module file. Solution: Back-out the patch which introduced the incomplete resolution - and add a similar patch later to avoid printing errors

Re: [build, lto] Only accept -fuse-linker-plugin if linker supports -plugin (PR lto/46944)

2011-04-26 Thread Ralf Wildenhues
Hello Rainer, * Rainer Orth wrote on Tue, Apr 26, 2011 at 05:28:19PM CEST: it's been a week since I answered your questions on this patch. Could you please have a look? Sorry for the delay. I'm practically AFK until the weekend or maybe next weekend, whenever I have connectivity again after

Re: Improve DSE in the presence of calls

2011-04-26 Thread Easwaran Raman
On Mon, Apr 25, 2011 at 10:03 AM, Jeff Law l...@redhat.com wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/22/11 14:19, Easwaran Raman wrote: Hi,  This patch improves RTL DSE by not assuming that calls read all memory locations. With this patch, calls are assumed to read any

Re: [patch, fortran] Make block names unique

2011-04-26 Thread Thomas Koenig
Hi Mikael, * decl.c (gfc_match_end): Check that the block name starts with block@. * parse.c (gfc_build_block_ns): Make block names unique by numbering them. Ping ** 0.25? Thomas OK Waiting for Emacs... Sendingfortran/ChangeLog Sendingfortran/decl.c Sending

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

2011-04-26 Thread Mikael Morin
On Tuesday 26 April 2011 18:52:58 Steve Kargl wrote: On Mon, Apr 25, 2011 at 11:15:35PM +0300, Janne Blomqvist wrote: On Mon, Apr 25, 2011 at 22:45, Steve Kargl s...@troutmask.apl.washington.edu wrote: On Mon, Apr 25, 2011 at 10:26:20PM +0300, Janne Blomqvist wrote: Hmm, I'd prefer

[PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Kaz Kojima
Hi, The attached target specific patch is to fix PR target/48767. In the problematic case, sh.c:sh_gimplify_va_arg_expr calls targetm.calls.must_pass_in_stack with void type as its 2nd argument which is unexpected by the callee. The patch is tested on sh4-unknown-linux-gnu with no new failures.

Re: Don't use linux.h for non-Linux targets

2011-04-26 Thread Thomas Schwinge
Hallo! First, Joseph, thanks for working on this clean-up! I'm sorry for the delay, but I have now reviewed all these patches (including those you have committed by now); looks all good. On Tue, 12 Apr 2011 18:37:50 + (UTC), Joseph S. Myers jos...@codesourcery.com wrote: This patch stops

Re: [PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Joseph S. Myers
I think you should add a testcase to gcc.c-torture/compile, unless there is already one that this patch fixes. -- Joseph S. Myers jos...@codesourcery.com

PATCH: PR rtl-optimization/48575: RTL vector patterns are limited to 26 elements

2011-04-26 Thread H.J. Lu
On Mon, Apr 4, 2011 at 6:05 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Mar 31, 2011 at 5:05 AM, Kenneth Zadeck zad...@naturalbridge.com wrote: we hit this limit trying to write the explicit semantics for a vec_interleave_evenv32qi. ;;(define_insn vec_interleave_evenv32qi ;;  [(set

[patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Cary Coutant
This patch adds a new option, -gmlt, that produces level 1 debug info plus line number tables and inlined subroutine information. (The option is short for minimum line tables, taken from a similar feature of HP's compilers.) We've been using this option at Google for about a year now, and have

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Andrew Pinski
On Tue, Apr 26, 2011 at 3:44 PM, Cary Coutant ccout...@google.com wrote: This patch adds a new option, -gmlt, that produces level 1 debug info plus line number tables and inlined subroutine information. (The option is short for minimum line tables, taken from a similar feature of HP's

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Cary Coutant
This patch adds a new option, -gmlt, that produces level 1 debug info plus line number tables and inlined subroutine information. (The option is short for minimum line tables, taken from a similar feature of HP's compilers.) What is the difference between -gmlt and -g1?  And why can't this

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Andrew Pinski
On Tue, Apr 26, 2011 at 3:57 PM, Cary Coutant ccout...@google.com wrote: If we do go with extending -g1, though. we want to be able to turn it on in our build scripts, yet allow a later -g option in the user's compiler options to enable full debug info. In this patch, I've made sure that -gmlt

Re: [PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Kaz Kojima
Joseph S. Myers jos...@codesourcery.com wrote: I think you should add a testcase to gcc.c-torture/compile, unless there is already one that this patch fixes. Ah, indeed. How about the attached testcase? BTW, is it valid C? Regards, kaz -- ---

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

2011-04-26 Thread Steve Kargl
On Wed, Apr 27, 2011 at 12:23:03AM +0200, Mikael Morin wrote: On Tuesday 26 April 2011 18:52:58 Steve Kargl wrote: On Mon, Apr 25, 2011 at 11:15:35PM +0300, Janne Blomqvist wrote: On Mon, Apr 25, 2011 at 22:45, Steve Kargl s...@troutmask.apl.washington.edu wrote: On Mon, Apr 25,

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Cary Coutant
Well I think -g1 -g should act like how -O2 -O works.  That is should be -g2. I think so, too, but I guess I was too timid to assume such a change might be acceptable. If the maintainers would go along with that, I'd have no problem revising the patch to make -g1 do line tables and inline info.

[pph] Mark C++ codes with proper TS_* type markers (issue4436059)

2011-04-26 Thread Diego Novillo
We were failing to save several fields in C++ specific tree codes because they were not properly marked with TS_* tree structure tags. For instance, we were not streaming EXPR_STMTs because they were not recognized as TS_EXPs in the streamer, so its arguments were never saved. Tested on x86_64.

Re: C++ PATCH for c++/48281 (ICE with nested initializer_list)

2011-04-26 Thread Jason Merrill
On 03/30/2011 02:05 PM, Jason Merrill wrote: First I noticed that the crash was due to non-constant CONSTRUCTORs with TREE_CONSTANT set. So I fixed that, in constructor-const.patch. It turns out that we need to apply this patch to 4.6 as well, for bug 48726. So I'm doing that. Jason

[pph] Fix many segmentation faults when restoring C++ ASTs (issue4441070)

2011-04-26 Thread Diego Novillo
I have been testing the C++ streamer with a collection of 6,576 C test cases taken from the GCC testsuite. These C programs build fine with g++, so I encapsulated them inside a single header file and created a .cc file that simply #includes it. The intent is to make sure we can PPH generate and

C++ PATCH to debug_tree for TYPENAME_TYPE and IDENTIFIER

2011-04-26 Thread Jason Merrill
A couple of small changes. It was inconvenient that debug_tree didn't print the name of a TYPENAME_TYPE, and once it did I noticed that the indentation on the IDENTIFIER bits was wrong. Tested x86_64-pc-linux-gnu, applied to trunk. commit 3d660910ee64db8a09091482370ccc1162b781ca Author: Jason

Re: [patch, libgfortran] Fix numerous formatting bugs

2011-04-26 Thread Janne Blomqvist
On Wed, Apr 27, 2011 at 07:09, Jerry DeLisle jvdeli...@frontier.com wrote: On 04/25/2011 07:36 AM, Janne Blomqvist wrote: On Mon, Apr 25, 2011 at 14:44, Jerry DeLislejvdeli...@frontier.com  wrote: On 04/25/2011 03:48 AM, Janne Blomqvist wrote: Now, for one of the testcase changes: ---