From b4876e0485f5d16a5d9e213c0b15466323170cf0 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Sat, 18 Jun 2011 21:59:38 +0000 Subject: [PATCH] sim: bfin: do not touch ASTAT[V] when shifting accumulators If we're shifting accumulators, we don't want to touch the V bit in ASTAT, so add size checks to the ashiftrt/lshiftrt helpers. Signed-off-by: Robin Getz Signed-off-by: Mike Frysinger --- sim/bfin/ChangeLog | 5 +++++ sim/bfin/bfin-sim.c | 7 ++++--- 2 files changed, 9 insertions(+), 3 deletions(-) diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog index 4e656d7..8fd629a 100644 --- a/sim/bfin/ChangeLog +++ b/sim/bfin/ChangeLog @@ -1,5 +1,10 @@ 2011-06-18 Robin Getz + * bfin-sim.c (ashiftrt): If size is 40, do not call SET_ASTATREG. + (lshiftrt): Likewise. + +2011-06-18 Robin Getz + * bfin-sim.c (decode_dsp32shift_0): Use get_unextended_acc rather than get_extended_acc in LSHIFT insns. diff --git a/sim/bfin/bfin-sim.c b/sim/bfin/bfin-sim.c index 2c71f6c..dbfce6c 100644 --- a/sim/bfin/bfin-sim.c +++ b/sim/bfin/bfin-sim.c @@ -713,8 +713,8 @@ ashiftrt (SIM_CPU *cpu, bu40 val, int cnt, int size) val |= sgn; SET_ASTATREG (an, val >> (size - 1)); SET_ASTATREG (az, val == 0); - /* XXX: Need to check ASTAT[v] behavior here. */ - SET_ASTATREG (v, 0); + if (size != 40) + SET_ASTATREG (v, 0); return val; } @@ -742,7 +742,8 @@ lshiftrt (SIM_CPU *cpu, bu64 val, int cnt, int size) } SET_ASTATREG (an, val >> (size - 1)); SET_ASTATREG (az, val == 0); - SET_ASTATREG (v, 0); + if (size != 40) + SET_ASTATREG (v, 0); return val; } -- 2.7.4