added missing "if"s as noted by Hubert Sack (the missing "if"s prohibited removal...
[fw/sdcc] / src / mcs51 / peeph.def
index d4eea55ec05a7a46ac5a259718b5f4dfddf2c003..6327c8b5a7f2e91977f993af6bc308d563e2e39d 100644 (file)
@@ -44,7 +44,7 @@ replace {
        mov     %1,a
 }
 
-replace {
+replace restart {
 // saving 1 byte, loosing 1 cycle but maybe allowing peephole 3.b to start
        mov     %1,#0x00
        mov     %2,#0x00
@@ -286,7 +286,7 @@ replace {
 } by {
        ;       Peephole 107    removed redundant ljmp
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        jc      %1
@@ -402,7 +402,7 @@ replace {
        ;       Peephole 115.b  jump optimization
        jz      %3
 %2:
-} labelRefCountChange(%2 -1)
+} if labelRefCountChange(%2 -1)
 
 replace {
        clr     a
@@ -1063,7 +1063,7 @@ replace {
        ;       Peephole 160.a  removed sjmp by inverse jump logic
        jc      %2
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        jc      %1
@@ -1073,7 +1073,7 @@ replace {
        ;       Peephole 160.b  removed sjmp by inverse jump logic
        jnc     %2
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        jnz     %1
@@ -1083,7 +1083,7 @@ replace {
        ;       Peephole 160.c  removed sjmp by inverse jump logic
        jz      %2
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        jz      %1
@@ -1093,7 +1093,7 @@ replace {
        ;       Peephole 160.d  removed sjmp by inverse jump logic
        jnz     %2
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        jnb     %3,%1
@@ -1103,7 +1103,7 @@ replace {
        ;       Peephole 160.e  removed sjmp by inverse jump logic
        jb      %3,%2
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        jb      %3,%1
@@ -1113,7 +1113,7 @@ replace {
        ;       Peephole 160.f  removed sjmp by inverse jump logic
        jnb     %3,%2
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        mov     %1,%2
@@ -1142,7 +1142,7 @@ replace {
        ;       Peephole 168    jump optimization
        jb      %1,%3
 %2:
-} labelRefCountChange(%2 -1)
+} if labelRefCountChange(%2 -1)
 
 replace {
        jb      %1,%2
@@ -1152,7 +1152,7 @@ replace {
        ;       Peephole 169    jump optimization
        jnb     %1,%3
 %2:
-} labelRefCountChange(%2 -1)
+} if labelRefCountChange(%2 -1)
 
 replace {
        clr     a
@@ -1893,7 +1893,7 @@ replace {
 } by {
        ;       Peephole 200.a  removed redundant sjmp
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        sjmp    %1
@@ -1903,7 +1903,7 @@ replace {
        ;       Peephole 200.b  removed redundant sjmp
 %2:
 %1:
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        push    acc
@@ -2211,6 +2211,18 @@ replace {
        ;       Peephole 225    removed redundant move to acc
 } if notVolatile %1
 
+replace {
+       clr     a
+       movx    @%1,a
+       inc     %1
+       clr     a
+} by {
+       clr     a
+       movx    @%1,a
+       inc     %1
+       ;       Peephole 226.a  removed unnecessary clr
+}
+
 replace {
        clr     a
        movx    @%1,a
@@ -2224,7 +2236,7 @@ replace {
        inc     %1
        movx    @%1,a
        inc     %1
-       ;       Peephole 226    removed unnecessary clr
+       ;       Peephole 226.b  removed unnecessary clr
 }
 
 replace {
@@ -2482,7 +2494,7 @@ replace {
        mov     %3,%4
 %1:
        ret
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 replace {
        sjmp    %1
@@ -2501,7 +2513,7 @@ replace {
        mov     dph,%6
 %1:
        ret
-} labelRefCountChange(%1 -1)
+} if labelRefCountChange(%1 -1)
 
 // applies to f.e. device/lib/log10f.c
 replace {
@@ -2602,133 +2614,180 @@ replace restart {
        addc    a,%1
 }
 
-// peepholes 241.a to 241.c and 241.d to 241.f need to be in order
+// peepholes 241.a to 241.d and 241.e to 241.h need to be in order
 replace {
-       cjne    r%1,#%2,%3
-       cjne    r%4,#%5,%3
-       cjne    r%6,#%7,%3
-       cjne    r%8,#%9,%3
+       cjne    r%2,#%3,%0
+       cjne    r%4,#%5,%0
+       cjne    r%6,#%7,%0
+       cjne    r%8,#%9,%0
        mov     a,#0x01
-       sjmp    %10
-%3:
+       sjmp    %1
+%0:
        clr     a
-%10:
+%1:
 } by {
        ;       Peephole 241.a  optimized compare
        clr     a
-       cjne    r%1,#%2,%3
-       cjne    r%4,#%5,%3
-       cjne    r%6,#%7,%3
-       cjne    r%8,#%9,%3
+       cjne    r%2,#%3,%0
+       cjne    r%4,#%5,%0
+       cjne    r%6,#%7,%0
+       cjne    r%8,#%9,%0
        inc     a
-%3:
-%10:
-} labelRefCountChange(%10 -1)
+%0:
+%1:
+} if labelRefCountChange(%1 -1)
 
-// applies to f.e. time.c
+// applies to generic pointer compare
 replace {
-       cjne    r%1,#%2,%3
-       cjne    r%4,#%5,%3
+       cjne    r%2,#%3,%0
+       cjne    r%4,#%5,%0
+       cjne    r%6,#%7,%0
        mov     a,#0x01
-       sjmp    %6
-%3:
+       sjmp    %1
+%0:
        clr     a
-%6:
+%1:
 } by {
        ;       Peephole 241.b  optimized compare
        clr     a
-       cjne    r%1,#%2,%3
-       cjne    r%4,#%5,%3
+       cjne    r%2,#%3,%0
+       cjne    r%4,#%5,%0
+       cjne    r%6,#%7,%0
        inc     a
-%3:
-%6:
-} labelRefCountChange(%6 -1)
+%0:
+%1:
+} if labelRefCountChange(%1 -1)
 
-// applies to f.e. malloc.c
+// applies to f.e. time.c
 replace {
-       cjne    r%1,#%2,%3
+       cjne    r%2,#%3,%0
+       cjne    r%4,#%5,%0
        mov     a,#0x01
-       sjmp    %4
-%3:
+       sjmp    %1
+%0:
        clr     a
-%4:
+%1:
 } by {
        ;       Peephole 241.c  optimized compare
        clr     a
-       cjne    r%1,#%2,%3
+       cjne    r%2,#%3,%0
+       cjne    r%4,#%5,%0
        inc     a
-%3:
-%4:
-} labelRefCountChange(%4 -1)
+%0:
+%1:
+} if labelRefCountChange(%1 -1)
+
+// applies to f.e. malloc.c
+replace {
+       cjne    r%2,#%3,%0
+       mov     a,#0x01
+       sjmp    %1
+%0:
+       clr     a
+%1:
+} by {
+       ;       Peephole 241.d  optimized compare
+       clr     a
+       cjne    r%2,#%3,%0
+       inc     a
+%0:
+%1:
+} if labelRefCountChange(%1 -1)
 
 // applies to f.e. j = (k!=0x1000);
 // with volatile idata long k;
 replace {
-       cjne    @r%1,#%2,%3
-       inc     r%1
-       cjne    @r%1,#%4,%3
-       inc     r%1
-       cjne    @r%1,#%5,%3
-       inc     r%1
-       cjne    @r%1,#%6,%3
+       cjne    @r%0,#%3,%1
+       inc     r%0
+       cjne    @r%0,#%4,%1
+       inc     r%0
+       cjne    @r%0,#%5,%1
+       inc     r%0
+       cjne    @r%0,#%6,%1
        mov     a,#0x01
-       sjmp    %7
-%3:
+       sjmp    %2
+%1:
        clr     a
-%7:
+%2:
 } by {
-       ;       Peephole 241.d  optimized compare
+       ;       Peephole 241.e  optimized compare
        clr     a
-       cjne    @r%1,#%2,%3
-       inc     r%1
-       cjne    @r%1,#%4,%3
-       inc     r%1
-       cjne    @r%1,#%5,%3
-       inc     r%1
-       cjne    @r%1,#%6,%3
+       cjne    @r%0,#%3,%1
+       inc     r%0
+       cjne    @r%0,#%4,%1
+       inc     r%0
+       cjne    @r%0,#%5,%1
+       inc     r%0
+       cjne    @r%0,#%6,%1
        inc     a
-%3:
-%7:
-} labelRefCountChange(%7 -1)
+%1:
+%2:
+} if labelRefCountChange(%2 -1)
+
+// applies to f.e. j = (p!=NULL);
+// with volatile idata char *p;
+replace {
+       cjne    @r%0,#%3,%1
+       inc     r%0
+       cjne    @r%0,#%4,%1
+       inc     r%0
+       cjne    @r%0,#%5,%1
+       mov     a,#0x01
+       sjmp    %2
+%1:
+       clr     a
+%2:
+} by {
+       ;       Peephole 241.f  optimized compare
+       clr     a
+       cjne    @r%0,#%3,%1
+       inc     r%0
+       cjne    @r%0,#%4,%1
+       inc     r%0
+       cjne    @r%0,#%5,%1
+       inc     a
+%1:
+%2:
+} if labelRefCountChange(%2 -1)
 
 // applies to f.e. j = (k!=0x1000);
 // with volatile idata int k;
 replace {
-       cjne    @r%1,#%2,%3
-       inc     r%1
-       cjne    @r%1,#%4,%3
+       cjne    @r%0,#%3,%1
+       inc     r%0
+       cjne    @r%0,#%4,%1
        mov     a,#0x01
-       sjmp    %7
-%3:
+       sjmp    %2
+%1:
        clr     a
-%7:
+%2:
 } by {
-       ;       Peephole 241.e  optimized compare
+       ;       Peephole 241.g  optimized compare
        clr     a
-       cjne    @r%1,#%2,%3
-       inc     r%1
-       cjne    @r%1,#%4,%3
+       cjne    @r%0,#%3,%1
+       inc     r%0
+       cjne    @r%0,#%4,%1
        inc     a
-%3:
-%7:
-} labelRefCountChange(%7 -1)
+%1:
+%2:
+} if labelRefCountChange(%2 -1)
 
 // applies to f.e. vprintf.asm (--stack-auto)
 replace {
-       cjne    @r%1,#%2,%3
+       cjne    @r%0,#%3,%1
        mov     a,#0x01
-       sjmp    %7
-%3:
+       sjmp    %2
+%1:
        clr     a
-%7:
+%2:
 } by {
-       ;       Peephole 241.f  optimized compare
+       ;       Peephole 241.h  optimized compare
        clr     a
-       cjne    @r%1,#%2,%3
+       cjne    @r%0,#%3,%1
        inc     a
-%3:
-%7:
-} labelRefCountChange(%7 -1)
+%1:
+%2:
+} if labelRefCountChange(%2 -1)
 
 // applies to f.e. scott-bool1.c
 replace {
@@ -3799,7 +3858,7 @@ replace {
        mov     r%1,a
 %4:
 %3:
-} labelRefCountChange(%3 -1)
+} if labelRefCountChange(%3 -1)
 
 // in_byte>>=1; if(in_bit) in_byte|=0x80;
 replace {
@@ -3839,7 +3898,7 @@ replace {
        mov     r%1,a
 %4:
 %3:
-} labelRefCountChange(%3 -1)
+} if labelRefCountChange(%3 -1)
 
 // out_bit=out_byte&0x80; out_byte<<=1;
 // helps f.e. writing data on a 3-wire (SPI) bus
@@ -3876,16 +3935,16 @@ replace {
        mov     %1,a
 } if notVolatile %1
 
-// Peepholes 259.x rely on the correct labelRefCount. Otherwise they are 
+// Peepholes 259.x rely on the correct labelRefCount. Otherwise they are
 // not compatible with peepholes 250.x
-// Peepholes 250.x add jumps to a previously unused label. If the 
+// Peepholes 250.x add jumps to a previously unused label. If the
 // labelRefCount is not increased, peepholes 259.x are (mistakenly) applied.
 // (Mail on sdcc-devel 2004-10-25)
 //
 // applies to f.e. vprintf.c
 replace {
        sjmp    %1
-%2:    
+%2:
        ret
 } by {
        sjmp    %1
@@ -3896,7 +3955,7 @@ replace {
 // applies to f.e. gets.c
 replace {
        ljmp    %1
-%2:    
+%2:
        ret
 } by {
        ljmp    %1
@@ -4421,6 +4480,40 @@ replace {
        mov     %1,a
 }
 
+replace {
+       cpl     c
+       cpl     c
+} by {
+       ;       Peephole 262    removed redundant cpl c
+}
+
+replace {
+       mov     %1,#%2
+       inc     %1
+       inc     %1
+       inc     %1
+} by {
+       ;       Peephole 263.a  optimized loading const
+       mov     %1,#(%2 + 3)
+}
+
+replace {
+       mov     %1,#%2
+       inc     %1
+       inc     %1
+} by {
+       ;       Peephole 263.b  optimized loading const
+       mov     %1,#(%2 + 2)
+}
+
+replace {
+       mov     %1,#%2
+       inc     %1
+} by {
+       ;       Peephole 263.c  optimized loading const
+       mov     %1,#(%2 + 1)
+}
+
 // should be one of the last peepholes
 replace{
 %1: