This is the mail archive of the mailing list for the binutils 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: [parisc-linux] [patch] Remove magic constant from gas/tc-hppa.c

Le samedi 21 octobre 2006 Ã 12:46 -0600, Matthew Wilcox a Ãcrit :

> >    /* Set the default machine type.  */
> > -  if (!bfd_set_arch_mach (stdoutput, bfd_arch_hppa, DEFAULT_LEVEL))
> > +#if TARGET_ARCH_SIZE == 64
> > +  if (!bfd_set_arch_mach (stdoutput, bfd_arch_hppa, bfd_mach_hppa10))
> > +#else
> > +  if (!bfd_set_arch_mach (stdoutput, bfd_arch_hppa, bfd_mach_hppa20w))
> > +#endif
> >      as_warn (_("could not set architecture and machine"));
> >  
> >    /* Folding of text and data segments fails miserably on the PA.
> My personal preference would be to see this patch as:
>  #if TARGET_ARCH_SIZE == 64
> -#define DEFAULT_LEVEL 25
> +#define DEFAULT_LEVEL bfd_mach_hppa20w
>  #else
> -#define DEFAULT_LEVEL 10
> +#define DEFAULT_LEVEL bfd_mach_hppa10
>  #endif
> ... particularly since you seem to have got them the wrong way round in
> this patch.

Oh, whups.  /me wonders with which toolchain he used to build the test

I opted for eliminating the DEFAULT_LEVEL because it's not clear at a
glance what it's a level of, it's only used in that one place, and the
construct isn't used in any of the other architectures.

In the tc-sparc.c file, they set a variable "mach" and just use that.  I
can do the same.  I'll cook it and resend after my afternoon honey-do's.

> I'd also dearly love it if someone could unify the hppa and hppa64 ports
> so we don't have to build two toolchains for one port.  Just like sparc,
> mips, s390 and i386.

In my local tree, I've been playing with extracting pieces out for use
in both with that in mind.  I'm a ways away from it.  I also don't have
an hppa-hpux system to test against, so I'm worried about accidentally
breaking that.  I should solve that and send some of the obvious pieces
of the hppa64-linux port in.

Jeff Bailey -

Attachment: signature.asc
Description: Ceci est une partie de message=?ISO-8859-1?Q?num=E9riquement?= =?ISO-8859-1?Q?_sign=E9e?=

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