Re: PATCH: Properly set ix86_gen_leave and ix86_gen_monitor

2012-03-12 Thread Uros Bizjak
On Mon, Mar 12, 2012 at 12:02 AM, H.J. Lu hongjiu...@intel.com wrote: leave_rex64 works on DImode and sse3_monitor64 works on Pmode.  This patch properly sets ix86_gen_leave and ix86_gen_monitor, depending on TARGET_64BIT and Pmode.  Tested on Linux/x86-64.  OK for trunk? 2012-03-11  H.J. Lu

Re: PATCH: Call gen_tls_global_dynamic_64_mode and en_tls_local_dynamic_base_64_mode

2012-03-12 Thread Uros Bizjak
On Mon, Mar 12, 2012 at 12:26 AM, H.J. Lu hongjiu...@intel.com wrote: Pmode may be DImode for x32.  This patch calls gen_tls_global_dynamic_64_mode and gen_tls_local_dynamic_base_64_mode, depending on Pmode.  Tested on Linux/x86-64.  OK for trunk? 2012-03-11  H.J. Lu  hongjiu...@intel.com  

Re: [Patch, Fortran] PR 52542 - Fix PROCEDURE() with Bind(C)

2012-03-12 Thread Paul Richard Thomas
Dear Tobias, Apart from s/Contribute/Contributed/ this is OK for trunk. In fact, I would say that it is obvious. Thanks for the patch. Paul On Sat, Mar 10, 2012 at 4:53 PM, Tobias Burnus bur...@net-b.de wrote: Tobias Burnus wrote: If the interface in a PROCEDURE() statement is Bind(C),

Re: [VMS/committed]: Do not define __size_t on VMS

2012-03-12 Thread Tristan Gingold
On Mar 9, 2012, at 6:22 PM, Joseph S. Myers wrote: On Fri, 9 Mar 2012, Tristan Gingold wrote: as it is already defined and used by the system headers. No regressions for x86_64-darwin after full bootstrap. Tristan. 2012-03-09 Tristan Gingold ging...@adacore.com *

Re: [PATCH] Move strip_float_extensions to tree.c

2012-03-12 Thread Richard Guenther
On Wed, 7 Mar 2012, Joseph S. Myers wrote: On Wed, 7 Mar 2012, Richard Guenther wrote: Now, convert.c is used from all frontends to implement convert () (that looks backwards - the language convert should be a langhook, called from convert implemented in convert.c). But well, I aint

Re: [PATCH] Fix PRs 52080, 52097 and 48124, rewrite bitfield expansion, enable the C++ memory model wrt bitfields everywhere

2012-03-12 Thread Richard Guenther
On Fri, 9 Mar 2012, Eric Botcazou wrote: This patch also completely replaces get_bit_range (which is where PR52097 ICEs) by a trivial implementation. How does it short-circuit the decision made by get_best_mode exactly? By making get_bit_range return non-zero in more cases? It will

Re: [PATCH] Fix up register_edge_assert_for_2 (PR tree-optimization/52533)

