From: SourceForge.net <no...@so...> - 2006-05-21 17:46:40
|
Patches item #1492559, was opened at 2006-05-21 19:21 Message generated for change (Comment added) made by maartenbrock You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300599&aid=1492559&group_id=599 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 2 Submitted By: Hubert Sack (hsack) Assigned to: Nobody/Anonymous (nobody) Summary: Additional peephole for mcs51 port Initial Comment: I added one more peephole rule It saves at least 1 cycle and if #00 is assigned, one additional byte. Furthermore some other peepholes (3.x) maybe allowed to start. ---------------------------------------------------------------------- >Comment By: Maarten Brock (maartenbrock) Date: 2006-05-21 19:46 Message: Logged In: YES user_id=888171 This is a dangerous rule you propose, because you assume the accumulator to be free. Consider: mov a,SBUF mov dpl,#0x00 mov dph,#0x00 movx @dptr,a This would go totally wrong: mov a,SBUF mov a,#0x00 mov dpl,a mov dph,a movx @dptr,a ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300599&aid=1492559&group_id=599 |