Re: rs6000 options change for rtems.h

2011-12-06 Thread Joel Sherrill
On 12/02/2011 06:37 PM, Joseph S. Myers wrote: On Fri, 2 Dec 2011, Joel Sherrill wrote: OK. I obviously read too much into the other uses. I did not intend to change semantics just account for the change making this not compile. How does the the new version look? This version is OK. Thanks

Re: rs6000 options change for rtems.h

2011-12-02 Thread Joseph S. Myers
On Fri, 2 Dec 2011, Joel Sherrill wrote: > OK. I obviously read too much into the other uses. > I did not intend to change semantics just account for > the change making this not compile. > > How does the the new version look? This version is OK. -- Joseph S. Myers jos...@codesourcery.com

Re: rs6000 options change for rtems.h

2011-12-02 Thread Joel Sherrill
On 12/02/2011 11:57 AM, Joseph S. Myers wrote: On Fri, 2 Dec 2011, Joel Sherrill wrote: On 12/02/2011 10:38 AM, Joseph S. Myers wrote: On Fri, 2 Dec 2011, Joel Sherrill wrote: 2011-12-02 Joel Sherrill * config/rs6000/rtems.h: Switch to using global_options_set in SUBSUBTARGET_O

Re: rs6000 options change for rtems.h

2011-12-02 Thread Joseph S. Myers
On Fri, 2 Dec 2011, Joel Sherrill wrote: > On 12/02/2011 10:38 AM, Joseph S. Myers wrote: > > On Fri, 2 Dec 2011, Joel Sherrill wrote: > > > > > 2011-12-02 Joel Sherrill > > > > > > * config/rs6000/rtems.h: Switch to using global_options_set > > > in SUBSUBTARGET_OVERRIDE_OPTIONS. > >

Re: rs6000 options change for rtems.h

2011-12-02 Thread Joel Sherrill
On 12/02/2011 10:38 AM, Joseph S. Myers wrote: On Fri, 2 Dec 2011, Joel Sherrill wrote: 2011-12-02 Joel Sherrill * config/rs6000/rtems.h: Switch to using global_options_set in SUBSUBTARGET_OVERRIDE_OPTIONS. Is it deliberate that you are removing the first part of each "if" conditio

Re: rs6000 options change for rtems.h

2011-12-02 Thread Joseph S. Myers
On Fri, 2 Dec 2011, Joel Sherrill wrote: > 2011-12-02 Joel Sherrill > > * config/rs6000/rtems.h: Switch to using global_options_set > in SUBSUBTARGET_OVERRIDE_OPTIONS. Is it deliberate that you are removing the first part of each "if" condition (thus, no longer checking TARGET_HARD_FL

rs6000 options change for rtems.h

2011-12-02 Thread Joel Sherrill
Hi, I have been testing with this for almost a month. It is my attempt to follow the changes I think Joseph made to other rs6000 targets. If this change looks right, I would like to commit it. Test results have been posted for it. Thanks. 2011-12-02 Joel Sherrill * config/rs6000/rtems.