This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
Re: Bootstrap failure on cygwin - undefined reference to `ix86_valid_decl_attribute_p'
- To: "Joseph S. Myers" <jsm28 at cam dot ac dot uk>
- Subject: Re: Bootstrap failure on cygwin - undefined reference to `ix86_valid_decl_attribute_p'
- From: Graham Stott <grahams at redhat dot com>
- Date: Fri, 29 Jun 2001 12:24:32 +0100
- Cc: Neil Booth <neil at daikokuya dot demon dot co dot uk>,"Billinghurst, David (CRTS)" <David dot Billinghurst at riotinto dot com>,gcc-bugs at gcc dot gnu dot org
- References: <Pine.LNX.4.33.0106291104450.5682-100000@kern.srcf.societies.cam.ac.uk>
"Joseph S. Myers" wrote:
>
> On Thu, 28 Jun 2001, Neil Booth wrote:
>
> > Sorry for the breakage. I'm going to commit this as obvious without a
> > bootstrap, I'm pretty sure it fixes it.
> >
> > Neil.
> >
> > config:
> > * i386/i386.c (ix86_valid_type_attribute_p): Give external
> > linkage, move declaration to...
> > * i386/i386-protos.h: ...here.
>
> Surely part of the problem here (or at least a related problem) is that
> the target initializer in i386.c isn't using the cygwin versions of the
> functions when necessary?
Joseph I agree Neils patch has broken the cygwin toolchain because the
cygwin specific validation routines arn't being used on cygwin.
The patch just allows a broken cygwin toolchain to be built.
I think the solution is to move the definition of the standard x86
TARGET_xxx macros from i386.c to i386.h and then have cygwin.h override
the standard x86 macros as needed.
>
> The old VALID_MACHINE_* macros still seem to be defined in alpha/vms.h and
> i386/cygwin.h - and those need to be moved over to the new scheme.
>
> --
> Joseph S. Myers
> jsm28@cam.ac.uk
Graham