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.
Regards,
Kaushik
2013-05-13 Kaushik Phatak <kaushik.pha...@kpitcummins.com>
* config/rl78/rl78.md (mulsi3_g13): Add additional 'nop' required
in multiply-accumulate mode
--- /home/fsfsrc/gcc-4.8.0-20121219/gcc/config/rl78/rl78.md 2013-01-25
16:26:27.000000000 +0530
+++ /home/new/gcc-4.8.0-20121219/gcc/config/rl78/rl78.md 2013-05-13
10:24:19.000000000 +0530
@@ -389,6 +389,7 @@
movw ax, %H2
movw 0xffff2, ax ; MDAH
nop ; mdc += mdal * mdah
+ nop ; Additional nop for MAC
movw ax, !0xf00e0 ; MDCL
movw %H0, ax
; end of mulsi macro"
