On 30 March 2011 12:41, Dmitry Eremin-Solenikov <dbarysh...@gmail.com> wrote:
> @@ -7172,10 +7210,7 @@ static void disas_arm_insn(CPUState * env,
> DisasContext *s)
> }
> if (insn & (1 << 20)) {
> /* Complete the load. */
> - if (rd == 15)
> - gen_bx(s, tmp);
> - else
> - store_reg(s, rd, tmp);
> + store_reg_from_load(env, s, rn, tmp);
> }
> break;
> case 0x08:
Shouldn't the argument to store_reg_from_load() be 'rd', not 'rn'?
I'm otherwise happy with this patch. Thanks for doing all the changes.
-- PMM
