Re: [patch] multi-arch INIT_FRAME_PC*; Was: New gdb 31 & 64 bit patches for S/390

[Andrew Cagney <ac131313@cygnus.com>]

> 
> Michael, the sparc case has me stumped.  tm-sparc.h defines both INIT_FRAME_PC() and INIT_FRAME_PC_FIRST() so (as far as theories go :-) shouldn't be affected by the change.
> 
>  

Found it.  It needs to be native:

make[1]: Entering directory `/horton/cagney/GDB/native/gdb'
gcc -c -g -O  -I/usr/include/v9  -I. -I/horton/cagney/GDB/src/gdb -I/horton/cagn
ey/GDB/src/gdb/config -DHAVE_CONFIG_H -I/horton/cagney/GDB/src/gdb/../include/op
code -I/horton/cagney/GDB/src/gdb/../readline/.. -I../bfd -I/horton/cagney/GDB/s
rc/gdb/../bfd  -I/horton/cagney/GDB/src/gdb/../include -I../intl -I/horton/cagne
y/GDB/src/gdb/../intl  -DUI_OUT=1  -Werror /horton/cagney/GDB/src/gdb/main.c
In file included from /horton/cagney/GDB/src/gdb/defs.h:1148,
                 from /horton/cagney/GDB/src/gdb/main.c:22:
/horton/cagney/GDB/src/gdb/gdbarch.h:691: warning: `CANNOT_STORE_REGISTER' redef
ined
/horton/cagney/GDB/src/gdb/config/sparc/tm-sparc.h:412: warning: this is the loc
ation of the previous definition
/horton/cagney/GDB/src/gdb/gdbarch.h:830: warning: `INIT_FRAME_PC_FIRST' redefin
ed
/horton/cagney/GDB/src/gdb/config/sparc/tm-sparc.h:538: warning: this is the loc
ation of the previous definition
/horton/cagney/GDB/src/gdb/gdbarch.h:844: warning: `INIT_FRAME_PC' redefined
/horton/cagney/GDB/src/gdb/config/sparc/tm-sparc.h:535: warning: this is the loc
ation of the previous definition
make[1]: *** [main.o] Error 1
make[1]: Leaving directory `/horton/cagney/GDB/native/gdb'
make: *** [all-gdb] Error 2
status=2


I'm going to tweek gdbarch.sh so that it is more pig-headed about this.

	Andrew

PS: Michael, time to use -Werror.