From: Sandeep D. <sa...@wi...> - 2001-10-27 18:06:56
|
I'll take a look @ this today.. in the meantime please don't commit the temp fixes make it harder to recreate the bugs.. Sandeep > -----Original Message----- > From: sdc...@li... > [mailto:sdc...@li...]On Behalf Of Johan Knol > Sent: Saturday, October 27, 2001 3:07 AM > To: sdc...@li... > Subject: Re: [sdcc-devel] [ sdcc-Bugs-474687 ] Nested loops compile to > evil assembly > > > BTW: This "fix" increases the sloc usage by about 500%, so it > wasn't such a > good idea, although it did fix the bug ;-( > > ----- Original Message ----- > From: Johan Knol <joh...@id...> > To: <sdc...@li...> > Sent: Friday, October 26, 2001 3:29 PM > Subject: Re: [sdcc-devel] [ sdcc-Bugs-474687 ] Nested loops > compile to evil > assembly > > > > Fixed in SDCClrange.c:1.11, although this may not be the > most optimal > > solution. Maybe Sandeep wants to have a second look at it. > > > > Johan > > > > ----- Original Message ----- > > From: <no...@so...> > > To: <no...@so...> > > Sent: Thursday, October 25, 2001 2:32 AM > > Subject: [sdcc-devel] [ sdcc-Bugs-474687 ] Nested loops > compile to evil > > assembly > > > > > > > Bugs item #474687, was opened at 2001-10-24 17:32 > > > You can respond by visiting: > > > > > > http://sourceforge.net/tracker/?func=detail&atid=100599&aid=47 > 4687&group_id= > > 599 > > > > > > Category: C-Front End > > > Group: known bugs > > > Status: Open > > > Resolution: None > > > Priority: 5 > > > Submitted By: Saul Thurrowgood (sault) > > > Assigned to: Nobody/Anonymous (nobody) > > > Summary: Nested loops compile to evil assembly > > > > > > Initial Comment: > > > Although this is a silly piece of C, it does show up a > > > bug with sdcc (also, couldn't most of the code be > > > replaced with djnz?): > > > > > > void main(void) > > > { > > > unsigned char i, j, k, l, m; > > > > > > for (i = 0xff; --i; ) > > > for (j = 0xff; --j; ) > > > for (k = 0xff; --k; ) > > > for (l = 0xff; --l; ) > > > for (m = 0xff; > > > --m; ); > > > } > > > > > > > > > Compiled with SDCC : mcs51 2.3.1 (Oct 23 2001) (UNIX) > > > $ sdcc test.c > > > ---------------------------------------------------------------- > > > > > > _main: > > > ar2 = 0x02 > > > ar3 = 0x03 > > > ar4 = 0x04 > > > ar5 = 0x05 > > > ar6 = 0x06 > > > ar7 = 0x07 > > > ar0 = 0x00 > > > ar1 = 0x01 > > > mov r2,#0xFF > > > 00113$: > > > dec r2 > > > mov a,r2 > > > jz 00116$ > > > 00128$: > > > mov r3,#0xFF > > > 00110$: > > > dec r3 > > > mov a,r3 > > > jz 00113$ > > > 00129$: > > > mov r3,#0xFF > > > 00107$: > > > dec r3 > > > mov a,r3 > > > jz 00110$ > > > 00130$: > > > mov r3,#0xFF > > > 00104$: > > > dec r3 > > > mov a,r3 > > > jz 00107$ > > > 00131$: > > > mov r3,#0xFF > > > 00101$: > > > dec r3 > > > mov a,r3 > > > jz 00104$ > > > 00132$: > > > sjmp 00101$ > > > 00116$: > > > ret > > > > > > > > > > ---------------------------------------------------------------------- > > > > > > You can respond by visiting: > > > > > > http://sourceforge.net/tracker/?func=detail&atid=100599&aid=47 4687&group_id= > 599 > > > > _______________________________________________ > > sdcc-devel mailing list > > sdc...@li... > > https://lists.sourceforge.net/lists/listinfo/sdcc-devel > > > > > _______________________________________________ > sdcc-devel mailing list > sdc...@li... > https://lists.sourceforge.net/lists/listinfo/sdcc-devel > _______________________________________________ sdcc-devel mailing list sdc...@li... https://lists.sourceforge.net/lists/listinfo/sdcc-devel |