marian

so basically it means the fix will be available in 4.4.1 ? Is it possible to get that branch and test it on my AT80C2051?

Another increase of code (from 8 to 19 instructions) is in the C code dbuf[3] = ledtable[(alarm_minute%10)]; In 3.5 we had: mov b,#0x0A mov a,_alarm_minute div ab mov a,b mov dptr,#_ledtable movc a,@a+dptr mov r7,a mov (_dbuf + 0x0003),r7 In 4.4 we now have mov r6,_alarm_minute mov r7,#0x00 mov __modsint_PARM_2,#0x0a mov (__modsint_PARM_2 + 1),r7 mov dpl, r6 mov dph, r7 lcall __modsint mov r6, dpl mov r7, dph mov a,r6 add a, #_ledtable mov dpl,a mov a,r7 addc a, #(_ledtable >> 8) mov dph,a clr a...

the code generated by newer SDCC is correct Maybe, but the old compiler version generated code works (on AT89C2051), so I wouldn't say that it is incorrect! Anyhow, the "old" code is 1972 bytes, so 76 bytes remains free to use...

Sorry, my mistake! I hope now is OK.

Here it is - I used sdcc -o /tmp/build/ -mmcs51 --opt-code-size --model-small --less-pedantic --fverbose-asm --i-code-in-asm src/main.c. Don't hesitate to as more details if necessary ; the binary has 2735 bytes, too big to entre the 2K of the flash!

Other big differences in code like dbuf[0] = ledtable[(hour/10)]; in 3.5 we have mov b,#0x0A mov a,r6 div ab mov dptr,#_ledtable movc a,@a+dptr mov r7,a mov _dbuf,r7 while in 4.4 we have ~~~ mov r7,#0x00 mov __divsint_PARM_2,#0x0a mov (__divsint_PARM_2 + 1),r7 mov dpl, r6 mov dph, r7 push ar7 push ar6 lcall __divsint mov r4, dpl mov r5, dph pop ar6 pop ar7 mov a,r4 add a, #_ledtable mov dpl,a mov a,r5 addc a, #(_ledtable >> 8) mov dph,a clr a movc a,@a+dptr mov r5,a mov _dbuf,r5 ~~~

Regression in code size in newer versions (> 3..5.0)