Re: arm-linux native build fails

[Kevin Buettner <kevinb at cygnus dot com>]

On Oct 31, 12:06pm, Philip Blundell wrote:

> I don't seem to be able to build the code in the HEAD at the moment:
> 
> gcc -c -g -O2    -I. -I../../src/gdb -I../../src/gdb/config -DHAVE_CONFIG_H -I../../src/gdb/../include/opcode -I../../src/gdb/../readline/.. -I../bfd -I../../src/gdb/../bfd  -I../../src/gdb/../include -I../intl -I../../src/gdb/../intl  -Wimplicit -Wreturn-type -Wcomment -Wtrigraphs -Wformat -Wparentheses -Wpointer-arith -Wuninitialized  ../../src/gdb/arm-linux-nat.c
> ../../src/gdb/arm-linux-nat.c:579: conflicting types for `fill_gregset'
> ../../src/gdb/gregset.h:52: previous declaration of `fill_gregset'
> ../../src/gdb/arm-linux-nat.c:611: conflicting types for `supply_gregset'
> ../../src/gdb/gregset.h:45: previous declaration of `supply_gregset'
> ../../src/gdb/arm-linux-nat.c:632: conflicting types for `fill_fpregset'
> ../../src/gdb/gregset.h:53: previous declaration of `fill_fpregset'
> ../../src/gdb/arm-linux-nat.c:658: conflicting types for `supply_fpregset'
> ../../src/gdb/gregset.h:46: previous declaration of `supply_fpregset'
> make[1]: *** [arm-linux-nat.o] Error 1

I didn't see these problems when I built earlier today, but I think
it's reasonable to change the function declarators for fill_gregset()
and supply_gregset() to use gdb_gregset_t instead of gregset_t. 
(Likewise, fill_fpregset and supply_fpregset should be changed to use
gdb_fpregset_t instead of fpregset_t.)

In fact, I recently made this change (at Andrew Haley's suggestion) to
ppc-linux-nat.c...

2000-09-29  Kevin Buettner  <kevinb@redhat.com>

	* ppc-linux-nat.c (supply_gregset, fill_gregset): Change type
	of first argument from gregset_t to gdb_gregset_t in order
	to match declarations in gregset.h.
	(supply_fpregset, fill_fpregset): Change type of first argument
	from fpregset_t to gdb_fpregset_t in order to match declarations
	in gregset.h.

Kevin