diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2012-08-01 03:22:32 +0000 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2012-08-01 03:22:32 +0000 |
| commit | 812cf361d8c2a75c4b0e658034be30d782000deb (patch) | |
| tree | 97b3ca34655d53f0ed97771e5c9ea2a9afc0ba95 /sim | |
| parent | f7727c5ce8954ce6360c67a63cedd73791354faf (diff) | |
| download | gdb-812cf361d8c2a75c4b0e658034be30d782000deb.tar.gz | |
sim: bfin: update return value in bfin_reg_{fetch,store}
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>
Diffstat (limited to 'sim')
| -rw-r--r-- | sim/bfin/ChangeLog | 6 | ||||
| -rw-r--r-- | sim/bfin/machs.c | 8 |
2 files changed, 10 insertions, 4 deletions
diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog index 5d09b5a6056..49f2d6b312d 100644 --- a/sim/bfin/ChangeLog +++ b/sim/bfin/ChangeLog @@ -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. diff --git a/sim/bfin/machs.c b/sim/bfin/machs.c index 9d0f26314e4..04e39b597ca 100644 --- a/sim/bfin/machs.c +++ b/sim/bfin/machs.c @@ -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 |