On 3/30/11, Peter Maydell <peter.mayd...@linaro.org> wrote:
> 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'?
Yeah, it seems so :)
Please commit the fixed patch.
--
With best wishes
Dmitry
