Re: [PATCH, i386] RTM support

2012-03-12 Thread Kirill Yukhin
I forgot to commit headers, but that was fixed in an hour. At the moment trunk builds K On Mon, Mar 12, 2012 at 6:36 PM, H.J. Lu wrote: > 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!

Re: [google] Use delete with size parameter in STL deallocate (issue5794070)

2012-03-12 Thread Xinliang David Li
ok. As Ollie said, it needs to be in google/gcc-4_7 as well. David On Mon, Mar 12, 2012 at 6:03 PM, Easwaran Raman wrote: > This patch makes -fsized-delete define a macro __GXX_DELETE_WITH_SIZE__. If > this macro is defined, the deallocate function in new_allocator.h invokes > the two parameter

Re: PATCH: Properly generate X32 IE sequence

2012-03-12 Thread H.J. Lu
On Mon, Mar 12, 2012 at 3:35 PM, H.J. Lu wrote: > On Mon, Mar 12, 2012 at 12:39 PM, Uros Bizjak wrote: >> On Sun, Mar 11, 2012 at 10:24 PM, H.J. Lu wrote: >> >>> Here is the patch which is equivalent to clearing MASK_TLS_DIRECT_SEG_REFS >>> when Pmode != word_mode.  We need to keep >>> >>>      

Re: [google] Use delete with size parameter in STL deallocate (issue5794070)

2012-03-12 Thread Ollie Wild
On Mon, Mar 12, 2012 at 8:03 PM, Easwaran Raman wrote: > > OK for google/main and google/4_6 branches? You will need to check this into google/gcc-4_7 as well. Ollie

[google] Use delete with size parameter in STL deallocate (issue5794070)

2012-03-12 Thread Easwaran Raman
This patch makes -fsized-delete define a macro __GXX_DELETE_WITH_SIZE__. If this macro is defined, the deallocate function in new_allocator.h invokes the two parameter delete operator with the size of the object. Tested by bootstrapping the compiler and ensuring no tests are broken. Also verified t

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

2012-03-12 Thread 關振德
Hi Ramana, We know the limit of the QEMU and already noticed failure due to the simulator. Like I said, this is used as the baseline. We are going to look at the failures carefully to categorize them. We noticed that some tests fail randomly on QEMU, these are marked as flaky and the validat

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

2012-03-12 Thread Ramana Radhakrishnan
On 12 March 2012 18:58, Doug Kwan wrote: > 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. Just out of curiosity, were these w

[Patch ARM] Fix definition of MALLOC_ABI_ALIGNMENT.

2012-03-12 Thread Ramana Radhakrishnan
Hi, This sets MALLOC_ABI_ALIGNMENT to 64 bits on ARM which should be safe on all AAPCS implementations given that this is a requirement on our alignment specifiers. I would expect this to be true for most modern malloc implementations on AAPCS targets - Ok ? Ramana 2012-03-12 Ramana Radhakris

Re: PATCH: Properly generate X32 IE sequence