2012-03-12 Thread Richard Guenther
On Fri, 9 Mar 2012, Jakub Jelinek wrote: Hi! My recent commit to tree-vrp.c on the trunk caused the following testcase to fail, the problem is that we would insert = 255 assertion for unsigned char expression (which doesn't say anything) and VRP insist that such ASSERT_EXPRs aren't added.

Re: [committed] Fix failure of gcc.dg/torture/pr52407.c on hppa*-*-hpux*

2012-03-12 Thread Richard Guenther
On Sat, 10 Mar 2012, John David Anglin wrote: Work around limited alignment of common. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to trunk. Ok for 4.7? Ok. Thanks, Richard.

[Patch]: Add include hacks for VMS headers

2012-03-12 Thread Tristan Gingold
Hi, this patch adds a few fix include entries for VMS so that: * it is possible to use a gcc for VMS to compile gcc (collisions between system and gcc headers), * no warnings when signal.h is included, * no warnings when resolv.h is included, * avoid the use of DEC-C built-ins in some standard

Re: [PATCH, i386]: Avoid partial reg stall with arith insn + setCC + movzbl sequence

2012-03-12 Thread Paolo Bonzini
Il 12/03/2012 09:52, Uros Bizjak ha scritto: +(define_peephole2 + [(parallel [(set (reg FLAGS_REG) (match_operand 0 )) + (match_operand 4 )]) + (set (match_operand:QI 1 register_operand ) + (match_operator:QI 2 ix86_comparison_operator + [(reg FLAGS_REG) (const_int

Re: [PATCH] Fix PRs 52080, 52097 and 48124, rewrite bitfield expansion, enable the C++ memory model wrt bitfields everywhere

2012-03-12 Thread Eric Botcazou
Btw, I _think_ I want GET_MODE_BITSIZE here - we cannot allow GET_MODE_BITSIZE GET_MODE_PRECISION as that would possibly access memory that is not allowed. Thus, what GET_MODE_* would identify the access size used for a MEM of that mode? I agree that GET_MODE_BITSIZE makes more sense than

Re: [PATCH] Do not handle SUBREG in apply_distributive_law (Re: RFC: allowing fwprop to propagate subregs)

2012-03-12 Thread Richard Guenther
On Wed, Mar 7, 2012 at 6:40 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Richard Kenner wrote: Given the current set of results, since I do not have any way to verify whether my simplify_set changes would actually trigger correctly, I'd rather propose to just remove the SUBREG case in

[PATCH][ARM] Do not call covert during RTL expansion

2012-03-12 Thread Richard Guenther
convert is supposed to be called only from frontend code as it may raise errors. The call in neon_dereference_pointer is from RTL expansion (and not necessary). Fixed as follows, I built a cross-cc1 for checking. Ok? Thanks, Richard. 2012-03-12 Richard Guenther rguent...@suse.de *

[PATCH] Fix some SRA and GIMPLE verification issues

2012-03-12 Thread Richard Guenther
This fixes issues I noticed when working on the bitfield expansion issues. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-03-05 Richard Guenther rguent...@suse.de * tree-sra.c (create_access_replacement): Only rename the replacement if

Re: [PATCH] Improve SCEV for array element

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 8:13 AM, Jiangning Liu jiangning@arm.com wrote: -Original Message- From: Richard Guenther [mailto:richard.guent...@gmail.com] Sent: Tuesday, March 06, 2012 9:12 PM To: Jiangning Liu Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Improve SCEV for array

Re: [PATCH, i386]: Avoid partial reg stall with arith insn + setCC + movzbl sequence

2012-03-12 Thread Uros Bizjak
On Mon, Mar 12, 2012 at 11:13 AM, Paolo Bonzini bonz...@gnu.org wrote: Il 12/03/2012 09:52, Uros Bizjak ha scritto: +(define_peephole2 +  [(parallel [(set (reg FLAGS_REG) (match_operand 0 )) +           (match_operand 4 )]) +   (set (match_operand:QI 1 register_operand ) +    

Re: [PATCH] Proper use of decl_function_context in dwar2out.c

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 12:18 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Mar 08, 2012 at 12:06:46PM +0100, Martin Jambor wrote:        /* For local statics lookup proper context die.  */ -      if (TREE_STATIC (decl) decl_function_context (decl)) -     context_die = lookup_decl_die

Re: [WIP PATCH] Re: Inefficient end-of-loop value computation - missed optimization somewhere?

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 3:29 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Richard Guenther wrote: On Tue, Feb 28, 2012 at 4:10 PM, Ulrich Weigand uweig...@de.ibm.com wrote: I'll still need to do proper testing and benchmarking, but I thought I'd post the patch anyway just as a heads-up ...

Re: [ping] Vectorizer patches for 4.8

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 3:56 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Hello, Ira Rosen posted a couple of vectorizer patches intended for 4.8:   http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00191.html   http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00223.html As she is no longer able to

Re: [PATCH] Further VRP improvements

2012-03-12 Thread Richard Guenther
On Thu, 8 Mar 2012, Jakub Jelinek wrote: Hi! This patch adds ASSERT_EXPRs for if ((int) var cmp CST) (for cmp , =, , =). As the testcase shows, various different tests are folded into that form. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? Ok. Thanks, Richard.

Re: Support for Runtime CPU type detection via builtins (issue5754058)

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 9:35 PM, Xinliang David Li davi...@google.com wrote: On Wed, Mar 7, 2012 at 5:51 AM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Mar 7, 2012 at 1:49 AM, Sriraman Tallam tmsri...@google.com wrote: Patch for CPU detection at run-time.

Re: [PATCH] Further VRP improvements (take 2)

2012-03-12 Thread Richard Guenther
On Fri, 9 Mar 2012, Jakub Jelinek wrote: On Thu, Mar 08, 2012 at 04:54:31PM +0100, Jakub Jelinek wrote: This patch adds ASSERT_EXPRs for if ((int) var cmp CST) (for cmp , =, , =). As the testcase shows, various different tests are folded into that form. Bootstrapped/regtested on

[PATCH] : Fix negative value in TEST_BIT from mem_overlaps_already_clobbered_arg_p()

2012-03-12 Thread mickael guene
Hi all, After applying patch from http://gcc.gnu.org/ml/gcc-patches/2012-02/msg00228.html we can have a negative value in call to TEST_BIT() macro in mem_overlaps_already_clobbered_arg_p() function for stored_args_map bitmap table. This can lead to reject a valid tail call optimization.

[VMS/committed]: Add -mpointer-size switch

2012-03-12 Thread Tristan Gingold
Hi, Now that '#pragma pointer_size' is supported by GCC for VMS, we can add the '-mpointer-size' switch (modeled on /POINTER_SIZE=xx DEC-C qualifier) to set the default pointer size. This also makes alpha64-dec-*vms* target obsolete (but still supported by alpha*-dec-*vms*). The patch has

Re: [patch, RFA] delete obsolete -madjust-unroll option for SH

2012-03-12 Thread Richard Guenther
On Sat, Mar 10, 2012 at 5:25 AM, Sandra Loosemore san...@codesourcery.com wrote: When I was working on cleanups to invoke.texi I noticed this bit in the SH Options section: @item -madjust-unroll @opindex madjust-unroll Throttle unrolling to avoid thrashing target registers. This option only

Re: [PATCH] Improve PHI-OPT when there are multiple phis

2012-03-12 Thread Richard Guenther
On Sat, Mar 10, 2012 at 8:55 AM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: Woops I forgot the patch. Ok. Thanks, Richard. Thanks, Andrew Pinski On Fri, Mar 9, 2012 at 11:45 AM, Andrew Pinski andrew.pin...@caviumnetworks.com wrote: On Tue, Jan 24, 2012 at 2:50 AM, Richard

Re: [PR51752] publication safety violations in loop invariant motion pass

2012-03-12 Thread Richard Guenther
On Fri, Mar 9, 2012 at 10:48 PM, Aldy Hernandez al...@redhat.com wrote: Note that partial PRE (enabled at -O3) can insert expressions into paths that did _not_ execute the expression.  For regular PRE you are right. Richard. I've thought about this some more, and Torvald's comment makes a

Re: Many regressions with: [patch] Cleanup fortran/convert.c

2012-03-12 Thread Richard Guenther
On Sat, Mar 10, 2012 at 12:21 PM, Steven Bosscher stevenb@gmail.com wrote: On Sat, Mar 10, 2012 at 11:19 AM, Tobias Burnus bur...@net-b.de wrote: Steven Bosscher wrote: This cleans up some remnants of the ancestors of fortran's convert.c, which was copied from GNAT IIRC. I would

Re: [committed] Skip gcc.dg/torture/pr52402.c execution on 32-bit hppa*-*-hpux*

2012-03-12 Thread Richard Guenther
On Sat, Mar 10, 2012 at 11:53 PM, John David Anglin d...@hiauly1.hia.nrc.ca wrote: Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11.  Committed to trunk. Ok for 4.7? Ok. Thanks, Richard. Dave -- J. David Anglin                                  dave.ang...@nrc-cnrc.gc.ca National

Re: [C++ Patch] for c++/52465

2012-03-12 Thread Dodji Seketeli
Hello Fabien, Fabien Chêne fabien.ch...@gmail.com a écrit: [...] Index: gcc/cp/decl.c === --- gcc/cp/decl.c (revision 184891) +++ gcc/cp/decl.c (working copy) @@ -8686,6 +8686,9 @@ grokdeclarator (const cp_declarator

Re: [PATCH] Don't insert white space in 'orig_option_with_args_text' for OPT_l

2012-03-12 Thread Joseph S. Myers
On Mon, 12 Mar 2012, Ludovic Court?s wrote: The patch below solves the problem in a gfortran-specific way. WDYT? I think that's the right approach for this issue. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH] Do not use lang_hooks.types.type_for_size in signed_or_unsigned_type_for

2012-03-12 Thread Richard Guenther
On Wed, 7 Mar 2012, Richard Guenther wrote: On Wed, 7 Mar 2012, Michael Matz wrote: Hi, On Wed, 7 Mar 2012, Richard Guenther wrote: FAIL: gcc.dg/tree-ssa/pr31261.c scan-tree-dump-times original return \\(char\\) -\\(unsigned char\\) c 31; 1 FAIL:

[Fortran-dev, patch, committed] Minor fixes

2012-03-12 Thread Tobias Burnus
I have committed a fix for one link issue (spurious GFC_DESCRIPTOR_STRIDE_BYTES which escaped the renaming to GFC_DESCRIPTOR_SM) - and changed data to base_addr for consistency. Rev. 185216. Additionally, I modified some tree-dump patterns for the extent and for the base_addr change.

Re: [ARM] atomics for rtems, aka bare metal

2012-03-12 Thread Sebastian Huber
Hello Richard, thank you very much for your implementation. On 03/09/2012 02:56 AM, Richard Henderson wrote: On 03/08/12 10:02, Joel Sherrill wrote: If so, it is just a matter of ifdef's to get the right code. Ok. Does arm-eabi have this support? We probably could just use the same code.

Doc: Merge Alpha/VMS and IA-64/VMS options

2012-03-12 Thread Tristan Gingold
Hi, this patch merges Alpha/VMS and IA-64/VMS options as there is only one .opt for them, so there is no need to duplicate the doc. It also documents -mpointer-size. Tested by 'make info' Ok for trunk ? Tristan. 2012-03-12 Tristan Gingold ging...@adacore.com * doc/invoke.texi

Re: [Fortran-dev, patch, committed] Minor fixes

2012-03-12 Thread Paul Richard Thomas
Dear Tobias, At some point, the extent calculation should be updated. Dumps like the following hurt, even if -O1 handles* them: (((D.1871-dim[0].lower_bound + D.1871-dim[0].extent) + -1) - D.1871-dim[0].lower_bound) + 1. [* maybe -fstrict-overflow and/or -fno-protect-parens is required in

[v3] libstdc++/52562

2012-03-12 Thread Paolo Carlini
Hi, barring unexpected issues in mainline will go in 4.7.1 too. Tested x86_64-linux. Thanks, Paolo. /// 2012-03-12 Paolo Carlini paolo.carl...@oracle.com PR libstdc++/52562 * libsupc++/typeinfo (type_info::name, before, operator==,

Re: [patch, RFA] delete obsolete -madjust-unroll option for SH

2012-03-12 Thread Kaz Kojima
Hi, Richard Guenther richard.guent...@gmail.com wrote: On Sat, Mar 10, 2012 at 5:25 AM, Sandra Loosemore san...@codesourcery.com wrote: When I was working on cleanups to invoke.texi I noticed this bit in the SH Options section: @item -madjust-unroll @opindex madjust-unroll Throttle

Re: [Patch]: Add include hacks for VMS headers

2012-03-12 Thread Bruce Korb
On Mon, Mar 12, 2012 at 3:05 AM, Tristan Gingold ging...@adacore.com wrote: Hi, this patch adds a few fix include entries for VMS so that: [things work] Looks good to me.

Re: [PATCH 07/10] addr32: Use word_mode instead of Pmode in loop expand

2012-03-12 Thread Jan Hubicka
On Sun, Mar 11, 2012 at 2:06 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Mar 8, 2012 at 3:22 AM, Uros Bizjak ubiz...@gmail.com wrote: On Fri, Mar 2, 2012 at 10:02 PM, H.J. Lu hongjiu...@intel.com wrote: This patches uses word_mode instead of Pmode in loop expand since word_mode may

[PATCH] More type_for_size call removals

2012-03-12 Thread Richard Guenther
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied as obvious. Richard. 2012-03-12 Richard Guenther rguent...@suse.de lto/ * lto-lang.c (builtin_type_for_size): Use lto_type_for_size. fortran/ * f95-lang.c (builtin_type_for_size): Use

[PATCH 0/3] Fixing expansion of misaligned MEM_REFs on strict-alignment targets

2012-03-12 Thread Martin Jambor
Hi, this is another iteration of my effort to fix expansion of misaligned memory accesses on strict-alignment platforms (which was suggested by Richi in http://gcc.gnu.org/ml/gcc-patches/2011-08/msg00931.html, my previous attempt was posted as

[PATCH 1/3] Misaligned top level MEM_REFs on LHS of assignments

2012-03-12 Thread Martin Jambor
Hi, this patch has not changed since I posted it the last time (http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01375.html) except for a fixed formatting of a comment. It has been already approved by Richi (http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01389.html) and I intend to commit it this week

[PATCH 2/3] Move MEM_REF expansion to a new function

2012-03-12 Thread Martin Jambor
Hi, when we expand a misaligned MEM_REF on the LHS, we must not call the code in expand_expr_real_1 if the subsequent patch is applied, because the code generates code extracting the contents of the memory to a register, which is of course bad if the intent is to write into that memory.

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-12 Thread Douglas Rupp
On 3/12/2012 4:48 AM, Tristan Gingold wrote: Hi, Now that '#pragma pointer_size' is supported by GCC for VMS, we can add the '-mpointer-size' switch (modeled on /POINTER_SIZE=xx DEC-C qualifier) to set the default pointer size. This also makes alpha64-dec-*vms* target obsolete (but still

Re: [VMS/committed]: Add -mpointer-size switch

2012-03-12 Thread Tristan Gingold
On Mar 12, 2012, at 3:14 PM, Douglas Rupp wrote: On 3/12/2012 4:48 AM, Tristan Gingold wrote: Hi, Now that '#pragma pointer_size' is supported by GCC for VMS, we can add the '-mpointer-size' switch (modeled on /POINTER_SIZE=xx DEC-C qualifier) to set the default pointer size. This also

Re: [Patch]: Add include hacks for VMS headers

2012-03-12 Thread Tristan Gingold
On Mar 12, 2012, at 3:00 PM, Bruce Korb wrote: On Mon, Mar 12, 2012 at 3:05 AM, Tristan Gingold ging...@adacore.com wrote: Hi, this patch adds a few fix include entries for VMS so that: [things work] Looks good to me. Thank you, now committed. Tristan.

Re: [PATCH] Fix PR49484, gthr requirements update (target maintainers have a looksee)

2012-03-12 Thread Richard Guenther
On Mon, Mar 5, 2012 at 1:17 PM, Richard Guenther richard.guent...@gmail.com wrote: On Wed, Jan 18, 2012 at 3:21 PM, Richard Guenther rguent...@suse.de wrote: This fixes PR49484 by protecting __gcov_flush against concurrent execution.  To be able to use the gthread facility I have to introduce

Re: Doc: Merge Alpha/VMS and IA-64/VMS options

2012-03-12 Thread Tristan Gingold
On Mar 12, 2012, at 2:07 PM, Richard Guenther wrote: On Mon, Mar 12, 2012 at 2:00 PM, Tristan Gingold ging...@adacore.com wrote: Hi, this patch merges Alpha/VMS and IA-64/VMS options as there is only one .opt for them, so there is no need to duplicate the doc. It also documents

Re: [PATCH 2/3] Move MEM_REF expansion to a new function

2012-03-12 Thread Richard Guenther
On Mon, 12 Mar 2012, Martin Jambor wrote: Hi, when we expand a misaligned MEM_REF on the LHS, we must not call the code in expand_expr_real_1 if the subsequent patch is applied, because the code generates code extracting the contents of the memory to a register, which is of course bad if

[doc] GCC 4.7 IRIX, Tru64 UNIX updates to install.texi

2012-03-12 Thread Rainer Orth
Reviewing install.texi for the 4.7 release, I found that it only needs trivial updates for Tru64 UNIX and IRIX. Tested with make doc/gccinstall.info doc/gccinstall.pdf. Ok for 4.7 branch? I won't apply the patch to mainline since I'm about to remove both ports there. Thanks. Rainer

Re: [doc] GCC 4.7 IRIX, Tru64 UNIX updates to install.texi

2012-03-12 Thread Richard Guenther
On Mon, 12 Mar 2012, Rainer Orth wrote: Reviewing install.texi for the 4.7 release, I found that it only needs trivial updates for Tru64 UNIX and IRIX. Tested with make doc/gccinstall.info doc/gccinstall.pdf. Ok for 4.7 branch? I won't apply the patch to mainline since I'm about to

[doc] GCC 4.7 Solaris updates to install.texi

2012-03-12 Thread Rainer Orth
The following patch updates install.texi for Solaris, and applies equally to the 4.7 branch and mainline. Tested with make doc/gccinstall.info doc/gccinstall.pdf, ok for mainline and 4.7 branch? Rainer 2012-03-11 Rainer Orth r...@cebitec.uni-bielefeld.de * doc/install.texi

Re: [PATCH 2/3] Move MEM_REF expansion to a new function

2012-03-12 Thread Michael Matz
Hi, On Mon, 12 Mar 2012, Martin Jambor wrote: when we expand a misaligned MEM_REF on the LHS, we must not call the code in expand_expr_real_1 if the subsequent patch is applied, because the code generates code extracting the contents of the memory to a register, which is of course bad if the

Re: [doc] GCC 4.7 Solaris updates to install.texi

2012-03-12 Thread Richard Guenther
On Mon, 12 Mar 2012, Rainer Orth wrote: The following patch updates install.texi for Solaris, and applies equally to the 4.7 branch and mainline. Tested with make doc/gccinstall.info doc/gccinstall.pdf, ok for mainline and 4.7 branch? Ok. Thanks, Richard. Rainer 2012-03-11

Re: [PATCH][1/n] No longer sign-extend sizetype constants, remove TYPE_IS_SIZETYPE

2012-03-12 Thread Richard Guenther
On Fri, 9 Mar 2012, Eric Botcazou wrote: Well. I suppose fixing that negative DECL_FIELD_OFFSET thing should be #1 priority. Tentative patch attached (you need an up-to-date tree). We cannot really get rid of the negative offsets for thin pointers, so the patch pulls them out of

[VMS/committed]: Fix ABI issue on vms/ia64

2012-03-12 Thread Tristan Gingold
Hi, ia64/VMS doesn't closely follow the 'standard' ia64 ABI (in order to be backward compatible with VAX). We recently found one mismatch while passing FP parameters. Manually tested and checked - no test case added as this require another compiler to test. Committed on trunk. Tristan.

Re: Remove obsolete OpenBSD/MIPS support

2012-03-12 Thread Rainer Orth
Richard Sandiford rdsandif...@googlemail.com writes: Rainer Orth r...@cebitec.uni-bielefeld.de writes: I'm currently working on removing the obsolete Tru64 UNIX and IRIX ports. When IRIX is gone, the obsoleted OpenBSD/MIPS is the only remaining port that uses MIPS_DEBUGGING_INFO (which I

Re: PATCH: Properly set ix86_gen_leave and ix86_gen_monitor

2012-03-12 Thread H.J. Lu
On Mon, Mar 12, 2012 at 12:40 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Mar 12, 2012 at 12:02 AM, H.J. Lu hongjiu...@intel.com wrote: leave_rex64 works on DImode and sse3_monitor64 works on Pmode.  This patch properly sets ix86_gen_leave and ix86_gen_monitor, depending on TARGET_64BIT

[build] Remove extra_passes

2012-03-12 Thread Rainer Orth
As suggested, this patch removes the now unused extra_passes mechanism. Bootstrapped with no regressions on i386-pc-solaris2.11, ok for mainline? Rainer 2012-03-09 Rainer Orth r...@cebitec.uni-bielefeld.de * config.gcc (extra_passes): Remove. * configure.ac

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-12 Thread Paolo Carlini
On 03/12/2012 05:06 PM, Rainer Orth wrote: I think the remaining changes are either obvious or covered by osf or testsuite maintainerships, so I've applied the patch. Thanks. Rainer Thanks, but please remove all the spurious 'libstdc++v3' from the ChangeLog entry and make sure it's wrapped to 80

Re: [C++ Patch] for c++/52465

2012-03-12 Thread Fabien Chêne
Salut Dodji, 2012/3/12 Dodji Seketeli do...@seketeli.org: [...] Index: gcc/cp/decl.c === --- gcc/cp/decl.c     (revision 184891) +++ gcc/cp/decl.c     (working copy) @@ -8686,6 +8686,9 @@ grokdeclarator (const cp_declarator *dec

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-12 Thread Rainer Orth
Paolo Carlini paolo.carl...@oracle.com writes: On 03/12/2012 05:06 PM, Rainer Orth wrote: I think the remaining changes are either obvious or covered by osf or testsuite maintainerships, so I've applied the patch. Thanks. Rainer Thanks, but please remove all the spurious 'libstdc++v3' from

Re: Remove non-GAS non-ELF support in alpha backend

2012-03-12 Thread Richard Henderson
On 03/12/12 09:22, Rainer Orth wrote: There are only two issues: * In alpha.c (alpha_option_override), it's unclear if the optimize 0 test can be removed completely. Yeah, I guess leave that for now. * During testing, I ran into this error: alpha.md:5861:

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-12 Thread Rainer Orth
David Daney david.da...@cavium.com writes: On 03/06/2012 05:14 AM, Rainer Orth wrote: Joseph S. Myersjos...@codesourcery.com writes: There's one particular issue: the change to java/io/File.java required my to regenerate the .class file in classpath. I've used Sun javac -target

Re: [PR51752] publication safety violations in loop invariant motion pass

2012-03-12 Thread Aldy Hernandez
On 03/10/12 08:14, Torvald Riegel wrote: On Fri, 2012-03-09 at 15:48 -0600, Aldy Hernandez wrote: Torvald is this what you were thinking of? Yes, but with an exit in the else branch or something that can cause x not being read after the condition. I _suppose_ that your original example would

Re: PATCH: Call gen_tls_global_dynamic_64_mode and en_tls_local_dynamic_base_64_mode

2012-03-12 Thread H.J. Lu
On Mon, Mar 12, 2012 at 12:46 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Mar 12, 2012 at 12:26 AM, H.J. Lu hongjiu...@intel.com wrote: Pmode may be DImode for x32.  This patch calls gen_tls_global_dynamic_64_mode and gen_tls_local_dynamic_base_64_mode, depending on Pmode.  Tested on

Re: Remove non-GAS non-ELF support in alpha backend

2012-03-12 Thread Joseph S. Myers
On Mon, 12 Mar 2012, Rainer Orth wrote: While having the osf removal patch reviewed, it turned out that with the last non-gas non-ELF alpha port gone, there's considerable cleanup potential in the alpha backend. The following patch implements this: I wonder whether it will be possible for a

Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-12 Thread Rainer Orth
With the IRIX port gone (shortly), there are no targets that define a non-default DWARF_OFFSET_SIZE or DWARF_INITIAL_LENGTH_SIZE any longer. I therefore propose to remove the code handling this. The following (pretty mechanical) patch does this. Bootstrapped without regressions on

[Patch,AVR]: Fix PR52488 (insane stack usage)

2012-03-12 Thread Georg-Johann Lay
...and now to the right mailing list... This patch cuts down insane stack offsets for 8-bit SP targets to a value the involved insns can handle. Background is that newlib generates such insane offsets (2050 bytes of stack for a device with 128 bytes of RAM) because of extension PR51345, i.e. new

Fix sgi typos

2012-03-12 Thread Rainer Orth
While working on the IRIX 6.5 removal patch, I came across a couple of typos. Fixed as follows. Will install as obvious, I only wonder what to do about the typo in ACATS? Rainer 2012-03-10 Rainer Orth r...@cebitec.uni-bielefeld.de libgcc: * config/arm/crtn.S: Fix

Re: Remove obsolete IRIX 6.5 support

2012-03-12 Thread Bruce Korb
On Mon, Mar 12, 2012 at 10:03 AM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: I'm happy with the stuff going away! :)

[RFA/ARM]: Correct Neon testsuite generation

2012-03-12 Thread Matthew Gretton-Dann
All, The commit to fix PR51534 did not update the testsuite (as no changes were expected there). Unfortunately, this means that I didn't notice that the Neon testsuite generator is broken. The attached patch fixes the generator. Checked by re-running the Neon testsuite and arm_neon.h

RFC: PATCH: Add -maddress-mode=short|long for x86

2012-03-12 Thread H.J. Lu
Hi, Trunk can now use SImode or DImode for Pmode with -mx32. Pmode == SImode will avoid many conversions from SImode to DImode: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50797 It will use 0x67 address size prefix instead. This patch adds a new command line option, -maddress-mode=short|long,

Re: [Fortran-dev, patch, committed] Minor fixes

2012-03-12 Thread Tobias Burnus
On 03/12/2012 01:50 PM, Tobias Burnus wrote: Additionally, I modified some tree-dump patterns for the extent and for the base_addr change. Three more changes: The libgfortran change should have been just a cleanup, but it seems to also fix a failure. Plus two dump changes. The number of

[Patch,AVR.testsuite,committed]: More test for address spaces (PR49868)

2012-03-12 Thread Georg-Johann Lay
http://gcc.gnu.org/viewcvs?view=revisionrevision=185255 This adds some test cases for the address space support. Some tests fail because of PR other/52545. Moreover, you will have to update the avrtest simulator because the test case for __flash1 needs its own linker script in order to locate

Re: [Patch ARM/ configury] Add fall-back check for gnu_unique_object

2012-03-12 Thread DJ Delorie
Looks OK to me.

[PATCH][Cilkplus] Elemental Function Mangling

2012-03-12 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch mainly affecting the C compiler. This patch will do the vector function mangling correctly for elemental functions. Thanking You, Yours Sincerely, Balaji V. Iyer.diff --git a/gcc/ChangeLog.cilk b/gcc/ChangeLog.cilk index

Re: Swap SECTION_EXCLUDE and SECTION_MACH_DEP

2012-03-12 Thread Sriraman Tallam
Already submitted by gjl in rev. 185259. -Sri. On Mon, Mar 12, 2012 at 10:27 AM, Richard Henderson r...@redhat.com wrote: On 03/12/12 10:26, Sriraman Tallam wrote: Hi Richard,   Is this ok for trunk? Asking you because you approved the patch the first time around. Thanks, -Sri. On

[PATCH,testsuite,committed] Skip some tests for powerpc*-*-darwin*

2012-03-12 Thread William J. Schmidt
Following in response to a report on PR46728. Committed as obvious. 2012-03-12 Bill Schmidt wschm...@linux.vnet.ibm.com PR tree-optimization/46728 * gcc.target/powerpc/pr46728-4.c: Skip for powerpc*-*-darwin*. * gcc.target/powerpc/pr46728-5.c: Likewise. *

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Ian Lance Taylor
Rainer Orth r...@cebitec.uni-bielefeld.de writes: libgo: * configure.ac (OSCFLAGS): Remove *-*-solaris2.8 handling. (libgo_cv_lib_makecontext_stack_top): Remove sparc*-*-solaris2.8* handling. * configure: Regenerate. As with the Irix 5 changes, just send the

[C++ Patch] PR 52299, bogus div by zero warning

2012-03-12 Thread Paolo Carlini
Hi, I handled this issue as outlined by Jakub in the audit trail. For the purpose of the bogus div by zero warning just using c_inhibit_evaluation_warnings appears to work fine. Tested x86_64-linux. Thanks, Paolo. /// /cp 2012-03-12 Paolo Carlini

Re: [PATCH,testsuite,committed] Skip some tests for powerpc*-*-darwin*

2012-03-12 Thread Rainer Orth
William J. Schmidt wschm...@linux.vnet.ibm.com writes: Index: gcc/testsuite/gcc.target/powerpc/pr46728-13.c === --- gcc/testsuite/gcc.target/powerpc/pr46728-13.c (revision 185247) +++

Re: [C++ Patch] PR 52299, bogus div by zero warning

2012-03-12 Thread Jason Merrill
OK. Jason

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Mike Stump
On Mar 12, 2012, at 10:44 AM, Rainer Orth wrote: Since even extended support for Solaris 8 ends by March 31st, this patch removes Solaris 8 support from mainline. One of the nice things about gcc is that gcc usually still works, long after a vendor has abandoned a machine. I rather like that

Re: Remove obsolete IRIX 6.5 support

2012-03-12 Thread Richard Sandiford
OK from my point of view, except... Rainer Orth r...@cebitec.uni-bielefeld.de writes: * config/mips/mips.h (TARGET_GPWORD): Remove IRIX 6 N64 handling. (TARGET_IRIX6): Remove. (TARGET_CPU_CPP_BUILTINS): Remove IRIX 6 handling. Don't define LANGUAGE_C, _LANGUAGE_C for

Re: [PATCH,testsuite,committed] Skip some tests for powerpc*-*-darwin*

2012-03-12 Thread Rainer Orth
Rainer Orth r...@cebitec.uni-bielefeld.de writes: +/* { dg-skip-if { powerpc*-*-darwin* } { * } { } } */ Please omit defaults to dg-skip-if: { * } { } are unnecessary and make the directive harder to read. Oh, I forgot: please indicate why you are skipping the test in the comment field,

Re: PATCH: Call gen_tls_global_dynamic_64_mode and en_tls_local_dynamic_base_64_mode

2012-03-12 Thread H.J. Lu
On Mon, Mar 12, 2012 at 9:50 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Mar 12, 2012 at 12:46 AM, Uros Bizjak ubiz...@gmail.com wrote: On Mon, Mar 12, 2012 at 12:26 AM, H.J. Lu hongjiu...@intel.com wrote: Pmode may be DImode for x32.  This patch calls gen_tls_global_dynamic_64_mode and

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-12 Thread Rainer Orth
Jason Merrill ja...@redhat.com writes: On 03/12/2012 01:41 PM, Joseph S. Myers wrote: As a *target macro* it makes sense to remove it - reomve it from defaults.h, make it purely internal to dwarf2out.c. But I think it makes sense to have it inside dwarf2out.c Agreed. Fine with me. I've

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Richard Kenner
One of the nice things about gcc is that gcc usually still works, long after a vendor has abandoned a machine. I rather like that gcc will just work, unlike vendor software, which often says, please buy a new machine. One doesn't have to remove support in gcc for something, just because a

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Rainer Orth
Mike Stump mikest...@comcast.net writes: On Mar 12, 2012, at 10:44 AM, Rainer Orth wrote: Since even extended support for Solaris 8 ends by March 31st, this patch removes Solaris 8 support from mainline. One of the nice things about gcc is that gcc usually still works, long after a vendor

Re: [Ping][PATCH, libstdc++-v3] Enable to cross-test libstdc++ on simulator

2012-03-12 Thread Mike Stump
On Mar 10, 2012, at 3:25 PM, Jonathan Wakely wrote: On 7 March 2012 05:22, Terry Guo wrote: Hello, Can anybody please review and approve the following simple patch? Thanks very much. http://gcc.gnu.org/ml/libstdc++/2011-08/msg00063.html Ok. Ok for the release branches as relevant, if

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Rainer Orth
ken...@vlsi1.ultra.nyu.edu (Richard Kenner) writes: One of the nice things about gcc is that gcc usually still works, long after a vendor has abandoned a machine. I rather like that gcc will just work, unlike vendor software, which often says, please buy a new machine. One doesn't have to

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Steven Bosscher
On Mon, Mar 12, 2012 at 7:37 PM, Mike Stump mikest...@comcast.net wrote: On Mar 12, 2012, at 10:44 AM, Rainer Orth wrote: Since even extended support for Solaris 8 ends by March 31st, this patch removes Solaris 8 support from mainline. One of the nice things about gcc is that gcc usually

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-12 Thread Jason Merrill
On 03/12/2012 02:45 PM, Rainer Orth wrote: What about DWARF_INITIAL_LENGTH_SIZE? Keep it as a macro inside dwarf2out.c, too? Please. Jason

[google/4.6] Add xfails for arm-grtev2-linux-gnueabi (issue5794063)

2012-03-12 Thread Doug Kwan
Hi Diego This patch adds arm-grtev2-linux-gnueabi.xfail for our 4.6 branch so that we can track regressions. This just established the test baseline. The failures need to be investigated. -Doug 2012-03-12 Doug Kwan dougk...@google.com *

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Mike Stump
On Mar 12, 2012, at 11:48 AM, Rainer Orth wrote: But both for those and Solaris 8, the time has come where maintaining them is more trouble than it's worth, The nice thing is, anyone that disagrees with you, will step forward. :-) Notice, I didn't disagree with you... I was only railing

Re: [google/4.6] Add xfails for arm-grtev2-linux-gnueabi (issue5794063)

2012-03-12 Thread Diego Novillo
On 12/03/12 14:58 , Doug Kwan wrote: 2012-03-12 Doug Kwandougk...@google.com * contrib/testsuite-management/arm-grtev2-linux-gnueabi.xfail: New file. OK. Diego.

Re: [PATCH,testsuite,committed] Skip some tests for

2012-03-12 Thread William J. Schmidt
OK, will do. I just copied the style of some other test cases that skipped darwin, assuming that was the preferred syntax. Thanks, Bill On Mon, 2012-03-12 at 19:40 +0100, Rainer Orth wrote: Rainer Orth r...@cebitec.uni-bielefeld.de writes: +/* { dg-skip-if { powerpc*-*-darwin* } { * } {

Re: [PATCH,testsuite,committed] Skip some tests for

2012-03-12 Thread Rainer Orth
William J. Schmidt wschm...@linux.vnet.ibm.com writes: OK, will do. I just copied the style of some other test cases that skipped darwin, assuming that was the preferred syntax. No problem, I'll probably make a pass over the whole testsuite to remove that cruft to avoid misleading others.

  1   2   >