diff options
| author | Mike Frysinger <vapier@gentoo.org> | 2023-12-20 23:59:28 -0500 |
|---|---|---|
| committer | Mike Frysinger <vapier@gentoo.org> | 2023-12-21 00:02:15 -0500 |
| commit | 2f84390fd496ef153d7402896466f6303f249479 (patch) | |
| tree | 85c82ec391446ba8fcf11dadf799eb430619ea56 | |
| parent | a4de6c88c9972325bb8ba89f5950fbb35b9cb4a7 (diff) | |
| download | gdb-2f84390fd496ef153d7402896466f6303f249479.zip gdb-2f84390fd496ef153d7402896466f6303f249479.tar.gz gdb-2f84390fd496ef153d7402896466f6303f249479.tar.bz2 | |
sim: bfin: clean up astat reg name decode a little
The compiler pointed out we checked AZ twice. Sort by name to avoid
that in the future, and to make it clearer that we have coverage of
all the bits. And add the bits we were missing.
The order here doesn't matter as it's just turning a pointer into a
human readable string when store tracing is enabled.
| -rw-r--r-- | sim/bfin/bfin-sim.c | 26 |
1 files changed, 14 insertions, 12 deletions
diff --git a/sim/bfin/bfin-sim.c b/sim/bfin/bfin-sim.c index 4fa5ccc..1a9b499 100644 --- a/sim/bfin/bfin-sim.c +++ b/sim/bfin/bfin-sim.c @@ -504,6 +504,16 @@ get_store_name (SIM_CPU *cpu, bu32 *p) return greg_names[4 * 8 + 2]; else if (p == &AWREG (1)) return greg_names[4 * 8 + 3]; + else if (p == &ASTATREG (ac0)) + return "ASTAT[ac0]"; + else if (p == &ASTATREG (ac0_copy)) + return "ASTAT[ac0_copy]"; + else if (p == &ASTATREG (ac1)) + return "ASTAT[ac1]"; + else if (p == &ASTATREG (an)) + return "ASTAT[an]"; + else if (p == &ASTATREG (aq)) + return "ASTAT[aq]"; else if (p == &ASTATREG (av0)) return "ASTAT[av0]"; else if (p == &ASTATREG (av0s)) @@ -512,22 +522,14 @@ get_store_name (SIM_CPU *cpu, bu32 *p) return "ASTAT[av1]"; else if (p == &ASTATREG (av1s)) return "ASTAT[av1s]"; + else if (p == &ASTATREG (az)) + return "ASTAT[az]"; else if (p == &ASTATREG (v)) return "ASTAT[v]"; - else if (p == &ASTATREG (vs)) - return "ASTAT[vs]"; else if (p == &ASTATREG (v_copy)) return "ASTAT[v_copy]"; - else if (p == &ASTATREG (az)) - return "ASTAT[az]"; - else if (p == &ASTATREG (an)) - return "ASTAT[an]"; - else if (p == &ASTATREG (az)) - return "ASTAT[az]"; - else if (p == &ASTATREG (ac0)) - return "ASTAT[ac0]"; - else if (p == &ASTATREG (ac0_copy)) - return "ASTAT[ac0_copy]"; + else if (p == &ASTATREG (vs)) + return "ASTAT[vs]"; else { /* Worry about this when we start to STORE() it. */ |