2012-03-12 Thread H.J. Lu
On Mon, Mar 12, 2012 at 12:39 PM, Uros Bizjak wrote: > On Sun, Mar 11, 2012 at 10:24 PM, H.J. Lu wrote: > >> Here is the patch which is equivalent to clearing MASK_TLS_DIRECT_SEG_REFS >> when Pmode != word_mode.  We need to keep >> >>          else if (Pmode == SImode) >>            { >>        

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Eric Botcazou
> * Although libgcc/config/sparc/sol2-unwind.h could be simplified quite a > bit, perhaps we could go further: I wonder if the remaining nframes = 2 > case really applies to Solaris 9. If not, one could massivly simplify > this code. Please no, let's keep being very conservative with this s

Re: [Patch, libfortran] Reduce default precision for list-directed and G0 real output

2012-03-12 Thread FX
> Thus, I suggest that the choice of d should be based on readability > and usefulness for the common case rather than guaranteeing an exact > roundtrip. I'm not sure I see the value in this choice. In addition to standard requirements, quality of implementation issues shouldn't be neglected. At

Re: [google] Add -gfission support to GCC (issue 5754090)

2012-03-12 Thread saugustine
On 2012/03/12 21:32:46, Cary wrote: [Revised patch to fix problem with ASM_FINAL_SPEC in the case where neither -c nor -o are specified.] Add GCC support for -gfission option. Debug info is partitioned into skeleton sections that will remain in the .o file, and "dwo" sections that will be mov

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:58 PM, Uros Bizjak wrote: > On Mon, Mar 12, 2012 at 7:42 PM, H.J. Lu wrote: > >>> 2012-03-12  H.J. Lu   >>> >>>        * config/i386/i386.c (ix86_gen_tls_global_dynamic_64): New. >>>        (ix86_gen_tls_local_dynamic_base_64): Likewise. >>>        (ix86_option_override

[google] Add -gfission support to GCC (issue5754090)

2012-03-12 Thread Cary Coutant
[Revised patch to fix problem with ASM_FINAL_SPEC in the case where neither -c nor -o are specified.] Add GCC support for -gfission option. Debug info is partitioned into skeleton sections that will remain in the .o file, and "dwo" sections that will be moved to a .dwo file. After compilation, t

[Patch, libfortran] Reduce default precision for list-directed and G0 real output

2012-03-12 Thread Janne Blomqvist
Hi, currently when writing a value of type real or complex using list-directed output, the G0 edit descriptor, or namelist output, gfortran chooses the number of significant digits such that a binary->ascii->binary roundtrip recovers the original value exactly, per IEEE 754-2008. Assuming, of cour

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

2012-03-12 Thread Gerald Pfeifer
On Mon, 12 Mar 2012, Rainer Orth wrote: > Tested with make doc/gccinstall.info doc/gccinstall.pdf, ok for mainline > and 4.7 branch? +Sun does not ship a C compiler with Solaris 2 before Solaris 10, though +you can download the Sun Studio compilers for free. In Solaris 10 and +11, GCC 3.4.3 is av

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

2012-03-12 Thread Mike Stump
On Mar 12, 2012, at 12:37 PM, Diego Novillo wrote: > Ideally, though, we would not even need this hack. 'make check' should > return 0 when every test is nominal. Period. Yeah, that pig has yet to achieve lift-off. :-)

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

2012-03-12 Thread Martin Jambor
Hi, On Mon, Mar 12, 2012 at 04:26:21PM +0100, Michael Matz wrote: > 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

[google] Add -gfission support to GCC (issue5754090)

2012-03-12 Thread Cary Coutant
Add GCC support for -gfission option. Debug info is partitioned into skeleton sections that will remain in the .o file, and "dwo" sections that will be moved to a .dwo file. After compilation, the gcc driver calls objcopy twice: once to extract the "dwo" sections into the .dwo file, and a second

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Tom Tromey
> "Rainer" == Rainer Orth writes: Rainer> libcpp: Rainer> * lex.c: Remove Solaris 8 reference. Also ok. Tom

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Tom Tromey
> "Rainer" == Rainer Orth writes: Rainer> libjava: Rainer> * configure.ac (THREADLIBS): Remove *-*-solaris2.8 handling. Rainer> * configure: Regenerate. This part is ok. Thanks. Tom

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

2012-03-12 Thread Eric Botcazou
> Thanks. That seems to get me into Ada bootstrap land together with > the following two patches applied (but not when I apply only the > first patch due to the stale TREE_OVERFLOW bits on type/decl sizes). You're welcome. Let me give them a whirl... -- Eric Botcazou

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Jonathan Wakely
On 12 March 2012 17:44, Rainer Orth wrote: > 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.

Re: Remove obsolete IRIX 6.5 support

2012-03-12 Thread Jonathan Wakely
On 12 March 2012 17:03, Rainer Orth wrote: > Continuing with the patches to remove obsolete ports, here's the IRIX > 6.5 removal patch. ... > * libstdc++-v3/doc/xml/manual/internals.xml still uses IRIX as an >  example.  I've kept that part since I guess it's up the libstdc++ >  maintainers to deci

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 7:42 PM, H.J. Lu wrote: >> 2012-03-12  H.J. Lu   >> >>        * config/i386/i386.c (ix86_gen_tls_global_dynamic_64): New. >>        (ix86_gen_tls_local_dynamic_base_64): Likewise. >>        (ix86_option_override_internal): Set ix86_gen_tls_global_dynamic_64 >>        and i

Re: PATCH: Properly generate X32 IE sequence

2012-03-12 Thread Uros Bizjak
On Sun, Mar 11, 2012 at 10:24 PM, H.J. Lu wrote: > Here is the patch which is equivalent to clearing MASK_TLS_DIRECT_SEG_REFS > when Pmode != word_mode.  We need to keep > >          else if (Pmode == SImode) >            { >              /* Always generate >                        movl %fs:0, %r

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

2012-03-12 Thread Diego Novillo
On 12/03/12 15:27 , Mike Stump wrote: So, I guess the question is, is there a down side to doing that? I can imagine a if [ -r "$srcdir/contrib/testsuite-management/$target.xfail ]; then $srcdir/contrib/testsuite-management/validate_failures.py bla bla fi Ye

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Bruce Korb
On 03/12/12 10:44, Rainer Orth wrote: fixincludes: * inclhack.def (math_exception): Remove duplicate. (solaris_cond_init): Remove. (solaris_sys_va_list): Remove Solaris 8 support. I think I probably hate those COND initializers more than any other thing that gets

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

2012-03-12 Thread Mike Stump
On Mar 12, 2012, at 12:29 PM, William J. Schmidt wrote: > Thanks, Rainer. Here's a revised patch. OK to commit? Ok. Thanks.

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

2012-03-12 Thread William J. Schmidt
On Mon, 2012-03-12 at 20:15 +0100, Rainer Orth wrote: > "William J. Schmidt" 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

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

2012-03-12 Thread Mike Stump
On Mar 12, 2012, at 11:59 AM, Diego Novillo wrote: > On 12/03/12 14:58 , Doug Kwan wrote: > >> 2012-03-12 Doug Kwan >> >> * contrib/testsuite-management/arm-grtev2-linux-gnueabi.xfail: >> New file. > > OK. Hum, kinda makes be wish we had all the xfail files for the release branches

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

2012-03-12 Thread Rainer Orth
"William J. Schmidt" 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. Rainer -- --

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 writes: > > >> +/* { dg-skip-if "" { powerpc*-*-darwin* } { "*" } { "" } } */ > > > > Plea

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

2012-03-12 Thread Mike Stump
On Mar 12, 2012, at 11:40 AM, Rainer Orth wrote: > Oh, I forgot: please indicate why you are skipping the test in the > comment field, like: > > /* { dg-skip-if "PR tree-optimization/46728" { powerpc*-*-darwin* } } */ "-mpowerpc-gpopt unsupported" is arguably a better comment, as one can see thi

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 Kwan * contrib/testsuite-management/arm-grtev2-linux-gnueabi.xfail: New file. OK. Diego.

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

[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 * contrib/testsuite-management/arm-grtev2-

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

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Steven Bosscher
On Mon, Mar 12, 2012 at 7:37 PM, Mike Stump 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 still works, long a

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 ha

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 rel

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Rainer Orth
Mike Stump 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 has abandoned a

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

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-12 Thread Rainer Orth
Jason Merrill 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 just been

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 9:50 AM, H.J. Lu wrote: > 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 Pmo

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

2012-03-12 Thread Rainer Orth
Rainer Orth 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, like: /* { dg-s

Re: Remove obsolete IRIX 6.5 support

2012-03-12 Thread Richard Sandiford
OK from my point of view, except... Rainer Orth 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 Objective-C. ...looking at

Re: Remove DWARF_OFFSET_SIZE, DWARF_INITIAL_LENGTH_SIZE

2012-03-12 Thread Jason Merrill
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. 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: [C++ Patch] PR 52299, bogus div by zero warning

2012-03-12 Thread Jason Merrill
OK. Jason

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

2012-03-12 Thread Rainer Orth
"William J. Schmidt" writes: > Index: gcc/testsuite/gcc.target/powerpc/pr46728-13.c > === > --- gcc/testsuite/gcc.target/powerpc/pr46728-13.c (revision 185247) > +++ gcc/testsuite/gcc.target/powerpc/pr46728-13.c (working copy

[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 PR c++/52299

Re: Remove obsolete Solaris 8 support

2012-03-12 Thread Ian Lance Taylor
Rainer Orth 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 libgo patches to me rather

[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 PR tree-optimization/46728 * gcc.target/powerpc/pr46728-4.c: Skip for powerpc*-*-darwin*. * gcc.target/powerpc/pr46728-5.c: Likewise. * gcc.target/powerpc/pr46728-8.c:

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

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

[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: [Patch ARM/ configury] Add fall-back check for gnu_unique_object

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

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

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,

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

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

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

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

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

[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: 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! :)

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:

[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

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

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~

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

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

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

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

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

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

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

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

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

  1   2   >