This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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: [PATCH] Remove i386bsd-tdep.o from x86_64-*-elf* target


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


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