> Please find below an updated patch. Let me know if ok to commit.
Yup, this is OK to commit. Thanks!
> Regards,
> Kaushik
>
> 2013-05-13 Kaushik Phatak
>
> * config/rl78/rl78.md (mulsi3_g13): Add additional 'nop' required
> in multiply-accumulate mode
Hi DJ,
Thanks for quick review.
>> +nop ; Additional nop for MAC
> >
>> mov a, #0x40
> Shouldn't the MOV after the nop add the extra cycle here?
Yes, this nop is not required. Tested without this and works OK.
Please find below an updated patch. Let me know if ok to commit.
Reg
> movwax, %h2
> movw0x2, ax ; MDAH
> nop ; mdc += mdal * mdah
> + nop ; Additional nop for MAC
>
> mov a, #0x40
> mov !0xf00e8, a ; MDUC
Shouldn't the MOV after the nop add the extra cycle here?
> movwax, %H2
>
