FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

sim: bfin: update return value in bfin_reg_{fetch,store}

Browse source 
The latest gdb sim-remote.c really wants a return value from the fetch/store
register functions, so update the Blackfin sim to avoid the warnings/errors.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
This commit is contained in:
Mike Frysinger 2012-08-01 03:22:32 +00:00
parent 7a2d09ccbf
commit b5539f23a0
2 changed files with 10 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
sim/bfin/ChangeLog
View file

@ -1,3 +1,9 @@
2012-07-31 Mike Frysinger <vapier@gentoo.org>
* machs.c (bfin_reg_fetch): Change return 0 to return -1, and
return -1 to return 4.
(bfin_reg_store): Likewise.
2012-06-15 Joel Brobecker <brobecker@adacore.com>
* config.in, configure: Regenerate.

8
sim/bfin/machs.c
View file

@ -1859,7 +1859,7 @@ bfin_reg_fetch (SIM_CPU *cpu, int rn, unsigned char *buf, int len)
else if (rn == SIM_BFIN_CC_REGNUM)
value = CCREG;
else
return 0; // will be an error in gdb
return -1;
/* Handle our KSP/USP shadowing in SP. While in supervisor mode, we
have the normal SP/USP behavior. User mode is tricky though. */
@ -1874,7 +1874,7 @@ bfin_reg_fetch (SIM_CPU *cpu, int rn, unsigned char *buf, int len)
bfin_store_unsigned_integer (buf, 4, value);
return -1; // disables size checking in gdb
return 4;
}
static int
@ -1893,9 +1893,9 @@ bfin_reg_store (SIM_CPU *cpu, int rn, unsigned char *buf, int len)
else if (rn == SIM_BFIN_CC_REGNUM)
SET_CCREG (value);
else
return 0; // will be an error in gdb
return -1;
return -1; // disables size checking in gdb
return 4;
}
static sim_cia