[PATCH] Remove i386bsd-tdep.o from x86_64-*-elf* target

Pedro Alves palves@redhat.com
Tue Mar 29 17:41:00 GMT 2016


On 03/25/2016 04:30 PM, Doug Evans wrote:
> Hi all.
> 
> I'm currently working on an x86_64 "bare metal" target and am finding
> various problems (patches to follow once I get things cleaned up).
> 
> In the process of cleaning things up, I'd like to make the following patch.
> It's wrong to have bsd-anything (or linux, or ...)
> in a bare $arch-elf target.
> 
> Ok?

This is fine with me.

Though I too would like to hear from Joel about the rtems bit.

> -x86_64-*-elf* | x86_64-*-rtems*)
> -    gdb_target_obs="amd64-tdep.o i386-tdep.o i387-tdep.o i386bsd-tdep.o"
> -     ;;
> +x86_64-*-elf*)
> +    gdb_target_obs="amd64-tdep.o i386-tdep.o i387-tdep.o"

...

> +x86_64-*-rtems*)
> +    gdb_target_obs="amd64-tdep.o i386-tdep.o i387-tdep.o i386bsd-tdep.o"
> +    ;;

I've seen rtems configure patches fly by before, and IIRC, they tend
to just just make rtems an alias for bare-metal elf.  That seems to
be supported by other gdb/configure.tgt !x86 rtems stanzas.

I'm curious since that file contains support for old BSD a.out
ports (that is, !elf), and I'm proposing removing a.out support
in another patch.

Joel, does rtems really need anything in i386bsd-tdep.c?

Thanks,
Pedro Alves



More information about the Gdb-patches mailing list