This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [PATCH] S/390 port modernization 1/4
- From: Ulrich Weigand <weigand at i1 dot informatik dot uni-erlangen dot de>
- To: jimb at redhat dot com (Jim Blandy)
- Cc: weigand at i1 dot informatik dot uni-erlangen dot de (Ulrich Weigand), gdb-patches at sources dot redhat dot com, uweigand at de dot ibm dot com
- Date: Thu, 4 Dec 2003 23:11:47 +0100 (CET)
- Subject: Re: [PATCH] S/390 port modernization 1/4
Jim Blandy wrote:
> The regset_from_core_section stuff is done exactly the way the i386
> and x86-64 do it, so I won't criticize it. But there are two things I
> wonder about:
>
> - Since core file formats are really OS-specific (ABI's often don't
> specify them), wouldn't it make more sense for the *-nat.c file to
> register an OS/ABI handler, and have that handler register the
> regset_from_core_section method?
Defining the core file format in *-nat.c would mean that a cross-gdb
wouldn't understand core files, right?
> - Why does everyone allocate the 'struct regset' objects dynamically?
> The regset layouts are structures in header files, so there's only a
> static set of them; one could simply define initialized instances of
> 'struct regset', and the regset_from_core_section method could
> return them. That seems simpler than allocating them dynamically,
> and caching them in the tdep structure.
OK, I'll change that.
Bye,
Ulrich
--
Dr. Ulrich Weigand
weigand@informatik.uni-erlangen.de