Re: mipxtx39 sim build failure

Mike Frysinger <[email protected]> Thu, 10 Nov 2016 22:37:36 -0800
Newsgroups	gmane.comp.gdb.devel
Message-ID	<[email protected]>
On 10 Nov 2016 15:53, Joel Sherrill wrote:
> I was updating the baseline RTEMS gdb configuration to
> 7.12 from 7.11 and noticed that our mipstx39 build
> (just used for the tx39 simulator) was still at 7.9. I bumped
> it to 7.12 and got the following build errors:
> 
> /usr/bin/gcc -O2 -pipe -I/data/home/joel/rtems-4.11-work/rtems-source-builder/rtems/build/tmp/sb-joel/4.12/rtems-mips/home/joel/rtems-4.11-work/tools/4.12/include -DHAVE_CONFIG_H  -DWITH_ALIGNMENT=NONSTRICT_ALIGNMENT -DWITH_TARGET_WORD_BITSIZE=32 -DWITH_TARGET_WORD_MSB=31 -DWITH_DEFAULT_TARGET_BYTE_ORDER=BFD_ENDIAN_BIG -DWITH_FLOATING_POINT=HARD_FLOATING_POINT -DWITH_TARGET_FLOATING_POINT_BITSIZE=32 -DWITH_HW=1 -DDEFAULT_INLINE=0 -DWITH_RESERVED_BITS=1  -Wall -Wdeclaration-after-statement -Wpointer-arith -Wpointer-sign -Wno-unused -Wunused-value -Wunused-function -Wno-switch -Wno-char-subscripts -Wmissing-prototypes -Wdeclaration-after-statement -Wempty-body -Wmissing-parameter-type -Wold-style-declaration -Wold-style-definition -Wformat-nonliteral   -DSUBTARGET_R3900=1 -DMIPS_MACH_DEFAULT=bfd_mach_mips8000   -I. -I../../../gdb-7.12/sim/mips -I../common -I../../../gdb-7.12/sim/mips/../common -I../../include -I../../../gdb-7.12/sim/mips/../..//include -I../../bfd -I../../../gdb-7.12/sim/mips/../..//bfd -I../../opcodes -I../../../gdb-7.12/sim/mips/../..//opcodes  -g -O2 -c -o dv-tx3904cpu.o -MT dv-tx3904cpu.o -MMD -MP -MF .deps/dv-tx3904cpu.Tpo ../../../gdb-7.12/sim/mips/dv-tx3904cpu.c
> In file included from ../../../gdb-7.12/sim/mips/dv-tx3904cpu.c:22:0:
> ../../../gdb-7.12/sim/mips/dv-tx3904cpu.c: In function ‘deliver_tx3904cpu_interrupt’:
> ../../../gdb-7.12/sim/mips/sim-main.h:679:64: error: ‘SD’ undeclared (first use in this function)
>   #define SignalExceptionNMIReset()            signal_exception (SD, CPU, cia, NMIReset)

the mips sim port is garbage in many ways.  you've found another one.
it's full of unstated variable names like assuming "SD" is the variable
name everywhere, as is "CPU".  it largely comes from the igen source.

looks like i broke this while trying to clean up the global state refs
in the mips sim.  the current_state was a global pointer to the sd that
is passed around that i punted.

at any rate, should be fixed now in master & the gdb-7.12 branch.
-mike
signature.asc (application/pgp-signature, 819 B)