[PATCH #1] mips*-kfreebsd-gnu support in BFD
Richard Sandiford
rdsandiford@googlemail.com
Thu Dec 23 12:10:00 GMT 2010
Hi Robert,
I was in the process of applying your patches, and noticed that the
config.bfd entry doesn't look quite right.
Robert Millan <rmh@gnu.org> writes:
> @@ -1015,6 +1015,16 @@ case "${targ}" in
> targ_selvecs="bfd_elf32_tradlittlemips_vec ecoff_big_vec ecoff_little_vec bfd_elf32_ntradbigmips_vec bfd_elf64_tradbigmips_vec bfd_elf32_ntradlittlemips_vec bfd_elf64_tradlittlemips_vec"
> want64=true
> ;;
> + mips*el-*-freebsd* | mips*el-*-kfreebsd*-gnu)
> + targ_defvec=bfd_elf32_tradlittlemips_freebsd_vec
> + targ_selvecs="bfd_elf32_ntradlittlemips_vec bfd_elf32_ntradbigmips_vec bfd_elf32_ntradlittlemips_vec bfd_elf32_tradbigmips_vec bfd_elf64_tradlittlemips_vec bfd_elf64_tradbigmips_vec bfd_elf32_ntradbigmips_freebsd_vec bfd_elf32_tradlittlemips_freebsd_vec bfd_elf32_tradbigmips_freebsd_vec bfd_elf64_tradlittlemips_freebsd_vec bfd_elf64_tradbigmips_freebsd_vec"
> + want64=true
Here you've got bfd_elf32_ntradlittlemips_vec twice (first and third),
no bfd_elf32_tradlittlemips_vec, and no bfd_elf32_ntradlittlemips_freebsd_vec.
Same sort of problems for the big-endian case. This leads to a lot of errors
when running the testsuite for mips64-unknown-kfreebsd-gnu
Also, your GAS patch says that N32 is the default for mips64,
but there's no corresponding config.bfd entry that sets
bfd_elf32_ntrad*mips_freebsd_vec to be the default for
mips64*-kfreebsd-gnu.
Sorry for not picking up on this first time round.
Richard
More information about the Binutils
mailing list