Re: [Qemu-devel] [PATCH] target-arm: fix UMAAL instruction

[Peter Maydell]

On 31 December 2010 19:11, Aurelien Jarno <address@hidden> wrote:
> UMAAL should use unsigned multiply instead of signed.

Looks like we've both been working on the same thing this
evening :-)

> This patch fixes this issue by handling UMAAL separately from
> UMULL/UMLAL/SMULL/SUMLAL are these instructions are different

"SMLAL as".

> It also explicitely list instructions in case and catch
> inexistant instruction as illegal.

"explicitly" and "nonexistent", if you're tweaking the commit msg
anyway.

(If you want to use the ARM ARM terminology these instruction
patterns are "UNDEFINED" rather than "illegal".)

> +                    case 4:
> +                        /* 64 bit mul double accumulate */

Can we list the instruction mnemonic in the comment too?
(Saves future readers having to wade through the decode logic...)
This case is UMAAL.

> +                    case 8: case 9: case 10: case 11:
> +                    case 12: case 13: case 14: case 15:

/* UMULL, UMLAL, SMULL, SMLAL. */

I've run the patch through my random-sequence test and it passes.

-- PMM