[PATCH] adds powerpc-*-freebsd? to mainline
Geoff Keating
geoffk@geoffk.org
Sat Nov 3 20:16:00 GMT 2001
> Date: Tue, 13 Nov 2001 12:42:46 -0800
> From: "David O'Brien" <obrien@FreeBSD.org>
> Cc: gcc-patches@gcc.gnu.org
> On Tue, Nov 13, 2001 at 02:06:46AM -0800, Geoff Keating wrote:
> > > --- elfos.h 2001/11/11 05:56:38 1.36
> > > +++ elfos.h 2001/11/12 21:53:27
> > > -#define USING_ELFOS_H
> > > +#define _USING_ELFOS_H
> >
> > This is not OK. Why make this change?
>
> Just to be the same in mainline and 3.0_branch.
You mean, you were trying to import a patch from the branch to the
mainline? That's not the right way to do it, it should only go the
other way; generally, the branch has more bugs and fewer features than
the mainline.
> > Symbols starting with an
> > underscore and an uppercase letter are reserved for the
> > implementation.
>
> Yes. I sometimes forget that GCC is not the system (implementation)
> compiler everywhere.
Even if GCC is the system compiler, elfos.h is not a system header,
so it shouldn't use namespace reserved for the system.
> Does this mean that other than this part of the patch, you are OK with my
> patch and it is OK to commit it?
No, not least because without that change it won't work; please re-submit.
--
- Geoffrey Keating <geoffk@geoffk.org> <geoffk@redhat.com>
More information about the Gcc-patches
mailing list