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 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_ and en_tls_local_dynamic_base_64_

2012-03-12 Thread Uros Bizjak
On Mon, Mar 12, 2012 at 12:26 AM, H.J. Lu wrote: > Pmode may be DImode for x32.  This patch calls > gen_tls_global_dynamic_64_ and gen_tls_local_dynamic_base_64_, > depending on Pmode.  Tested on Linux/x86-64.  OK for trunk? > > 2012-03-11  H.J. Lu   > >        * config/i386/i386.c (legitimize_tl

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 wrote: > Tobias Burnus wrote: >> >> If the interface in a PROCEDURE() statement is Bind(C), also the pro

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 >> >> * ginclude/

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

2012-03-12 Thread Uros Bizjak
Hello! Attached patch improves setCC + movzbl to xor + setcc peephole2 to also handle CC setting arithmetic instructions. 2012-03-12 Uros Bizjak * config/i386/i386.md (setcc + movzbl to xor + setcc peephole2): Also convert sequences with CC setting arithmetic instruction. Tes

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 a

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 wi

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 a

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 ma

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 FLAG

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 th

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 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 apply_distri

[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 * config/arm/arm.c

[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 * tree-sra.c (create_access_replacement): Only rename the replacement if we can rewrite it i

Re: [PATCH] Improve SCEV for array element

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 8:13 AM, Jiangning Liu 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 element >

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 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" "") >> +     (matc

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 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 (DECL_CONTEX

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 wrote: > Richard Guenther wrote: >> On Tue, Feb 28, 2012 at 4:10 PM, Ulrich Weigand 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 ... >> > >> > Does this look reason

Re: [ping] Vectorizer patches for 4.8

2012-03-12 Thread Richard Guenther
On Thu, Mar 8, 2012 at 3:56 PM, Ulrich Weigand 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 contribute

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

2012-03-12 Thread Ludovic Courtès
Hi, Thanks for the review. "Joseph S. Myers" skribis: > On Fri, 9 Mar 2012, Ludovic Courtès wrote: > >> * gcc/opts-common.c (generate_option): Don't insert white space in >> `canonical_option' when OPT_INDEX is OPT_l. > > No, opts-common.c should not have special cases for individual

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. Thank

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 wrote: > On Wed, Mar 7, 2012 at 5:51 AM, Richard Guenther > wrote: >> On Wed, Mar 7, 2012 at 1:49 AM, Sriraman Tallam wrote: >>> Patch for CPU detection at run-time. >>> === >>> >>> Patch for CPU detection at run-t

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

Re: [PATCH] Make powerpc honor PROCESSOR_DEFAULT{,64} in tm*.h files

2012-03-12 Thread Richard Guenther
On Fri, Mar 9, 2012 at 5:57 PM, Michael Meissner wrote: > On Fri, Mar 09, 2012 at 11:04:17AM -0500, David Edelsohn wrote: >> Let's try this on trunk and hopefully it can be included in the >> release candidate. > > Committed, subversion id 185146. Ok for the branch. Thanks, Richard. > -- > Mich

[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. Thi

[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 many

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 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 has an effect if

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 wrote: > Woops I forgot the patch. Ok. Thanks, Richard. > Thanks, > Andrew Pinski > > On Fri, Mar 9, 2012 at 11:45 AM, Andrew Pinski > wrote: >> On Tue, Jan 24, 2012 at 2:50 AM, Richard Guenther >> wrote: >>> On Tue, Jan 24, 2012 at 7:34 AM, Andr

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 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 lot o

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 wrote: > On Sat, Mar 10, 2012 at 11:19 AM, Tobias Burnus 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 bootstrap&test this, but trunk >>>

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 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 Research Council o

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

2012-03-12 Thread Dodji Seketeli
Hello Fabien, Fabien Chêne 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 *dec >type

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

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 * doc/invoke.texi (VMS Options): Merge Al

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

2012-03-12 Thread Richard Guenther
On Mon, Mar 12, 2012 at 2:00 PM, Tristan Gingold 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 -mpointer-size. > > Tested by 'make info' > > Ok for trunk ? Ok. Thanks, Rich

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 requir

[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 PR libstdc++/52562 * libsupc++/typeinfo (type_info::name, before, operator==, operator!=): Mark noexcept in C++11

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

2012-03-12 Thread Kaz Kojima
Hi, Richard Guenther wrote: > On Sat, Mar 10, 2012 at 5:25 AM, Sandra Loosemore > 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 regist

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 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 wrote: > > On Thu, Mar 8, 2012 at 3:22 AM, Uros Bizjak wrote: > >> On Fri, Mar 2, 2012 at 10:02 PM, H.J. Lu wrote: > >> > >>> This patches uses word_mode instead of Pmode in loop expand since > >>> word_mode may have bigger size than Pmode.  OK for trun

[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 lto/ * lto-lang.c (builtin_type_for_size): Use lto_type_for_size. fortran/ * f95-lang.c (builtin_type_for_size): Use gfc_type_for_size. c-co

[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 http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01378.htm

[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 3/3] Misaligned MEM_REF reads

2012-03-12 Thread Martin Jambor
Hi, this patch is very similar to the one I posted before (http://gcc.gnu.org/ml/gcc-patches/2012-02/msg01377.html) except that it is now adjusted to sit on top of the new one before this and does not ignore complex numbers. There are more movmisalign_optab generations in this function. There is

[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. Therefore

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 sup

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.

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 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 wrote: > On Wed, Jan 18, 2012 at 3:21 PM, Richard Guenther wrote: >> >> This fixes PR49484 by protecting __gcov_flush against concurrent >> execution.  To be able to use the gthread facility I have to >> introduce the requirement that __GTHREAD_MUT

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 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 -mpointer-size

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

Re: [PATCH, i386] RTM support

2012-03-12 Thread H.J. Lu
On Sun, Mar 11, 2012 at 1:16 AM, Kirill Yukhin wrote: >> >> The patch is OK for mainline, if there are no further comments in next 24h. > > Thank you! > > According to Tobias's input, I've added few lines about RTM to > doc/invoke.texi. If no objections - I'll commit the patch tomorrow. > I think

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

2012-03-12 Thread Ramana Radhakrishnan
On 12 March 2012 10:22, Richard Guenther wrote: > > 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? Ok. From my

[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 20

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 > t

[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 * doc/install.texi (Specific, i?86-*-solaris2.[89]

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

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-

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. 2012-

Re: Remove obsolete OpenBSD/MIPS support

2012-03-12 Thread Rainer Orth
Richard Sandiford writes: > Rainer Orth 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 plan to remove as >> a followup once IRIX is gone).

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 wrote: > On Mon, Mar 12, 2012 at 12:02 AM, H.J. Lu 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/x8

[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 * config.gcc (extra_passes): Remove. * configure.ac (extra_passes): Don't substitute.

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 8

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

2012-03-12 Thread Fabien Chêne
Salut Dodji, 2012/3/12 Dodji Seketeli : [...] >> 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 >>      

Remove non-GAS non-ELF support in alpha backend

2012-03-12 Thread Rainer Orth
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: * HAVE_STAMP_H was only defined by osf5.h, not configure, so the code is stale. * All

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-12 Thread Rainer Orth
Paolo Carlini 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 the > ChangeLog entry

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: `*builtin_setjmp_re

Re: Remove obsolete Tru64 UNIX V5.1B support

2012-03-12 Thread Rainer Orth
David Daney writes: > On 03/06/2012 05:14 AM, Rainer Orth wrote: >> "Joseph S. Myers" 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 1.5 for that and hope I got

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_ and en_tls_local_dynamic_base_64_

2012-03-12 Thread H.J. Lu
On Mon, Mar 12, 2012 at 12:46 AM, Uros Bizjak wrote: > On Mon, Mar 12, 2012 at 12:26 AM, H.J. Lu wrote: > >> Pmode may be DImode for x32.  This patch calls >> gen_tls_global_dynamic_64_ and gen_tls_local_dynamic_base_64_, >> depending on Pmode.  Tested on Linux/x86-64.  OK for trunk? >> >> 2012-0

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

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

2012-03-12 Thread Richard Henderson
On 03/12/12 09:57, Joseph S. Myers wrote: > I wonder whether it will be possible for a future cleanup to make alpha > targets use toplevel elfos.h, and for Alpha GNU/Linux toplevel gnu-user.h > and linux.h as well? Probably we can, at least to a fairly large degree. r~

[Patch,AVR]; PR52499 fix MODE_CODE_BASE_REG_CLASS type

2012-03-12 Thread Georg-Johann Lay
This fixes a problem if avr-gcc is built with C++ Lightly tested. Ok for trunk? Johann PR target/52499 * config/avr/avr.c (avr_mode_code_base_reg_class): Change return type from reg_class_t to enum reg_class. * config/avr/avr-protos.h (avr_mode_code_base_reg_clas

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 i386-pc-solaris

[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 libgcc: * config/arm/crtn.S: Fix typo. gcc/testsuite:

Re: Remove obsolete IRIX 6.5 support

2012-03-12 Thread Bruce Korb
On Mon, Mar 12, 2012 at 10:03 AM, Rainer Orth 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 generators

Re: [google/4.6] Add powerpc64-grtev2-linux-gnu.xfail (issue5794044)

2012-03-12 Thread Diego Novillo
On Fri, Mar 9, 2012 at 16:54, Doug Kwan wrote: > Hi Diego, > >    This patch adds the XFAIL file for powerpc64-grtev2-linux-gnu in 4.6. > > -Doug > > 2012-03-09   Doug Kwan   > >        * contrib/testsuite-management/powerpc64-grtev2-linux-gnu.xfail: >        New file. OK. BTW, these failures ar

Re: Swap SECTION_EXCLUDE and SECTION_MACH_DEP

2012-03-12 Thread Sriraman Tallam
Hi Richard, Is this ok for trunk? Asking you because you approved the patch the first time around. Thanks, -Sri. On Fri, Mar 9, 2012 at 3:10 PM, Sriraman Tallam wrote: > Please see: > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=52545 > > SECTION_EXCLUDE should not take up bits reserved for SE

Re: Swap SECTION_EXCLUDE and SECTION_MACH_DEP

2012-03-12 Thread Richard Henderson
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 Fri, Mar 9, 2012 at 3:10 PM, Sriraman Tallam wrote: >> Please see: >> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=525

libgcc gcov_fork failure on AIX

2012-03-12 Thread David Edelsohn
Richi, Your patch for PR gcov/49484 now causes a bootstrap failure on AIX: /farm/dje/src/src/libgcc/libgcov.c: In function '__gcov_fork': /farm/dje/src/src/libgcc/libgcov.c:1008:5: error: 'mx' undeclared (first use in this function) /farm/dje/src/src/libgcc/libgcov.c:1008:5: note: each undeclared

Re: Remove obsolete IRIX 6.5 support

2012-03-12 Thread Joseph S. Myers
On Mon, 12 Mar 2012, Rainer Orth wrote: > * config/mips/mips-modes.def: Remove IRIX 6 reference. > * config/mips/gnu-user64.h (MIPS_TFMODE_FORMAT): Remove. > * config/mips/mips.c (mips_option_override) [MIPS_TFMODE_FORMAT]: > Remove. I think this means that mips_extended_f

Re: [google/4.6] Add powerpc64-grtev2-linux-gnu.xfail (issue5794044)

2012-03-12 Thread 關振德
Don't know. I am establishing the base line so that we can notice regressions. We will look at them closely one we started tracking the tests. -Doug On Mon, Mar 12, 2012 at 10:23 AM, Diego Novillo wrote: > On Fri, Mar 9, 2012 at 16:54, Doug Kwan wrote: >> Hi Diego, >> >>    This patch adds th

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-12 Thread Joseph S. Myers
On Mon, 12 Mar 2012, Rainer Orth wrote: > 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. As a *target macro* it makes sense to remove it - reomve

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

2012-03-12 Thread Gerald Pfeifer
On Mon, 12 Mar 2012, Richard Guenther wrote: >>> 2012-01-18  Richard Guenther   >>> >>>        * gthr.h (__GTHREAD_MUTEX_INIT_FUNCTION): Adjust specification. >>>        * gthr-posix.h (__GTHREAD_MUTEX_INIT_FUNCTION): Define. >>>        (__gthread_mutex_init_function): New function. >>>        * gt

RE: [Patch,AVR]; PR52499 fix MODE_CODE_BASE_REG_CLASS type

2012-03-12 Thread Weddington, Eric
> -Original Message- > From: Georg-Johann Lay > Sent: Monday, March 12, 2012 11:08 AM > To: gcc-patches@gcc.gnu.org > Cc: Denis Chertykov; Weddington, Eric > Subject: [Patch,AVR]; PR52499 fix MODE_CODE_BASE_REG_CLASS type > > This fixes a problem if avr-gcc is built with C++ > > Lightl

Remove obsolete Solaris 8 support

2012-03-12 Thread Rainer Orth
Since even extended support for Solaris 8 ends by March 31st, this patch removes Solaris 8 support from mainline. Given that extended support for Solaris 9 lasts until October 2014, I guess that will remain in GCC for at least one major release. Again, I don't expect the patch to be controversial

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 failu

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

2012-03-12 Thread Georg-Johann Lay
http://gcc.gnu.org/viewcvs?view=revision&revision=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 1398870..c1b1

Re: Remove obsolete IRIX 6.5 support

2012-03-12 Thread Ian Lance Taylor
Rainer Orth writes: > * I'm removing IRIX-specific parts of libgo. Given that libgo is > imported from upstream (and supposed to work or made work on the 4.7 > branch), I don't know if this a good idea. Yeah, it's not. Just send me the mainline patches to libgo rather than applying them yo

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 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

  1   2   >