This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
Re: [RFA] sh-sim loose ends
- From: Joern Rennecke <joern dot rennecke at superh dot com>
- To: msnyder at redhat dot com (Michael Snyder)
- Cc: amylaar at fairadsl dot co dot uk (Joern Rennecke), joern dot rennecke at superh dot com, gdb-patches at sources dot redhat dot com
- Date: Thu, 12 Feb 2004 22:39:19 +0000 (GMT)
- Subject: Re: [RFA] sh-sim loose ends
> * gencode.c (movt): Modifies R[n]; call 'L' macro.
That doesn't make sense, 'L' simulates the data read memory latency of an
SH[123]. movt doesn't incur such a latency.
> (trapa): Factor out duplicate variable 'imm' (same as 'i').
OK. But I don't see why you need the cast to long.
> (sleep, trapa, ppi): Use SET_NIP to modify nip.
I don't see any need for this. RAISE_EXCEPTION already clears
saved_state.asregs.insn_end , so that takes care of the
exceptions that might arise during sleep.
For trapa, that leaves just the possibility that we fail to miss a
loop bound that is set to somewhere inside a profiler trap; I think
you deserve whatever you get when you do that.
Similar for ppi; are you afrais that we fail to miss a loop bound
that is set to field_b of a ppi insn?