Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2012-01-24 Thread David Edelsohn
On Tue, Jan 24, 2012 at 6:20 AM, Alan Modra wrote: > On Mon, Dec 05, 2011 at 07:25:35PM -0500, David Edelsohn wrote: >> Is this fixed or is someone writing a follow-up patch? > > This does the trick.  You could also use >        case ${host}:${with_multisubdir} in >            *:*32*) >          

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2012-01-24 Thread Rainer Orth
Alan Modra writes: > On Mon, Dec 05, 2011 at 07:25:35PM -0500, David Edelsohn wrote: >> Is this fixed or is someone writing a follow-up patch? > > This does the trick. You could also use > case ${host}:${with_multisubdir} in > *:*32*) > ;; > powerpc64-*:*

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2012-01-24 Thread Paolo Bonzini
On 01/24/2012 12:20 PM, Alan Modra wrote: On Mon, Dec 05, 2011 at 07:25:35PM -0500, David Edelsohn wrote: Is this fixed or is someone writing a follow-up patch? This does the trick. You could also use case ${host}:${with_multisubdir} in *:*32*) ;;

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2012-01-24 Thread Alan Modra
On Mon, Dec 05, 2011 at 07:25:35PM -0500, David Edelsohn wrote: > Is this fixed or is someone writing a follow-up patch? This does the trick. You could also use case ${host}:${with_multisubdir} in *:*32*) ;; powerpc64-*:* | *:*64*) t

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-12-06 Thread Rainer Orth
David Edelsohn writes: > On Mon, Nov 28, 2011 at 11:03 PM, Alan Modra wrote: >> On Mon, Nov 28, 2011 at 05:15:08PM +0100, Rainer Orth wrote: >>> Alan Modra writes: >>> >>> > On Sun, Nov 27, 2011 at 08:36:38PM -0500, David Edelsohn wrote: >>> >> I was hoping that other targets affected by the pa

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-12-05 Thread David Edelsohn
On Mon, Nov 28, 2011 at 11:03 PM, Alan Modra wrote: > On Mon, Nov 28, 2011 at 05:15:08PM +0100, Rainer Orth wrote: >> Alan Modra writes: >> >> > On Sun, Nov 27, 2011 at 08:36:38PM -0500, David Edelsohn wrote: >> >> I was hoping that other targets affected by the patch would test it as >> >> well

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-28 Thread Alan Modra
On Mon, Nov 28, 2011 at 05:15:08PM +0100, Rainer Orth wrote: > Alan Modra writes: > > > On Sun, Nov 27, 2011 at 08:36:38PM -0500, David Edelsohn wrote: > >> I was hoping that other targets affected by the patch would test it as > >> well. > > > > powerpc-linux and powerpc64-linux look good. > >

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-28 Thread Rainer Orth
Alan Modra writes: > On Sun, Nov 27, 2011 at 08:36:38PM -0500, David Edelsohn wrote: >> I was hoping that other targets affected by the patch would test it as well. > > powerpc-linux and powerpc64-linux look good. Thanks for the confirmation. However, Andreas Tobler pointed out in private mail

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-27 Thread Alan Modra
On Sun, Nov 27, 2011 at 08:36:38PM -0500, David Edelsohn wrote: > I was hoping that other targets affected by the patch would test it as well. powerpc-linux and powerpc64-linux look good. -- Alan Modra Australia Development Lab, IBM

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-27 Thread David Edelsohn
On Mon, Nov 21, 2011 at 11:01 AM, Rainer Orth wrote: > As reported in the PR, powerpc-rtems bootstrap (and certainly several > other powerpc targets) is currently broken since rs6000/ibm-ldouble.c > appears twice in LIB2ADD, once from rs6000/t-ppccomm and > rs6000/t-ppccomm-ldbl, which confuses ma

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-24 Thread Rainer Orth
Joel, > Works for me. I posted test results for powerpc-rtems4.11 > at http://gcc.gnu.org/ml/gcc-testresults/2011-11/msg02314.html > > From the rtems perspective, you can commit it. thanks for the confirmation. It would be great if one of the powerpc maintainers could review it. Thanks.

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-23 Thread Joel Sherrill
Thanks. Works for me. I posted test results for powerpc-rtems4.11 at http://gcc.gnu.org/ml/gcc-testresults/2011-11/msg02314.html From the rtems perspective, you can commit it. --joel On 11/21/2011 12:08 PM, Rainer Orth wrote: Joel Sherrill writes: Does this patch apply OK for others? Ran

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-21 Thread Rainer Orth
Joel Sherrill writes: > Does this patch apply OK for others? > > Ranier.. you can just send me the impacted files if you like. I have > no local changes to libgcc. > > $ cat /tmp/libgcc-t-savresfgpr.patch | patch -p1 > patching file libgcc/config.host > Hunk #1 succeeded at 843 (offset -9 lines)

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-21 Thread Joel Sherrill
On 11/21/2011 11:25 AM, Rainer Orth wrote: Paolo Bonzini writes: Wrong patch attached. Indeed ;-) Does this patch apply OK for others? Ranier.. you can just send me the impacted files if you like. I have no local changes to libgcc. $ cat /tmp/libgcc-t-savresfgpr.patch | patch -p1 patchin

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-21 Thread Rainer Orth
Paolo Bonzini writes: > Wrong patch attached. Indeed ;-) # HG changeset patch # Parent e270cbe9e8641882fbcae2c23e927cdfd2be182f Cleanup rs6000/t-ppccomm configurations (PR other/51022) diff --git a/libgcc/config.host b/libgcc/config.host --- a/libgcc/config.host +++ b/libgcc/config.host @@ -85

Re: [build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-21 Thread Paolo Bonzini
On 11/21/2011 05:01 PM, Rainer Orth wrote: As reported in the PR, powerpc-rtems bootstrap (and certainly several other powerpc targets) is currently broken since rs6000/ibm-ldouble.c appears twice in LIB2ADD, once from rs6000/t-ppccomm and rs6000/t-ppccomm-ldbl, which confuses make. It took me a

[build] Cleanup rs6000/t-ppccomm configurations (PR other/51022)

2011-11-21 Thread Rainer Orth
As reported in the PR, powerpc-rtems bootstrap (and certainly several other powerpc targets) is currently broken since rs6000/ibm-ldouble.c appears twice in LIB2ADD, once from rs6000/t-ppccomm and rs6000/t-ppccomm-ldbl, which confuses make. It took me a while to understand how we got into that sit