X-Git-Url: https://git.gag.com/?a=blobdiff_plain;f=src%2Fpic%2Fpcoderegs.c;h=61ea9a9718d394338c6907e8750759fddfa55429;hb=0892d3692ded8c67ab4df6b26f603bb8d243e39d;hp=5b0b9d92c407394fac5138960dd6c46a01801d8c;hpb=42ac194ab811f6198733c7fd82b07de8da31831f;p=fw%2Fsdcc diff --git a/src/pic/pcoderegs.c b/src/pic/pcoderegs.c index 5b0b9d92..61ea9a97 100644 --- a/src/pic/pcoderegs.c +++ b/src/pic/pcoderegs.c @@ -21,7 +21,7 @@ -------------------------------------------------------------------------*/ /* - pcoderegs.c +pcoderegs.c The purpose of the code in this file is to optimize the register usage. @@ -36,563 +36,821 @@ #include "pcoderegs.h" #include "pcodeflow.h" -extern void pCodeInsertAfter(pCode *pc1, pCode *pc2); extern void dbg_dumpregusage(void); extern pCode * findPrevInstruction(pCode *pci); +extern pBranch * pBranchAppend(pBranch *h, pBranch *n); void unlinkpCode(pCode *pc); - +extern int pCodeSearchCondition(pCode *pc, unsigned int cond, int contIfSkip); +char *pCode2str(char *str, int size, pCode *pc); +void SAFE_snprintf(char **str, size_t *size, const char *format, ...); int total_registers_saved=0; +int register_optimization=1; /*-----------------------------------------------------------------* - * void AddRegToFlow(regs *reg, pCodeFlow *pcfl) - *-----------------------------------------------------------------*/ +* void AddRegToFlow(regs *reg, pCodeFlow *pcfl) +*-----------------------------------------------------------------*/ /* void AddRegToFlow(regs *reg, pCodeFlow *pcfl) { - if(!reg || ! pcfl || !isPCFL(pcflow)) - return; - - if(!pcfl->registers) - pcfl->registers = newSet(); + if(!reg || ! pcfl || !isPCFL(pcflow)) + return; + if(!pcfl->registers) + pcfl->registers = newSet(); + } */ /*-----------------------------------------------------------------* - * - *-----------------------------------------------------------------*/ +* +*-----------------------------------------------------------------*/ void dbg_regusage(set *fregs) { - regs *reg; - pCode *pcfl; - pCode *pc; - - - for (reg = setFirstItem(fregs) ; reg ; - reg = setNextItem(fregs)) { - - if(elementsInSet(reg->reglives.usedpCodes)) { - - fprintf (stderr, "%s addr=0x%03x rIdx=0x%03x", - reg->name, - reg->address, - reg->rIdx); - - pcfl = setFirstItem(reg->reglives.usedpFlows); - if(pcfl) - fprintf(stderr, "\n used in seq"); - - while(pcfl) { - fprintf(stderr," 0x%03x",pcfl->seq); - pcfl = setNextItem(reg->reglives.usedpFlows); - } - - pcfl = setFirstItem(reg->reglives.assignedpFlows); - if(pcfl) - fprintf(stderr, "\n assigned in seq"); - - while(pcfl) { - fprintf(stderr," 0x%03x",pcfl->seq); - pcfl = setNextItem(reg->reglives.assignedpFlows); - } - - pc = setFirstItem(reg->reglives.usedpCodes); - if(pc) - fprintf(stderr, "\n used in instructions "); - - while(pc) { - pcfl = PCODE(PCI(pc)->pcflow); - if(pcfl) - fprintf(stderr," 0x%03x:",pcfl->seq); - fprintf(stderr,"0x%03x",pc->seq); - - pc = setNextItem(reg->reglives.usedpCodes); - } - - fprintf(stderr, "\n"); - } - } + regs *reg; + pCode *pcfl; + pCode *pc; + + + for (reg = setFirstItem(fregs) ; reg ; + reg = setNextItem(fregs)) { + + if(elementsInSet(reg->reglives.usedpCodes)) { + + fprintf (stderr, "%s addr=0x%03x rIdx=0x%03x", + reg->name, + reg->address, + reg->rIdx); + + pcfl = setFirstItem(reg->reglives.usedpFlows); + if(pcfl) + fprintf(stderr, "\n used in seq"); + + while(pcfl) { + fprintf(stderr," 0x%03x",pcfl->seq); + pcfl = setNextItem(reg->reglives.usedpFlows); + } + + pcfl = setFirstItem(reg->reglives.assignedpFlows); + if(pcfl) + fprintf(stderr, "\n assigned in seq"); + + while(pcfl) { + fprintf(stderr," 0x%03x",pcfl->seq); + pcfl = setNextItem(reg->reglives.assignedpFlows); + } + + pc = setFirstItem(reg->reglives.usedpCodes); + if(pc) + fprintf(stderr, "\n used in instructions "); + + while(pc) { + pcfl = PCODE(PCI(pc)->pcflow); + if(pcfl) + fprintf(stderr," 0x%03x:",pcfl->seq); + fprintf(stderr,"0x%03x",pc->seq); + + pc = setNextItem(reg->reglives.usedpCodes); + } + + fprintf(stderr, "\n"); + } + } } /*-----------------------------------------------------------------* - * - *-----------------------------------------------------------------*/ +* +*-----------------------------------------------------------------*/ void dbg_dumpregusage(void) { - - fprintf(stderr,"*** Register Usage ***\n"); - fprintf(stderr,"InternalRegs:\n"); - dbg_regusage(dynInternalRegs); - fprintf(stderr,"AllocRegs:\n"); - dbg_regusage(dynAllocRegs); - fprintf(stderr,"StackRegs:\n"); - dbg_regusage(dynStackRegs); - fprintf(stderr,"DirectRegs:\n"); - dbg_regusage(dynDirectRegs); - fprintf(stderr,"DirectBitRegs:\n"); - dbg_regusage(dynDirectBitRegs); - fprintf(stderr,"ProcessorRegs:\n"); - dbg_regusage(dynProcessorRegs); - + + fprintf(stderr,"*** Register Usage ***\n"); + fprintf(stderr,"InternalRegs:\n"); + dbg_regusage(dynInternalRegs); + fprintf(stderr,"AllocRegs:\n"); + dbg_regusage(dynAllocRegs); + fprintf(stderr,"StackRegs:\n"); + dbg_regusage(dynStackRegs); + fprintf(stderr,"DirectRegs:\n"); + dbg_regusage(dynDirectRegs); + fprintf(stderr,"DirectBitRegs:\n"); + dbg_regusage(dynDirectBitRegs); + fprintf(stderr,"ProcessorRegs:\n"); + dbg_regusage(dynProcessorRegs); + } /*-----------------------------------------------------------------* - * void pCodeRegMapLiveRangesInFlow(pCodeFlow *pcfl) - *-----------------------------------------------------------------*/ +* void pCodeRegMapLiveRangesInFlow(pCodeFlow *pcfl) +*-----------------------------------------------------------------*/ void pCodeRegMapLiveRangesInFlow(pCodeFlow *pcfl) { - - pCode *pc=NULL; - pCode *pcprev=NULL; - - regs *reg; - - if(!pcfl) - return; - - - pc = findNextInstruction(pcfl->pc.next); - - while(isPCinFlow(pc,PCODE(pcfl))) { - - - reg = getRegFromInstruction(pc); - - if(reg) { -/* - fprintf(stderr, "flow seq %d, inst seq %d %s ",PCODE(pcfl)->seq,pc->seq,reg->name); - fprintf(stderr, "addr = 0x%03x, type = %d rIdx=0x%03x\n", - reg->address,reg->type,reg->rIdx); -*/ - - addSetIfnotP(& (PCFL(pcfl)->registers), reg); - - if((PCC_REGISTER | PCC_LITERAL) & PCI(pc)->inCond) - addSetIfnotP(& (reg->reglives.usedpFlows), pcfl); - - if(PCC_REGISTER & PCI(pc)->outCond) - addSetIfnotP(& (reg->reglives.assignedpFlows), pcfl); - - addSetIfnotP(& (reg->reglives.usedpCodes), pc); - } - - - pcprev = pc; - pc = findNextInstruction(pc->next); - - } - + + pCode *pc=NULL; + pCode *pcprev=NULL; + + regs *reg; + + if(!pcfl) + return; + + + pc = findNextInstruction(pcfl->pc.next); + + while(isPCinFlow(pc,PCODE(pcfl))) { + + + reg = getRegFromInstruction(pc); + + if(reg) { + /* + fprintf(stderr, "flow seq %d, inst seq %d %s ",PCODE(pcfl)->seq,pc->seq,reg->name); + fprintf(stderr, "addr = 0x%03x, type = %d rIdx=0x%03x\n", + reg->address,reg->type,reg->rIdx); + */ + + addSetIfnotP(& (PCFL(pcfl)->registers), reg); + + if((PCC_REGISTER | PCC_LITERAL) & PCI(pc)->inCond) + addSetIfnotP(& (reg->reglives.usedpFlows), pcfl); + + if(PCC_REGISTER & PCI(pc)->outCond) + addSetIfnotP(& (reg->reglives.assignedpFlows), pcfl); + + addSetIfnotP(& (reg->reglives.usedpCodes), pc); + } + + + pcprev = pc; + pc = findNextInstruction(pc->next); + + } + } /*-----------------------------------------------------------------* - * void pCodeRegMapLiveRanges(pBlock *pb) - *-----------------------------------------------------------------*/ +* void pCodeRegMapLiveRanges(pBlock *pb) +*-----------------------------------------------------------------*/ void pCodeRegMapLiveRanges(pBlock *pb) { - pCode *pcflow; - - for( pcflow = findNextpCode(pb->pcHead, PC_FLOW); - pcflow != NULL; - pcflow = findNextpCode(pcflow->next, PC_FLOW) ) { - - if(!isPCFL(pcflow)) { - fprintf(stderr, "pCodeRegMapLiveRanges - pcflow is not a flow object "); - continue; - } - pCodeRegMapLiveRangesInFlow(PCFL(pcflow)); - } - - - for( pcflow = findNextpCode(pb->pcHead, PC_FLOW); - pcflow != NULL; - pcflow = findNextpCode(pcflow->next, PC_FLOW) ) { - - regs *r = setFirstItem(PCFL(pcflow)->registers); - //fprintf(stderr,"flow seq %d\n", pcflow->seq); - while (r) { - //fprintf(stderr, " %s\n",r->name); - r = setNextItem(PCFL(pcflow)->registers); - - } - - } + pCode *pcflow; + + for( pcflow = findNextpCode(pb->pcHead, PC_FLOW); + pcflow != NULL; + pcflow = findNextpCode(pcflow->next, PC_FLOW) ) { + + if(!isPCFL(pcflow)) { + fprintf(stderr, "pCodeRegMapLiveRanges - pcflow is not a flow object "); + continue; + } + pCodeRegMapLiveRangesInFlow(PCFL(pcflow)); + } + +#if 0 + for( pcflow = findNextpCode(pb->pcHead, PC_FLOW); + pcflow != NULL; + pcflow = findNextpCode(pcflow->next, PC_FLOW) ) { + + regs *r = setFirstItem(PCFL(pcflow)->registers); + fprintf(stderr,"flow seq %d\n", pcflow->seq); + + while (r) { + fprintf(stderr, " %s\n",r->name); + r = setNextItem(PCFL(pcflow)->registers); + + } + + } +#endif + + // dbg_dumpregusage(); + +} -// dbg_dumpregusage(); +/*-----------------------------------------------------------------* +* +*-----------------------------------------------------------------*/ +static void Remove1pcode(pCode *pc, regs *reg, int debug_code) +{ + + pCode *pcn=NULL; + + if(!reg || !pc) + return; + + deleteSetItem (&(reg->reglives.usedpCodes),pc); + + if(PCI(pc)->label) { + pcn = findNextInstruction(pc->next); + + if(pcn) + PCI(pcn)->label = pBranchAppend(PCI(pcn)->label,PCI(pc)->label); + } + + if(PCI(pc)->cline) { + if(!pcn) + pcn = findNextInstruction(pc->next); + + if(pcn) { + if(PCI(pcn)->cline) { + //fprintf(stderr, "source line has been optimized completely out\n"); + //pc->print(stderr,pc); + } else { + PCI(pcn)->cline = PCI(pc)->cline; + } + } + } + + + if(1){ + /* + Debug stuff. Comment out the instruction we're about to delete. + */ + + char buff1[256]; + size_t size = 256; + + char *pbuff,**ppbuff; + pbuff = buff1; + ppbuff = &pbuff; + + SAFE_snprintf(ppbuff,&size, ";%d", debug_code); + pCode2str(*ppbuff, size, pc); + pCodeInsertBefore(pc, newpCodeCharP(buff1)); + //fprintf(stderr,"removing instruction:\n%s\n",buff1); + } + + pc->destruct(pc); + } - /*-----------------------------------------------------------------* - * void RemoveRegsFromSet(set *regset) - * - *-----------------------------------------------------------------*/ +* void RemoveRegsFromSet(set *regset) +* +*-----------------------------------------------------------------*/ void RemoveRegsFromSet(set *regset) { - regs *reg; - int used; - for (reg = setFirstItem(regset) ; reg ; - reg = setNextItem(regset)) { - - - used = elementsInSet(reg->reglives.usedpCodes); - - if(used <= 1) { - - //fprintf(stderr," reg %s isfree=%d, wasused=%d\n",reg->name,reg->isFree,reg->wasUsed); - - if(used == 0) { - //fprintf(stderr," getting rid of reg %s\n",reg->name); - reg->isFree = 1; - reg->wasUsed = 0; - } else { - pCode *pc; - - - pc = setFirstItem(reg->reglives.usedpCodes); - if(isPCI(pc)) { - if(PCI(pc)->label) { - pCode *pcn = findNextInstruction(pc->next); - - if(pcn && PCI(pcn)->label) { - //fprintf(stderr,"can't delete instruction with label...\n"); - //pc->print(stderr,pc); - continue; - } - /* Move the label to the next instruction */ - - PCI(pcn)->label = PCI(pc)->label; - - } - - if(isPCI_SKIP(pc)) - fprintf(stderr, "WARNING, a skip instruction is being optimized out\n"); - - //fprintf(stderr," removing reg %s because it is used only once\n",reg->name); - unlinkpCode(pc); - deleteSetItem (&(reg->reglives.usedpCodes),pc); - reg->isFree = 1; - reg->wasUsed = 0; - total_registers_saved++; // debugging stats. + regs *reg; + int used; + + while(regset) { + reg = regset->item; + regset = regset->next; + + used = elementsInSet(reg->reglives.usedpCodes); + + if(used <= 1) { + + //fprintf(stderr," reg %s isfree=%d, wasused=%d\n",reg->name,reg->isFree,reg->wasUsed); + if(used == 0) { + //fprintf(stderr," getting rid of reg %s\n",reg->name); + reg->isFree = 1; + reg->wasUsed = 0; + } else { + pCode *pc; + + + pc = setFirstItem(reg->reglives.usedpCodes); + + if(reg->type == REG_SFR || reg->type == REG_STK || reg->isPublic || reg->isExtern) { + //fprintf(stderr, "not removing SFR reg %s even though used only once\n",reg->name); + continue; + } + + + if(isPCI(pc)) { + if(PCI(pc)->label) { + pCode *pcn = findNextInstruction(pc->next); + + if(pcn && PCI(pcn)->label) { + //fprintf(stderr,"can't delete instruction with label...\n"); + //pc->print(stderr,pc); + continue; + } + /* Move the label to the next instruction */ + + PCI(pcn)->label = PCI(pc)->label; + + } + + if(isPCI_SKIP(pc)) { + regs *r = getRegFromInstruction(pc); + fprintf(stderr, "WARNING, a skip instruction is being optimized out\n"); + pc->print(stderr,pc); + fprintf(stderr,"reg %s, type =%d\n",r->name, r->type); + } + //fprintf(stderr," removing reg %s because it is used only once\n",reg->name); + Remove1pcode(pc, reg, 1); + /* + unlinkpCode(pc); + deleteSetItem (&(reg->reglives.usedpCodes),pc); + */ + reg->isFree = 1; + reg->wasUsed = 0; + total_registers_saved++; // debugging stats. + } + } + } + } - } - } - - } } /*-----------------------------------------------------------------* - * void RemoveUnusedRegisters(void) - * - *-----------------------------------------------------------------*/ +* void RemoveUnusedRegisters(void) +* +*-----------------------------------------------------------------*/ void RemoveUnusedRegisters(void) { - /* First, get rid of registers that are used only one time */ - - - RemoveRegsFromSet(dynInternalRegs); - RemoveRegsFromSet(dynAllocRegs); - RemoveRegsFromSet(dynStackRegs); - /* - don't do DirectRegs yet - there's a problem with arrays - RemoveRegsFromSet(dynDirectRegs); - */ - RemoveRegsFromSet(dynDirectBitRegs); - - if(total_registers_saved) fprintf(stderr, " *** Saved %d registers ***\n", total_registers_saved); + /* First, get rid of registers that are used only one time */ + + //RemoveRegsFromSet(dynInternalRegs); + RemoveRegsFromSet(dynAllocRegs); + RemoveRegsFromSet(dynStackRegs); + /* + don't do DirectRegs yet - there's a problem with arrays + RemoveRegsFromSet(dynDirectRegs); + */ + RemoveRegsFromSet(dynDirectBitRegs); + + if(total_registers_saved) DFPRINTF((stderr, " *** Saved %d registers ***\n", total_registers_saved)); } /*-----------------------------------------------------------------* - * - *-----------------------------------------------------------------*/ -static void Remove2pcodes(pCode *pcflow, pCode *pc1, pCode *pc2, regs *reg) +* +*-----------------------------------------------------------------*/ +static void Remove2pcodes(pCode *pcflow, pCode *pc1, pCode *pc2, regs *reg, int can_free) { - if(!reg) - return; - - if(pc1) { - deleteSetItem (&(reg->reglives.usedpCodes),pc1); - pc1->destruct(pc1); - } - - if(pc2) { - deleteSetItem (&(reg->reglives.usedpCodes),pc2); - pc2->destruct(pc2); - - deleteSetItem (&(PCFL(pcflow)->registers), reg); - - reg->isFree = 1; - reg->wasUsed = 0; - - } - - pCodeRegMapLiveRangesInFlow(PCFL(pcflow)); + static int debug_code=99; + if(!reg) + return; + + //fprintf(stderr,"%s\n",__FUNCTION__); + if(pc1) + Remove1pcode(pc1, reg, debug_code++); + + if(pc2) { + Remove1pcode(pc2, reg, debug_code++); + deleteSetItem (&(PCFL(pcflow)->registers), reg); + + if(can_free) { + reg->isFree = 1; + reg->wasUsed = 0; + } + + } + + pCodeRegMapLiveRangesInFlow(PCFL(pcflow)); } /*-----------------------------------------------------------------* - * - *-----------------------------------------------------------------*/ +* +*-----------------------------------------------------------------*/ int regUsedinRange(pCode *pc1, pCode *pc2, regs *reg) { - int i=0; - regs *testreg; - - do { - testreg = getRegFromInstruction(pc1); - if(testreg && (testreg->rIdx == reg->rIdx)) { - return 1; - } - - pc1 = findNextInstruction(pc1->next); - - } while (pc1 && (pc1 != pc2) && (i++ < 100)) ; - - if(i >= 100) - fprintf(stderr, "warning, regUsedinRange searched through too many pcodes\n"); - - return 0; + int i=0; + regs *testreg; + + do { + testreg = getRegFromInstruction(pc1); + if(testreg && (testreg->rIdx == reg->rIdx)) { + return 1; + } + + pc1 = findNextInstruction(pc1->next); + + } while (pc1 && (pc1 != pc2) && (i++ < 100)) ; + + if(i >= 100) + fprintf(stderr, "warning, regUsedinRange searched through too many pcodes\n"); + + return 0; } /*-----------------------------------------------------------------* - * void pCodeRegOptimeRegUsage(pBlock *pb) - *-----------------------------------------------------------------*/ -void OptimizeRegUsage(set *fregs) +* void pCodeOptime2pCodes(pCode *pc1, pCode *pc2) +* +* ADHOC pattern checking +* Now look for specific sequences that are easy to optimize. +* Many of these sequences are characteristic of the compiler +* (i.e. it'd probably be a waste of time to apply these adhoc +* checks to hand written assembly.) +* +* +*-----------------------------------------------------------------*/ +int pCodeOptime2pCodes(pCode *pc1, pCode *pc2, pCode *pcfl_used, regs *reg, int can_free, int optimize_level) { - regs *reg; - int used; - - - while(fregs) { - pCode *pcfl_used, *pcfl_assigned; - - /* Step through the set by directly accessing the 'next' pointer. - * We could also step through by using the set API, but the - * the (debug) calls to print instructions affect the state - * of the set pointers */ - - reg = fregs->item; - fregs = fregs->next; - - - pcfl_used = setFirstItem(reg->reglives.usedpFlows); - pcfl_assigned = setFirstItem(reg->reglives.assignedpFlows); - - used = elementsInSet(reg->reglives.usedpCodes); - if(used == 2) { - - /* - * In this section, all registers that are used in only in two - * instructions are examined. If possible, they're optimized out. - */ - - pCode *pc1, *pc2; - pCode *pct1, *pct2; /* two temporaries */ - regs *reg1, *reg2; - -/* - fprintf (stderr, "OptimizeRegUsage: %s addr=0x%03x rIdx=0x%03x used=%d\n", - reg->name, - reg->address, - reg->rIdx, used); -*/ - pc1 = setFirstItem(reg->reglives.usedpCodes); - pc2 = setNextItem(reg->reglives.usedpCodes); - - if(pcfl_used && pcfl_assigned) { - - /* - expected case - the register has been assigned a value and is - subsequently used - */ - - //fprintf(stderr," used only twice\n"); - if(pcfl_used->seq == pcfl_assigned->seq) { - - //fprintf(stderr, " and used in same flow\n"); - if(pc2->seq < pc1->seq) { - pct1 = pc2; - pc2 = pc1; - pc1 = pct1; - } - - //pc1->print(stderr,pc1); - //pc2->print(stderr,pc2); - - - /* ADHOC pattern checking - * Now look for specific sequences that are easy to optimize. - * Many of these sequences are characteristic of the compiler - * (i.e. it'd probably be a waste of time to apply these adhoc - * checks to hand written assembly.) - * - */ - if((PCI(pc1)->op == POC_CLRF) && (PCI(pc2)->op == POC_MOVFW) ){ - - /* - clrf reg - stuff... - movf reg,w - - can be replaced with - - stuff... - movlw 0 - */ - - pCode *newpc; - //fprintf(stderr, " CLRF/MOVFW. instruction after MOVFW is:\n"); - pct1 = findNextInstruction(pc2->next); - //t->print(stderr,t); - - if(PCI(pct1)->op == POC_MOVWF) { - newpc = newpCode(POC_CLRF, PCI(pct1)->pcop); - pct1->destruct(pct1); - } else { - newpc = newpCode(POC_MOVLW, newpCodeOpLit(0)); - } - - pCodeInsertAfter(pc2, newpc); - PCI(newpc)->pcflow = PCFL(pcfl_used); - newpc->seq = pc2->seq; - - Remove2pcodes(pcfl_used, pc1, pc2, reg); - total_registers_saved++; // debugging stats. - - } else if((PCI(pc1)->op == POC_CLRF) && (PCI(pc2)->op == POC_IORFW) ){ - //fprintf(stderr, " CLRF/IORWF.\n"); - - Remove2pcodes(pcfl_used, pc1, pc2, reg); - total_registers_saved++; // debugging stats. - - } else if(PCI(pc1)->op == POC_MOVWF) { - - pct2 = findNextInstruction(pc2->next); - - if(PCI(pc2)->op == POC_MOVFW) { - //fprintf(stderr, " MOVWF/MOVFW. instruction after MOVFW is:\n"); - // t->print(stderr,t); - - if(PCI(pct2)->op == POC_MOVWF) { - reg2 = getRegFromInstruction(pct2); - if(reg2 && !regUsedinRange(pc1,pc2,reg2)) { - pct2->seq = pc1->seq; - unlinkpCode(pct2); - pCodeInsertAfter(pc1,pct2); - Remove2pcodes(pcfl_used, pc1, pc2, reg); - total_registers_saved++; // debugging stats. - continue; - } - } - } - - pct1 = findPrevInstruction(pc1->prev); - if(pct1 && - (PCI(pct1)->pcflow == PCI(pc1)->pcflow) && - (PCI(pct1)->op == POC_MOVFW)) { - - reg1 = getRegFromInstruction(pct1); - if(reg1 && !regUsedinRange(pc1,pc2,reg1)) { - /* - movf reg1,w - movwf reg - + pCode *pct1, *pct2; + regs *reg1, *reg2; + + int t = total_registers_saved; + + if(pc2->seq < pc1->seq) { + pct1 = pc2; + pc2 = pc1; + pc1 = pct1; + } + + //fprintf(stderr,"pCodeOptime2pCodes\n"); + //pc1->print(stderr,pc1); + //pc2->print(stderr,pc2); + + if((PCI(pc1)->op == POC_CLRF) && (PCI(pc2)->op == POC_MOVFW) ){ + pCode *newpc; + int regUsed = 0; + int wUsed = 0; + int wSaved = 0; + /* + clrf reg ; pc1 stuff... - opcode reg,w - */ - pct2 = newpCode(PCI(pc2)->op, PCI(pct1)->pcop); - pCodeInsertAfter(pc2, pct2); - PCI(pct2)->pcflow = PCFL(pcfl_used); - pct2->seq = pc2->seq; - - Remove2pcodes(pcfl_used, pc1, pc2, reg); - Remove2pcodes(pcfl_used, pct1, NULL, reg1); + movf reg,w ; pc2 + + can be replaced with (only if following instructions are not going to use W and reg is not used again later) + + stuff... + movlw 0 or clrf reg + */ + DFPRINTF((stderr, " optimising CLRF reg ... MOVF reg,W to ... MOVLW 0\n")); + pct2 = findNextInstruction(pc2->next); + + if(pct2 && PCI(pct2)->op == POC_MOVWF) { + wSaved = wUsed = 1; /* Maybe able to replace with clrf pc2->next->reg. */ + } else { + wUsed = pCodeSearchCondition(pct2,PCC_W,1) > 0; + } + regUsed = regUsedinRange(pct2,0,reg); + if ((regUsed&&wUsed) || (pCodeSearchCondition(pct2,PCC_Z,0) > 1)) { + /* Do not optimise as exisiting code is required. */ + } else { + /* Can optimise. */ + if(regUsed) { + newpc = newpCode(POC_CLRF, PCI(pc1)->pcop); + } else if(wSaved && !wUsed) { + newpc = newpCode(POC_CLRF, PCI(pct2)->pcop); + pct2->destruct(pct2); + } else { + newpc = newpCode(POC_MOVLW, newpCodeOpLit(0)); + } + + pCodeInsertAfter(pc2, newpc); + PCI(newpc)->pcflow = PCFL(pcfl_used); + newpc->seq = pc2->seq; + + Remove2pcodes(pcfl_used, pc1, pc2, reg, can_free); + total_registers_saved++; // debugging stats. + } + } else if((PCI(pc1)->op == POC_CLRF) && (PCI(pc2)->op == POC_IORFW) ){ + DFPRINTF((stderr, " optimising CLRF/IORFW\n")); + + pct2 = findNextInstruction(pc2->next); + + if(pCodeSearchCondition(pct2, PCC_Z,0) > 0) { + pct2 = newpCode(POC_IORLW, newpCodeOpLit(0)); + pct2->seq = pc2->seq; + PCI(pct2)->pcflow = PCFL(pcfl_used); + pCodeInsertAfter(pc1,pct2); + } + Remove2pcodes(pcfl_used, pc1, pc2, reg, can_free); total_registers_saved++; // debugging stats. - - } - } - - - } - - } else { - // fprintf(stderr, " and used in different flows\n"); - - //pc1->print(stderr,pc1); - //pc2->print(stderr,pc2); - } - - } else if(pcfl_used) { - - /* - register has been used twice without ever being assigned */ - fprintf(stderr,"WARNING %s: reg %s used without being assigned\n",__FUNCTION__,reg->name); - - } else { - //fprintf(stderr,"WARNING %s: reg %s assigned without being used\n",__FUNCTION__,reg->name); - Remove2pcodes(pcfl_assigned, pc1, pc2, reg); - total_registers_saved++; // debugging stats. - } - } else { - - if(used && !pcfl_used && pcfl_assigned) { - pCode *pc; - - //fprintf(stderr,"WARNING %s: reg %s assigned without being used\n",__FUNCTION__,reg->name); - - pc = setFirstItem(reg->reglives.usedpCodes); - while(pc) { - pcfl_assigned = PCI(pc)->pcflow; - deleteSetItem (&(PCFL(pcfl_assigned)->registers), reg); - deleteSetItem (&(reg->reglives.usedpCodes),pc); - pc->destruct(pc); - - pc = setNextItem(reg->reglives.usedpCodes); - } - - - reg->isFree = 1; - reg->wasUsed = 0; - - total_registers_saved++; // debugging stats. - } - - } - + + } else if(PCI(pc1)->op == POC_MOVWF) { + // Optimising MOVWF reg ... + + pct2 = findNextInstruction(pc2->next); + + if(PCI(pc2)->op == POC_MOVFW) { + // Optimising MOVWF reg ... MOVF reg,W + + if(PCI(pct2)->op == POC_MOVWF) { + /* + Change: + + movwf reg ; pc1 + stuff... + movf reg,w ; pc2 + movwf reg2 ; pct2 + + To: ( as long as 'stuff' does not use reg2 or if following instructions do not use W or reg is not used later) + + movwf reg2 + stuff... + + */ + reg2 = getRegFromInstruction(pct2); + /* Check reg2 is not used for something else before it is loaded with reg */ + if (reg2 && !regUsedinRange(pc1,pc2,reg2)) { + pCode *pct3 = findNextInstruction(pct2->next); + /* Check following instructions are not relying on the use of W or the Z flag condiction */ + if ((pCodeSearchCondition(pct3,PCC_Z,0) < 1) || (pCodeSearchCondition(pct3,PCC_W,0) < 1)) { + DFPRINTF((stderr, " optimising MOVF reg ... MOVF reg,W MOVWF reg2 to MOVWF reg2 ...\n")); + pct2->seq = pc1->seq; + unlinkpCode(pct2); + pCodeInsertBefore(pc1,pct2); + if(regUsedinRange(pct2,0,reg)) + Remove2pcodes(pcfl_used, pc2, NULL, reg, can_free); + else + Remove2pcodes(pcfl_used, pc1, pc2, reg, can_free); + + total_registers_saved++; // debugging stats. + return 1; + } + } + } + } + + pct1 = findPrevInstruction(pc1->prev); + if(pct1 && (PCI(pct1)->pcflow == PCI(pc1)->pcflow)) { + + if ( (PCI(pct1)->op == POC_MOVFW) && + (PCI(pc2)->op == POC_MOVFW)) { + + reg1 = getRegFromInstruction(pct1); + if(reg1 && !regUsedinRange(pc1,pc2,reg1)) { + DFPRINTF((stderr, " optimising MOVF reg1,W MOVWF reg ... MOVF reg,W\n")); + /* + Change: + + movf reg1,w + movwf reg + + stuff... + movf reg,w + + To: + + stuff... + + movf reg1,w + + Or, if we're not deleting the register then the "To" is: + + stuff... + + movf reg1,w + movwf reg + */ + pct2 = newpCode(PCI(pc2)->op, PCI(pct1)->pcop); + pCodeInsertAfter(pc2, pct2); + PCI(pct2)->pcflow = PCFL(pcfl_used); + pct2->seq = pc2->seq; + + if(can_free) { + Remove2pcodes(pcfl_used, pc1, pc2, reg, can_free); + } else { + /* If we're not freeing the register then that means (probably) + * the register is needed somewhere else.*/ + unlinkpCode(pc1); + pCodeInsertAfter(pct2, pc1); + + Remove2pcodes(pcfl_used, pc2, NULL, reg, can_free); + } + + Remove2pcodes(pcfl_used, pct1, NULL, reg1, 0); + total_registers_saved++; // debugging stats. + + } + /* Note this else if is same conditional as the if - DELETE ME + } else if ( (PCI(pct1)->op == POC_MOVWF) && + (PCI(pc2)->op == POC_MOVFW)) { + DFPRINTF(stderr, " optimising MOVF reg1,W MOVWF reg ... MOVF reg,W\n"); + if(optimize_level > 1 && can_free) { + pct2 = newpCode(POC_MOVFW, PCI(pc1)->pcop); + pCodeInsertAfter(pc2, pct2); + Remove2pcodes(pcfl_used, pc1, pc2, reg, 1); + total_registers_saved++; // debugging stats. + } + */ + } + } } + return (total_registers_saved != t); } + /*-----------------------------------------------------------------* - * void pCodeRegOptimeRegUsage(pBlock *pb) - *-----------------------------------------------------------------*/ -void pCodeRegOptimizeRegUsage(void) +* void pCodeRegOptimeRegUsage(pBlock *pb) +*-----------------------------------------------------------------*/ +void OptimizeRegUsage(set *fregs, int optimize_multi_uses, int optimize_level) { + regs *reg; + int used; + pCode *pc1=NULL, *pc2=NULL; + + + while(fregs) { + pCode *pcfl_used, *pcfl_assigned; + + /* Step through the set by directly accessing the 'next' pointer. + * We could also step through by using the set API, but the + * the (debug) calls to print instructions affect the state + * of the set pointers */ + + reg = fregs->item; + fregs = fregs->next; + /* + if (strcmp(reg->name,"_SubState")==0) + fprintf(stderr,"Reg: %s\n",reg->name); + */ + + if(reg->type == REG_SFR || reg->type == REG_STK || reg->isPublic || reg->isExtern|| reg->isFixed) { + //fprintf(stderr,"skipping SFR: %s\n",reg->name); + continue; + } + + pcfl_used = setFirstItem(reg->reglives.usedpFlows); + pcfl_assigned = setFirstItem(reg->reglives.assignedpFlows); + + used = elementsInSet(reg->reglives.usedpCodes); + if(used == 2) { + /* + In this section, all registers that are used in only in two + instructions are examined. If possible, they're optimized out. + */ + + /* + fprintf (stderr, "OptimizeRegUsage: %s addr=0x%03x rIdx=0x%03x type=%d used=%d\n", + reg->name, + reg->address, + reg->rIdx, reg->type, used); + */ + pc1 = setFirstItem(reg->reglives.usedpCodes); + pc2 = setNextItem(reg->reglives.usedpCodes); + + if(pcfl_used && pcfl_assigned) { + /* + expected case - the register has been assigned a value and is + subsequently used + */ + + //fprintf(stderr," used only twice\n"); + if(pcfl_used->seq == pcfl_assigned->seq) { + + //fprintf(stderr, " and used in same flow\n"); + + pCodeOptime2pCodes(pc1, pc2, pcfl_used, reg, 1,optimize_level); + + } else { + // fprintf(stderr, " and used in different flows\n"); + + } + + } else if(pcfl_used) { + + /* register has been used twice without ever being assigned */ + fprintf(stderr,"WARNING %s: reg %s used without being assigned\n",__FUNCTION__,reg->name); + + } else { + fprintf(stderr,"WARNING %s: reg %s assigned without being used\n",__FUNCTION__,reg->name); + Remove2pcodes(pcfl_assigned, pc1, pc2, reg, 1); + total_registers_saved++; // debugging stats. + } + } else { + + /* register has been used either once, or more than twice */ + + if(used && !pcfl_used && pcfl_assigned) { + pCode *pc; + + fprintf(stderr,"WARNING %s: reg %s assigned without being used\n",__FUNCTION__,reg->name); + + pc = setFirstItem(reg->reglives.usedpCodes); + while(pc) { + + pcfl_assigned = PCODE(PCI(pc)->pcflow); + Remove1pcode(pc, reg,2); + + deleteSetItem (&(PCFL(pcfl_assigned)->registers), reg); + /* + deleteSetItem (&(reg->reglives.usedpCodes),pc); + pc->destruct(pc); + */ + pc = setNextItem(reg->reglives.usedpCodes); + } + + + reg->isFree = 1; + reg->wasUsed = 0; + + total_registers_saved++; // debugging stats. + } else if( (used > 2) && optimize_multi_uses) { + + set *rset1=NULL; + set *rset2=NULL; + int searching=1; + + pCodeFlow *pcfl1=NULL, *pcfl2=NULL; + + /* examine the number of times this register is used */ + + + rset1 = reg->reglives.usedpCodes; + while(rset1 && searching) { + + pc1 = rset1->item; + rset2 = rset1->next; + + if(pc1 && isPCI(pc1) && ( (pcfl1 = PCI(pc1)->pcflow) != NULL) ) { + + //while(rset2 && searching) { + if(rset2) { + + pc2 = rset2->item; + if(pc2 && isPCI(pc2) && ( (pcfl2 = PCI(pc2)->pcflow) != NULL) ) { + if(pcfl2 == pcfl1) { + + if(pCodeOptime2pCodes(pc1, pc2, pcfl_used, reg, 0,optimize_level)) + searching = 0; + } + } + + //rset2 = rset2->next; + + } + } + rset1 = rset1->next; + } + } + } + } +} - int passes = 4; - int saved = 0; - - do { - saved = total_registers_saved; - - /* Identify registers used in one flow sequence */ - OptimizeRegUsage(dynAllocRegs); - OptimizeRegUsage(dynStackRegs); - OptimizeRegUsage(dynDirectRegs); +/*-----------------------------------------------------------------* +* void pCodeRegOptimeRegUsage(pBlock *pb) +*-----------------------------------------------------------------*/ +void pCodeRegOptimizeRegUsage(int level) +{ + + int passes; + int saved = 0; + int t = total_registers_saved; + + if(!register_optimization) + return; +#define OPT_PASSES 4 + passes = OPT_PASSES; + + do { + saved = total_registers_saved; + + /* Identify registers used in one flow sequence */ + OptimizeRegUsage(dynAllocRegs,level, (OPT_PASSES-passes)); + OptimizeRegUsage(dynStackRegs,level, (OPT_PASSES-passes)); + OptimizeRegUsage(dynDirectRegs,0, (OPT_PASSES-passes)); + + if(total_registers_saved != saved) + DFPRINTF((stderr, " *** pass %d, Saved %d registers, total saved %d ***\n", + (1+OPT_PASSES-passes),total_registers_saved-saved,total_registers_saved)); + + passes--; + + } while( passes && ((total_registers_saved != saved) || (passes==OPT_PASSES-1)) ); + + if(total_registers_saved == t) + DFPRINTF((stderr, "No registers saved on this pass\n")); - if(total_registers_saved != saved) - fprintf(stderr, " *** Saved %d registers, total saved %d ***\n", total_registers_saved-saved,total_registers_saved); + /* + fprintf(stderr,"dynamically allocated regs:\n"); + dbg_regusage(dynAllocRegs); + fprintf(stderr,"stack regs:\n"); + dbg_regusage(dynStackRegs); + fprintf(stderr,"direct regs:\n"); + dbg_regusage(dynDirectRegs); + */ +} - } while( passes-- && (total_registers_saved != saved)); -/* - fprintf(stderr,"dynamically allocated regs:\n"); - dbg_regusage(dynAllocRegs); - fprintf(stderr,"stack regs:\n"); - dbg_regusage(dynStackRegs); - fprintf(stderr,"direct regs:\n"); - dbg_regusage(dynDirectRegs); -*/ +/*-----------------------------------------------------------------* +* void RegsUnMapLiveRanges(set *regset) +* +*-----------------------------------------------------------------*/ +void RegsSetUnMapLiveRanges(set *regset) +{ + regs *reg; + + while(regset) { + reg = regset->item; + regset = regset->next; + + + deleteSet(®->reglives.usedpCodes); + deleteSet(®->reglives.usedpFlows); + deleteSet(®->reglives.assignedpFlows); + + } + +} +void RegsUnMapLiveRanges(void) +{ + + RegsSetUnMapLiveRanges(dynAllocRegs); + RegsSetUnMapLiveRanges(dynStackRegs); + RegsSetUnMapLiveRanges(dynDirectRegs); + RegsSetUnMapLiveRanges(dynProcessorRegs); + RegsSetUnMapLiveRanges(dynDirectBitRegs); + RegsSetUnMapLiveRanges(dynInternalRegs); + }