This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PING] Enable pragma redefine_extname on all targets


On Fri, 31 Jul 2009, Richard Guenther wrote:

> On Fri, Jul 31, 2009 at 5:11 PM, Olatunji Ruwase<tjruwase@google.com> wrote:
> > On Fri, Jul 31, 2009 at 3:14 AM, Richard
> > Guenther<richard.guenther@gmail.com> wrote:
> >> On Fri, Jul 31, 2009 at 1:20 AM, Olatunji Ruwase<tjruwase@google.com> wrote:
> >>> http://gcc.gnu.org/ml/gcc-patches/2009-07/msg01425.html
> >>
> >> Did you address Josephs comments? ?If so please re-post the fixed patch.
> >>
> >
> > Sorry I did not attach the fixed patch earlier. The patch now
> > addresses Joseph's comments
> > as follows :
> > Documentation and testcases updated in this patch to reflect that
> > #pragma redefine_extname is
> > now supported on all platforms.
> >
> > The affected Solaris specific testcases all pass on x86_64-unknown-linux-gnu.
> >
> > Bootstrapped and tested on x86_64-unknown-linux-gnu.
> 
> The middle-end changes are ok, Joseph needs to approve the C
> frontend changes.

The patch appears to add a bogus blank line at the start of two C front 
end files and to fail to adjust the indentation of the body of an if when 
removing the "if" line.

-- 
Joseph S. Myers
joseph@codesourcery.com

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]