Here's the adjustment to movem that I promised earlier. Do you agree it's a bit cleaner?
Also, I'll note that we have to catch the two illegal addressing modes somehow. This is one way; one could also add entries to register_m68k_insns that specifically cover the modes. (Noting, for example, that EXTSB already overlaps MOVEM via the illegal addressing mode 000 (data register direct)). Also, we should not abort qemu on illegal insn. We should signal the exception normally, which will get propagated appropriately. In the case of m68k-linux-user, via forced SIGILL. For m68k-softmmu, via exception into the kernel, which can then kill the user process. r~ Laurent Vivier (1): target-m68k: implement 680x0 movem Richard Henderson (1): target-m68k: Do not cpu_abort on undefined insns target-m68k/translate.c | 138 +++++++++++++++++++++++++++++++++++++++--------- 1 file changed, 112 insertions(+), 26 deletions(-) -- 2.7.4
