cleaned up the mess I left behind
[fw/sdcc] / src / SDCCloop.c
index 73782f3f814188365090c08d95cffbff11d2dd1f..55021f34798aa1a0d80e920e8a9a29bd86e22f6a 100644 (file)
    under the terms of the GNU General Public License as published by the
    Free Software Foundation; either version 2, or (at your option) any
    later version.
-   
+
    This program is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU General Public License for more details.
-   
+
    You should have received a copy of the GNU General Public License
    along with this program; if not, write to the Free Software
    Foundation, 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
-   
+
    In other words, you are welcome to use, share and improve this program.
    You are forbidden to forbid anyone else to use, share and improve
-   what you give them.   Help stamp out software-hoarding!  
+   what you give them.   Help stamp out software-hoarding!
 -------------------------------------------------------------------------*/
-#include <stdio.h>
-#include <stdlib.h>
-#include "SDCCglobl.h"
-#include "SDCCast.h"
-#include "SDCCmem.h"
-#include "SDCCy.h"
-#include "SDCChasht.h"
-#include "SDCCbitv.h"
-#include "SDCCset.h"
-#include "SDCCicode.h"
-#include "SDCClabel.h"
-#include "SDCCBBlock.h"
-#include "SDCCdflow.h"
-#include "SDCCloop.h"
-
-DEFSETFUNC(isDefAlive);
-
-STACK_DCL(regionStack,eBBlock *, MAX_NEST_LEVEL * 10);
+
+#include "common.h"
+#include "newalloc.h"
+
+DEFSETFUNC (isDefAlive);
+
+STACK_DCL (regionStack, eBBlock *, MAX_NEST_LEVEL * 10);
 
 /*-----------------------------------------------------------------*/
 /* newInduction - creates a new induction variable                 */
 /*-----------------------------------------------------------------*/
-induction *newInduction (operand *sym, unsigned int op, 
-                        long constVal, iCode *ic, operand *asym)
+induction *
+newInduction (operand * sym, unsigned int op,
+             long constVal, iCode * ic, operand * asym)
 {
-    induction *ip;
-
-    ALLOC(ip,sizeof(induction));
+  induction *ip;
 
-    ip->sym = sym;
-    ip->asym= asym;
-    ip->op = op;
-    ip->cval = constVal;   
-    ip->ic = ic;
+  ip = Safe_alloc ( sizeof (induction));
 
-    return ip;
+  ip->sym = sym;
+  ip->asym = asym;
+  ip->op = op;
+  ip->cval = constVal;
+  ip->ic = ic;
+  updateSpillLocation(ic,1);
+  return ip;
 }
 
 /*-----------------------------------------------------------------*/
 /* newRegion - allocate & returns a loop structure                 */
 /*-----------------------------------------------------------------*/
-region *newRegion ()
+region *
+newRegion ()
 {
-    region *lp ;
+  region *lp;
 
-    ALLOC(lp,sizeof(region));  
+  lp = Safe_alloc ( sizeof (region));
 
-    return lp;
+  return lp;
 }
 
 
 /*-----------------------------------------------------------------*/
 /* pinduction - prints induction                                   */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(pinduction)
+DEFSETFUNC (pinduction)
 {
-    induction *ip = item;
-    iCodeTable *icTab;
-
-    fprintf(stdout,"\t");
-    printOperand(ip->sym,stdout);
-    icTab = getTableEntry(ip->ic->op);
-    icTab->iCodePrint(stdout,ip->ic,icTab->printName);
-    fprintf(stdout," %04d\n",(int)ip->cval);
-    return 0;
+  induction *ip = item;
+  iCodeTable *icTab;
+
+  fprintf (stdout, "\t");
+  printOperand (ip->sym, stdout);
+  icTab = getTableEntry (ip->ic->op);
+  icTab->iCodePrint (stdout, ip->ic, icTab->printName);
+  fprintf (stdout, " %04d\n", (int) ip->cval);
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* pregion - prints loop information                                */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(pregion)
+DEFSETFUNC (pregion)
 {
-    region *lp = item;
-
-    printf("================\n");
-    printf(" loop with entry -- > "); printEntryLabel(lp->entry,ap);
-    printf("\n");
-    printf(" loop body --> "); applyToSet(lp->regBlocks,printEntryLabel);
-    printf("\n");
-    printf(" loop exits --> "); applyToSet(lp->exits,printEntryLabel);
-    printf("\n");
-    return 0;
+  region *lp = item;
+
+  printf ("================\n");
+  printf (" loop with entry -- > ");
+  printEntryLabel (lp->entry, ap);
+  printf ("\n");
+  printf (" loop body --> ");
+  applyToSet (lp->regBlocks, printEntryLabel);
+  printf ("\n");
+  printf (" loop exits --> ");
+  applyToSet (lp->exits, printEntryLabel);
+  printf ("\n");
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* backEdges - returns a list of back edges                        */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(backEdges)
+DEFSETFUNC (backEdges)
 {
-    edge *ep = item;
-    V_ARG(set **,bEdges);
-
-    /* if this is a back edge ; to determine this we check */
-    /* to see if the 'to' is in the dominator list of the  */
-    /* 'from' if yes then this is a back edge              */
-    if (bitVectBitValue (ep->from->domVect,ep->to->bbnum)) {
-       addSetHead (bEdges,ep);
-       return 1;
+  edge *ep = item;
+  V_ARG (set **, bEdges);
+
+  /* if this is a back edge ; to determine this we check */
+  /* to see if the 'to' is in the dominator list of the  */
+  /* 'from' if yes then this is a back edge              */
+  if (bitVectBitValue (ep->from->domVect, ep->to->bbnum))
+    {
+      addSetHead (bEdges, ep);
+      return 1;
     }
 
-    return 0;
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* intersectLoopSucc - returns intersection of loop Successors     */
 /*-----------------------------------------------------------------*/
-static bitVect *intersectLoopSucc ( set *lexits, eBBlock **ebbs)
-{    
-    bitVect *succVect = NULL;    
-    eBBlock *exit = setFirstItem(lexits);
-    
-    if (!exit)
-       return NULL;
-
-    succVect = bitVectCopy(exit->succVect);
-
-    for (exit = setNextItem(lexits); exit ; 
-        exit = setNextItem(lexits)) {
-       succVect = bitVectIntersect(succVect,
-                                exit->succVect);                               
+static bitVect *
+intersectLoopSucc (set * lexits, eBBlock ** ebbs)
+{
+  bitVect *succVect = NULL;
+  eBBlock *exit = setFirstItem (lexits);
+
+  if (!exit)
+    return NULL;
+
+  succVect = bitVectCopy (exit->succVect);
+
+  for (exit = setNextItem (lexits); exit;
+       exit = setNextItem (lexits))
+    {
+      succVect = bitVectIntersect (succVect,
+                                  exit->succVect);
     }
-    
-    return succVect ;
+
+  return succVect;
 }
 
 
 /*-----------------------------------------------------------------*/
 /* loopInsert will insert a block into the loop set                */
 /*-----------------------------------------------------------------*/
-static void loopInsert (set **regionSet, eBBlock *block)
+static void 
+loopInsert (set ** regionSet, eBBlock * block)
 {
-    if (!isinSet (*regionSet,block)) {
-       addSetHead (regionSet,block);
-       STACK_PUSH(regionStack,block);
+  if (!isinSet (*regionSet, block))
+    {
+      addSetHead (regionSet, block);
+      STACK_PUSH (regionStack, block);
     }
 }
 
 /*-----------------------------------------------------------------*/
 /* insertIntoLoop - insert item into loop                          */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(insertIntoLoop)
+DEFSETFUNC (insertIntoLoop)
 {
-    eBBlock *ebp = item ;
-    V_ARG(set **,regionSet);
-    
-    loopInsert (regionSet,ebp);
-    return 0;
+  eBBlock *ebp = item;
+  V_ARG (set **, regionSet);
+
+  loopInsert (regionSet, ebp);
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* isNotInBlocks - will return 1 if not is blocks                  */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(isNotInBlocks)
+DEFSETFUNC (isNotInBlocks)
 {
-    eBBlock *ebp = item;
-    V_ARG(set *,blocks);    
+  eBBlock *ebp = item;
+  V_ARG (set *, blocks);
 
-    if (! isinSet (blocks,ebp))                
-       return 1;    
+  if (!isinSet (blocks, ebp))
+    return 1;
 
-    return 0;  
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* hasIncomingDefs - has definitions coming into the loop. i.e.    */
 /* check to see if the preheaders outDefs has any definitions      */
 /*-----------------------------------------------------------------*/
-int hasIncomingDefs (region *lreg, operand *op)
+int 
+hasIncomingDefs (region * lreg, operand * op)
 {
-    eBBlock *preHdr = lreg->entry->preHeader;
+  eBBlock *preHdr = lreg->entry->preHeader;
 
-    if (preHdr && bitVectBitsInCommon(preHdr->outDefs,OP_DEFS(op)))
-       return 1;
-    return 0;
+  if (preHdr && bitVectBitsInCommon (preHdr->outDefs, OP_DEFS (op)))
+    return 1;
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* findLoopEndSeq - will return the sequence number of the last    */
 /* iCode with the maximum dfNumber in the region                   */
 /*-----------------------------------------------------------------*/
-int findLoopEndSeq (region *lreg)
+int 
+findLoopEndSeq (region * lreg)
 {
-    eBBlock *block;
-    eBBlock *lblock;
-
-    for (block = lblock =setFirstItem(lreg->regBlocks); block;
-        block = setNextItem(lreg->regBlocks)) {
-       if (block != lblock && block->lSeq > lblock->lSeq)
-           lblock = block;
+  eBBlock *block;
+  eBBlock *lblock;
+
+  for (block = lblock = setFirstItem (lreg->regBlocks); block;
+       block = setNextItem (lreg->regBlocks))
+    {
+      if (block != lblock && block->lSeq > lblock->lSeq)
+       lblock = block;
     }
 
-    return lblock->lSeq;
+  return lblock->lSeq;
 }
 
 /*-----------------------------------------------------------------*/
 /* addToExitsMarkDepth - will add the the exitSet all blocks that  */
 /* have exits, will also update the depth field in the blocks      */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(addToExitsMarkDepth)
+DEFSETFUNC (addToExitsMarkDepth)
 {
-    eBBlock *ebp = item ;
-    V_ARG(set *,loopBlocks);
-    V_ARG(set **,exits);    
-    V_ARG(int, depth);
-    V_ARG(region *,lr);
-
-    /* mark the loop depth of this block */
-    if (!ebp->depth)
-       ebp->depth = depth;
-
-    /* put the loop region info in the block */
-    /* NOTE: here we will update only the inner most loop
-       that it is a part of */
-    if (!ebp->partOfLoop)
-       ebp->partOfLoop = lr;
-
-    /* if any of the successors go out of the loop then */
-    /* we add this one to the exits */
-    if ( applyToSet(ebp->succList,isNotInBlocks,loopBlocks)) {
-       addSetHead (exits,ebp);
-       return 1;
+  eBBlock *ebp = item;
+  V_ARG (set *, loopBlocks);
+  V_ARG (set **, exits);
+  V_ARG (int, depth);
+  V_ARG (region *, lr);
+
+  /* mark the loop depth of this block */
+  //if (!ebp->depth)
+  if (ebp->depth<depth)
+    ebp->depth = depth;
+
+  /* put the loop region info in the block */
+  /* NOTE: here we will update only the inner most loop
+     that it is a part of */
+  if (!ebp->partOfLoop)
+    ebp->partOfLoop = lr;
+
+  /* if any of the successors go out of the loop then */
+  /* we add this one to the exits */
+  if (applyToSet (ebp->succList, isNotInBlocks, loopBlocks))
+    {
+      addSetHead (exits, ebp);
+      return 1;
     }
-   
-    return 0;
+
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* createLoop - will create a set of region                        */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(createLoop)
+DEFSETFUNC (createLoop)
 {
-    edge *ep = item;
-    V_ARG(set **,allRegion);
-    region *aloop = newRegion();
-    eBBlock *block;
-
-    /* make sure regionStack is empty */
-    while (!STACK_EMPTY(regionStack))
-       STACK_POP(regionStack);
-
-    /* add the entryBlock */
-    addSet (&aloop->regBlocks,ep->to);
-    loopInsert (&aloop->regBlocks,ep->from);
-
-    while (!STACK_EMPTY(regionStack)) {
-       block = STACK_POP(regionStack);
-       /* if block != entry */
-       if (block != ep->to)
-           applyToSet(block->predList,insertIntoLoop,&aloop->regBlocks);
+  edge *ep = item;
+  V_ARG (set **, allRegion);
+  V_ARG (eBBlock **,ebbs);
+  V_ARG (int,count);
+  region *aloop = newRegion ();
+  eBBlock *block;
+  int dfMin = count ,dfMax =0, i;
+
+  /* make sure regionStack is empty */
+  while (!STACK_EMPTY (regionStack))
+    STACK_POP (regionStack);
+
+  /* add the entryBlock */
+  addSet (&aloop->regBlocks, ep->to);
+  loopInsert (&aloop->regBlocks, ep->from);
+
+  while (!STACK_EMPTY (regionStack))
+    {
+      block = STACK_POP (regionStack);
+      /* if block != entry */
+      if (block != ep->to)
+       applyToSet (block->predList, insertIntoLoop, &aloop->regBlocks);
     }
 
-    aloop->entry = ep->to ;
-       
-    /* now add it to the set */
-    addSetHead (allRegion,aloop);
-    return 0;
+  aloop->entry = ep->to;
+
+  /* set max & min dfNum for loopRegion */
+  for ( block = setFirstItem(aloop->regBlocks); block; 
+       block = setNextItem(aloop->regBlocks)) {
+      if (block->dfnum > dfMax) dfMax = block->dfnum;
+      if (block->dfnum < dfMin) dfMin = block->dfnum;
+  }
+
+  /* all blocks that have dfnumbers between dfMin & dfMax are also
+     part of loop */
+  for (i = 0 ; i < count ; i++) {
+    if (ebbs[i]->dfnum > dfMin && 
+          ebbs[i]->dfnum < dfMax &&
+         !isinSet(aloop->regBlocks,ebbs[i])) {
+         if (!ebbs[i]->partOfLoop) {
+           ebbs[i]->partOfLoop = aloop;
+         }
+      }
+  }
+
+  /* and if this is a conditional block, the other side of the IFX 
+     (if any, that could have a greater dfnum) is too */
+  {
+    // just a burp, but I'm getting close :)
+  }
+  
+
+  /* now add it to the set */
+  addSetHead (allRegion, aloop);
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* dominatedBy - will return 1 if item is dominated by block       */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(dominatedBy) 
+DEFSETFUNC (dominatedBy)
 {
-    eBBlock *ebp = item;
-    V_ARG(eBBlock *,block);
+  eBBlock *ebp = item;
+  V_ARG (eBBlock *, block);
 
-    return bitVectBitValue (ebp->domVect,block->bbnum);
+  return bitVectBitValue (ebp->domVect, block->bbnum);
 }
 
 /*-----------------------------------------------------------------*/
 /* addDefInExprs - adds an expression into the inexpressions       */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(addDefInExprs)
+DEFSETFUNC (addDefInExprs)
 {
-    eBBlock *ebp = item;
-    V_ARG(cseDef *,cdp);
-    V_ARG(eBBlock **,ebbs);
-    V_ARG(int,count);
-
-    addSetHead(&ebp->inExprs,cdp);
-    cseBBlock (ebp,0,ebbs,count);
-    return 0;
+  eBBlock *ebp = item;
+  V_ARG (cseDef *, cdp);
+  V_ARG (eBBlock **, ebbs);
+  V_ARG (int, count);
+
+  addSetHead (&ebp->inExprs, cdp);
+  cseBBlock (ebp, 0, ebbs, count);
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
-/* assignmentsToSym - for a set of blocks determine # time assigned*/
+/* assignmentsToSym - for a set of blocks determine # time assigned */
 /*-----------------------------------------------------------------*/
- int assignmentsToSym (set *sset, operand *sym)
-{   
-    eBBlock *ebp ;
-    int assigns = 0;
-    set *blocks = setFromSet(sset);
-
-    for (ebp = setFirstItem(blocks) ; ebp ;
-        ebp = setNextItem(blocks)) {       
-       
-       /* get all the definitions for this symbol
-          in this block */
-       bitVect *defs = bitVectIntersect(ebp->ldefs,OP_DEFS(sym));
-       assigns += bitVectnBitsOn(defs);
-       setToNull((void **)&defs);
-       
+int 
+assignmentsToSym (set * sset, operand * sym)
+{
+  eBBlock *ebp;
+  int assigns = 0;
+  set *blocks = setFromSet (sset);
+
+  for (ebp = setFirstItem (blocks); ebp;
+       ebp = setNextItem (blocks))
+    {
+
+      /* get all the definitions for this symbol
+         in this block */
+      bitVect *defs = bitVectIntersect (ebp->ldefs, OP_DEFS (sym));
+      assigns += bitVectnBitsOn (defs);
+      setToNull ((void **) &defs);
+
     }
 
-    return assigns;
+  return assigns;
 }
 
 
 /*-----------------------------------------------------------------*/
 /* isOperandInvariant - determines if an operand is an invariant   */
 /*-----------------------------------------------------------------*/
-int isOperandInvariant (operand *op, region *theLoop, set *lInvars)
+int 
+isOperandInvariant (operand * op, region * theLoop, set * lInvars)
 {
-    int opin = 0 ;
-    /* operand is an invariant if it is a                */
-    /*       a. constants .                              */
-    /*       b. that have defintions reaching loop entry */
-    /*       c. that are already defined as invariant    */
-    /*       d. has no assignments in the loop           */    
-    if (op) {
-       if (IS_OP_LITERAL(op))
-           opin = 1 ;
-       else
-           if (IS_SYMOP(op) &&
-               OP_SYMBOL(op)->addrtaken)
-               opin = 0 ;
-           else
-               if (ifDefSymIs(theLoop->entry->inExprs,op))
-                   opin = 1 ;
-               else
-                   if (ifDefSymIs(lInvars,op))
-                       opin = 1 ;              
-                   else
-                       if (IS_SYMOP(op)         &&
-                           ! IS_OP_GLOBAL(op)   &&
-                           ! IS_OP_VOLATILE(op) &&                             
-                           assignmentsToSym (theLoop->regBlocks,op) == 0 )
-                           opin = 1 ;              
-    } else opin++ ;
-
-    return opin ;
+  int opin = 0;
+  /* operand is an invariant if it is a                */
+  /*       a. constants .                              */
+  /*       b. that have defintions reaching loop entry */
+  /*       c. that are already defined as invariant    */
+  /*       d. has no assignments in the loop           */
+  if (op)
+    {
+      if (IS_OP_LITERAL (op))
+       opin = 1;
+      else if (IS_SYMOP (op) &&
+              OP_SYMBOL (op)->addrtaken)
+       opin = 0;
+      else if (ifDefSymIs (theLoop->entry->inExprs, op))
+       opin = 1;
+      else if (ifDefSymIs (lInvars, op))
+       opin = 1;
+      else if (IS_SYMOP (op) &&
+              !IS_OP_GLOBAL (op) &&
+              !IS_OP_VOLATILE (op) &&
+              assignmentsToSym (theLoop->regBlocks, op) == 0)
+       opin = 1;
+    }
+  else
+    opin++;
+
+  return opin;
 }
 
 /*-----------------------------------------------------------------*/
 /* pointerAssigned - will return 1 if pointer set found            */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(pointerAssigned)
+DEFSETFUNC (pointerAssigned)
 {
-    eBBlock *ebp = item;
-    V_ARG(operand *,op);
+  eBBlock *ebp = item;
+  V_ARG (operand *, op);
 
-    return ebp->hasFcall || bitVectBitValue(ebp->ptrsSet,op->key);
+  return ebp->hasFcall || bitVectBitValue (ebp->ptrsSet, op->key);
 }
 
 /*-----------------------------------------------------------------*/
 /* hasNonPtrUse - returns true if operand has non pointer usage    */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(hasNonPtrUse)
+DEFSETFUNC (hasNonPtrUse)
 {
-    eBBlock *ebp = item;
-    V_ARG(operand *,op);
-    iCode *ic = usedInRemaining(op,ebp->sch);
+  eBBlock *ebp = item;
+  V_ARG (operand *, op);
+  iCode *ic = usedInRemaining (op, ebp->sch);
 
-    if (ic && !POINTER_SET(ic) && !POINTER_GET(ic))
-       return 1;
+  if (ic && !POINTER_SET (ic) && !POINTER_GET (ic))
+    return 1;
+
+  return 0;
 
-    return 0;
-    
 }
 
 /*-----------------------------------------------------------------*/
 /* loopInvariants - takes loop invariants out of region            */
 /*-----------------------------------------------------------------*/
- int loopInvariants( region *theLoop , eBBlock **ebbs, int count)
-{      
-    eBBlock *lBlock ;
-    set *lInvars = NULL ;
-    
-    int change = 0 ;
-
-    /* if the preHeader does not exist then do nothing */
-    /* or no exits then do nothing ( have to think about this situation */
-    if (theLoop->entry->preHeader == NULL ||
-       theLoop->exits == NULL )
-       return 0;
-
-    /* we will do the elimination for those blocks        */
-    /* in the loop that dominates all exits from the loop */
-    for (lBlock = setFirstItem(theLoop->regBlocks); lBlock;
-        lBlock = setNextItem(theLoop->regBlocks)) {
-       
-       iCode *ic;
-       int domsAllExits ;
-       int i ;
-
-       /* mark the dominates all exits flag */
-       domsAllExits = ( applyToSet (theLoop->exits,dominatedBy,lBlock) ==
-                        elementsInSet (theLoop->exits));
-       
-
-       /* now we go thru the instructions of this block and */
-       /* collect those instructions with invariant operands*/
-       for ( ic = lBlock->sch ; ic ; ic = ic->next ) {
-           
-           int lin , rin ;
-           cseDef *ivar ;
-
-           if (SKIP_IC(ic) || POINTER_SET(ic) || ic->op == IFX)
-               continue ;
-
-           /* if result is volatile then skip */
-           if (IC_RESULT(ic) &&  
-               ( isOperandVolatile(IC_RESULT(ic),TRUE) || 
-                 IS_OP_PARM(IC_RESULT(ic))))
-               continue ;
-
-           lin = rin = 0 ;
-           
-           /* special case */
-           /* if address of then it is an invariant */
-           if (ic->op == ADDRESS_OF && 
-               IS_SYMOP(IC_LEFT(ic)) &&
-               IS_AGGREGATE(operandType(IC_LEFT(ic))))
-               lin++;  
-           else
-               /* check if left operand is an invariant */
-               if ( (lin = isOperandInvariant (IC_LEFT(ic),theLoop,lInvars)))  
-                   /* if this is a pointer get then make sure
-                      that the pointer set does not exist in 
-                      any of the blocks */
-                   if (POINTER_GET(ic) && 
-                       ( applyToSet (theLoop->regBlocks,pointerAssigned,IC_LEFT(ic)) ))
-                       lin = 0;                                    
-
-           /* do the same for right */
-           rin = isOperandInvariant (IC_RIGHT(ic),theLoop, lInvars);       
-
-           /* if this is a POINTER_GET then special case, make sure all
-              usages within the loop are POINTER_GET any other usage
-              would mean that this is not an invariant , since the pointer
-              could then be passed as a parameter */
-           if (POINTER_GET(ic) &&
-               applyToSet(theLoop->regBlocks,hasNonPtrUse,IC_LEFT(ic)))
-               continue ;
-
-           /* if both the left & right are invariants : then check that*/
-           /* this definition exists in the out definition of all the  */
-           /* blocks, this will ensure that this is not assigned any   */
-           /* other value in the loop , and not used in this block     */
-           /* prior to this definition which means only this definition*/
-           /* is used in this loop                                     */
-           if (lin && rin && IC_RESULT(ic)) {
-               eBBlock *sBlock ;
-               set *lSet = setFromSet(theLoop->regBlocks);
-
-               /* if this block does not dominate all exists */
-               /* make sure this defintion is not used anywhere else */
-               if (!domsAllExits) {
-
-                   if (isOperandGlobal(IC_RESULT(ic)))
-                       continue;
-                   /* for successors for all exits */
-                   for ( sBlock = setFirstItem(theLoop->exits); sBlock; 
-                         sBlock = setNextItem(theLoop->exits)) {
-                       
-                       for(i=0; i < count; ebbs[i++]->visited = 0);
-                       lBlock->visited = 1;
-                       if ( applyToSet (sBlock->succList,isDefAlive,ic))
-                           break ;
+int 
+loopInvariants (region * theLoop, eBBlock ** ebbs, int count)
+{
+  eBBlock *lBlock;
+  set *lInvars = NULL;
+
+  int change = 0;
+  int fCallsInBlock;
+
+  /* if the preHeader does not exist then do nothing */
+  /* or no exits then do nothing ( have to think about this situation */
+  if (theLoop->entry->preHeader == NULL ||
+      theLoop->exits == NULL)
+    return 0;
+
+  /* we will do the elimination for those blocks        */
+  /* in the loop that dominates all exits from the loop */
+  for (lBlock = setFirstItem (theLoop->regBlocks); lBlock;
+       lBlock = setNextItem (theLoop->regBlocks))
+    {
+
+      iCode *ic;
+      int domsAllExits;
+      int i;
+
+      /* mark the dominates all exits flag */
+      domsAllExits = (applyToSet (theLoop->exits, dominatedBy, lBlock) ==
+                     elementsInSet (theLoop->exits));
+
+      /* find out if we have a function call in this block */
+      for (ic = lBlock->sch, fCallsInBlock=0; ic; ic = ic->next) {
+       if (SKIP_IC(ic)) {
+         fCallsInBlock++;
+       }
+      }
+
+      /* now we go thru the instructions of this block and */
+      /* collect those instructions with invariant operands */
+      for (ic = lBlock->sch; ic; ic = ic->next)
+       {
+
+         int lin, rin;
+         cseDef *ivar;
+
+         /* TODO this is only needed if the call is between
+            here and the definition, but I am too lazy to do that now */
+
+         /* if there are function calls in this block */
+         if (fCallsInBlock) {
+
+           /* if this is a pointer get */
+           if (POINTER_GET(ic)) {
+             continue;
+           }
+
+           /* if this is an assignment from a global */
+           if (ic->op=='=' && isOperandGlobal(IC_RIGHT(ic))) {
+             continue;
+           }
+         }
+
+         if (SKIP_IC (ic) || POINTER_SET (ic) || ic->op == IFX)
+           continue;
+
+         /* if result is volatile then skip */
+         if (IC_RESULT (ic) &&
+             (isOperandVolatile (IC_RESULT (ic), TRUE) ||
+              IS_OP_PARM (IC_RESULT (ic))))
+           continue;
+
+         /* if result depends on a volatile then skip */
+         if ((IC_LEFT(ic) && isOperandVolatile(IC_LEFT(ic), TRUE)) ||
+             (IC_RIGHT(ic) && isOperandVolatile(IC_RIGHT(ic), TRUE)))
+           continue;
+
+         lin = rin = 0;
+         
+         /* special case */
+         /* if address of then it is an invariant */
+         if (ic->op == ADDRESS_OF &&
+             IS_SYMOP (IC_LEFT (ic)) &&
+             IS_AGGREGATE (operandType (IC_LEFT (ic))))
+           lin++;
+         else {
+           /* check if left operand is an invariant */
+           if ((lin = isOperandInvariant (IC_LEFT (ic), theLoop, lInvars)))
+             /* if this is a pointer get then make sure
+                that the pointer set does not exist in
+                any of the blocks */
+             if (POINTER_GET (ic) &&
+                 (applyToSet (theLoop->regBlocks, 
+                              pointerAssigned, IC_LEFT (ic))))
+               lin = 0;
+         }
+         
+         /* do the same for right */
+         rin = isOperandInvariant (IC_RIGHT (ic), theLoop, lInvars);
+         
+         /* if this is a POINTER_GET then special case, make sure all
+            usages within the loop are POINTER_GET any other usage
+            would mean that this is not an invariant , since the pointer
+            could then be passed as a parameter */
+         if (POINTER_GET (ic) &&
+             applyToSet (theLoop->regBlocks, hasNonPtrUse, IC_LEFT (ic)))
+           continue;
+
+         /* if both the left & right are invariants : then check that */
+         /* this definition exists in the out definition of all the  */
+         /* blocks, this will ensure that this is not assigned any   */
+         /* other value in the loop , and not used in this block     */
+         /* prior to this definition which means only this definition */
+         /* is used in this loop                                     */
+         if (lin && rin && IC_RESULT (ic))
+           {
+             eBBlock *sBlock;
+             set *lSet = setFromSet (theLoop->regBlocks);
+
+             /* if this block does not dominate all exists */
+             /* make sure this defintion is not used anywhere else */
+             if (!domsAllExits)
+               {
+
+                 if (isOperandGlobal (IC_RESULT (ic)))
+                   continue;
+                 /* for successors for all exits */
+                 for (sBlock = setFirstItem (theLoop->exits); sBlock;
+                      sBlock = setNextItem (theLoop->exits))
+                   {
+
+                     for (i = 0; i < count; ebbs[i++]->visited = 0);
+                     lBlock->visited = 1;
+                     if (applyToSet (sBlock->succList, isDefAlive, ic))
+                       break;
                    }
 
-                   /* we have found usage */
-                   if (sBlock )
-                       continue ;
+                 /* we have found usage */
+                 if (sBlock)
+                   continue;
                }
-               
-               /* now make sure this is the only definition */
-               for (sBlock = setFirstItem(lSet); sBlock ;
-                    sBlock = setNextItem (lSet)) {
-                   /* if this is the block make sure the definition */
-                   /* reaches the end of the block */
-                   if (sBlock == lBlock ) {
-                       if (! ifDiCodeIs (sBlock->outExprs,ic))
-                           break;
+
+             /* now make sure this is the only definition */
+             for (sBlock = setFirstItem (lSet); sBlock;
+                  sBlock = setNextItem (lSet))
+               {
+                 /* if this is the block make sure the definition */
+                 /* reaches the end of the block */
+                 if (sBlock == lBlock)
+                   {
+                     if (!ifDiCodeIs (sBlock->outExprs, ic))
+                       break;
                    }
-                   else
-                       if (bitVectBitsInCommon (sBlock->defSet,OP_DEFS(IC_RESULT(ic))))
-                           break;
+                 else if (bitVectBitsInCommon (sBlock->defSet, OP_DEFS (IC_RESULT (ic))))
+                   break;
                }
-               
-               if (sBlock)
-                   continue ; /* another definition present in the block */
-
-               /* now check if it exists in the in of this block */
-               /* if not then it was killed before this instruction */
-               if (! bitVectBitValue (lBlock->inDefs,ic->key))
-                   continue ;
-
-               /* now we know it is a true invariant */
-               /* remove it from the insts chain & put */
-               /* in the invariant set                */
-               OP_SYMBOL(IC_RESULT(ic))->isinvariant = 1;
-               remiCodeFromeBBlock (lBlock,ic);
-               
-               /* maintain the data flow */
-               /* this means removing from definition from the */
-               /* defset of this block and adding it to the    */
-               /* inexpressions of all blocks within the loop  */
-               bitVectUnSetBit (lBlock->defSet,ic->key);
-               bitVectUnSetBit (lBlock->ldefs,ic->key);
-               ivar = newCseDef(IC_RESULT(ic),ic);
-               applyToSet (theLoop->regBlocks, addDefInExprs, ivar,ebbs,count);
-               addSet(&lInvars,ivar);
+
+             if (sBlock)
+               continue;       /* another definition present in the block */
+
+             /* now check if it exists in the in of this block */
+             /* if not then it was killed before this instruction */
+             if (!bitVectBitValue (lBlock->inDefs, ic->key))
+               continue;
+
+             /* now we know it is a true invariant */
+             /* remove it from the insts chain & put */
+             /* in the invariant set                */
+             OP_SYMBOL (IC_RESULT (ic))->isinvariant = 1;
+             remiCodeFromeBBlock (lBlock, ic);
+
+             /* maintain the data flow */
+             /* this means removing from definition from the */
+             /* defset of this block and adding it to the    */
+             /* inexpressions of all blocks within the loop  */
+             bitVectUnSetBit (lBlock->defSet, ic->key);
+             bitVectUnSetBit (lBlock->ldefs, ic->key);
+             ivar = newCseDef (IC_RESULT (ic), ic);
+             applyToSet (theLoop->regBlocks, addDefInExprs, ivar, ebbs, count);
+             addSet (&lInvars, ivar);
            }
        }
-    } /* for all loop blocks */
-
-    /* if we have some invariants then */
-    if (lInvars) {
-       eBBlock *preHdr= theLoop->entry->preHeader ;
-       iCode *icFirst = NULL , *icLast = NULL ;
-       cseDef *cdp;
-
-       /* create an iCode chain from it */
-       for (cdp = setFirstItem(lInvars); cdp ; cdp = setNextItem(lInvars)) {
-
-           /* maintain data flow .. add it to the */
-           /* defSet & outExprs of the preheader  */
-           preHdr->defSet = bitVectSetBit (preHdr->defSet,cdp->diCode->key);
-           cdp->diCode->lineno = preHdr->ech->lineno;
-           addSetHead (&preHdr->outExprs,cdp);
-           
-
-           if (!icFirst)
-               icFirst = cdp->diCode;
-           if (icLast) {
-               icLast->next = cdp->diCode;
-               cdp->diCode->prev = icLast;
-               icLast = cdp->diCode ;
-           } else 
-               icLast = cdp->diCode;   
-           change++ ;
+    }                          /* for all loop blocks */
+
+  /* if we have some invariants then */
+  if (lInvars)
+    {
+      eBBlock *preHdr = theLoop->entry->preHeader;
+      iCode *icFirst = NULL, *icLast = NULL;
+      cseDef *cdp;
+
+      /* create an iCode chain from it */
+      for (cdp = setFirstItem (lInvars); cdp; cdp = setNextItem (lInvars))
+       {
+
+         /* maintain data flow .. add it to the */
+         /* ldefs defSet & outExprs of the preheader  */
+         preHdr->defSet = bitVectSetBit (preHdr->defSet, cdp->diCode->key);
+         preHdr->ldefs = bitVectSetBit (preHdr->ldefs, cdp->diCode->key);
+         cdp->diCode->lineno = preHdr->ech->lineno;
+         addSetHead (&preHdr->outExprs, cdp);
+
+
+         if (!icFirst)
+           icFirst = cdp->diCode;
+         if (icLast)
+           {
+             icLast->next = cdp->diCode;
+             cdp->diCode->prev = icLast;
+             icLast = cdp->diCode;
+           }
+         else
+           icLast = cdp->diCode;
+         change++;
        }
-       
-       /* add the instruction chain to the end of the 
-          preheader for this loop, preheaders will always
-          have atleast a label */
-       preHdr->ech->next = icFirst ;
-       icFirst->prev = preHdr->ech ;
-       preHdr->ech = icLast;
-       icLast->next = NULL;
-       
+
+      /* add the instruction chain to the end of the
+         preheader for this loop, preheaders will always
+         have atleast a label */
+      preHdr->ech->next = icFirst;
+      icFirst->prev = preHdr->ech;
+      preHdr->ech = icLast;
+      icLast->next = NULL;
+
     }
-    return change ;
+  return change;
 }
 
 /*-----------------------------------------------------------------*/
-/* addressTaken - returns true if the symbol is found in the addrof*/
+/* addressTaken - returns true if the symbol is found in the addrof */
 /*-----------------------------------------------------------------*/
-int addressTaken (set *sset ,operand *sym)
+int 
+addressTaken (set * sset, operand * sym)
 {
-    set *loop;
-    eBBlock *ebp ;
-    set *loop2;   
-
-    for (loop = sset; loop ; loop = loop->next) {
-       ebp = loop->item;
-       loop2 = ebp->addrOf ;
-       while (loop2) {
-           if (isOperandEqual((operand *)loop2->item,sym))
-               return 1;
-           loop2 = loop2->next;
+  set *loop;
+  eBBlock *ebp;
+  set *loop2;
+
+  for (loop = sset; loop; loop = loop->next)
+    {
+      ebp = loop->item;
+      loop2 = ebp->addrOf;
+      while (loop2)
+       {
+         if (isOperandEqual ((operand *) loop2->item, sym))
+           return 1;
+         loop2 = loop2->next;
        }
     }
 
-    return 0;
+  return 0;
 }
 
 
 /*-----------------------------------------------------------------*/
 /* findInduction :- returns 1 & the item if the induction is found */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(findInduction)
+DEFSETFUNC (findInduction)
 {
-    induction *ip = item;
-    V_ARG(operand *,sym);
-    V_ARG(induction **,ipp);
-
-    if (isOperandEqual(ip->sym,sym)) {
-       *ipp = ip;
-       return 1;
+  induction *ip = item;
+  V_ARG (operand *, sym);
+  V_ARG (induction **, ipp);
+
+  if (isOperandEqual (ip->sym, sym))
+    {
+      *ipp = ip;
+      return 1;
     }
 
-    return 0;
+  return 0;
 }
 
 /*-----------------------------------------------------------------*/
 /* findDefInRegion - finds the definition within the region        */
 /*-----------------------------------------------------------------*/
-iCode *findDefInRegion (set *regBlocks, operand *defOp, eBBlock **owner)
+iCode *
+findDefInRegion (set * regBlocks, operand * defOp, eBBlock ** owner)
 {
-    eBBlock *lBlock ;
-    
-    /* for all blocks in the region */
-    for (lBlock = setFirstItem(regBlocks); lBlock ;
-        lBlock = setNextItem(regBlocks)) {
-
-       /* if a definition for this exists */
-       if (bitVectBitsInCommon(lBlock->defSet,OP_DEFS(defOp))) {
-           iCode *ic;
-
-           /* go thru the instruction chain to find it */
-           for (ic = lBlock->sch ; ic ; ic = ic->next )                
-               if (bitVectBitValue (OP_DEFS(defOp),ic->key)) {
-                   if (owner)
-                       *owner = lBlock ;
-                   return ic;
-               }
+  eBBlock *lBlock;
+
+  /* for all blocks in the region */
+  for (lBlock = setFirstItem (regBlocks); lBlock;
+       lBlock = setNextItem (regBlocks))
+    {
+
+      /* if a definition for this exists */
+      if (bitVectBitsInCommon (lBlock->defSet, OP_DEFS (defOp)))
+       {
+         iCode *ic;
+
+         /* go thru the instruction chain to find it */
+         for (ic = lBlock->sch; ic; ic = ic->next)
+           if (bitVectBitValue (OP_DEFS (defOp), ic->key))
+             {
+               if (owner)
+                 *owner = lBlock;
+               return ic;
+             }
        }
     }
 
-    return NULL ;
+  return NULL;
 }
 
 /*-----------------------------------------------------------------*/
 /* basicInduction - finds the basic induction variables in a loop  */
 /*-----------------------------------------------------------------*/
-set *basicInduction (region *loopReg , eBBlock **ebbs, int count)
+set *
+basicInduction (region * loopReg, eBBlock ** ebbs, int count)
 {
-    eBBlock *lBlock ;
-    set *indVars = NULL ;
-
-    /* i.e. all assignments of the form a := a +/- const*/
-    /* for all blocks within the loop do */
-    for ( lBlock = setFirstItem(loopReg->regBlocks); lBlock ;
-         lBlock = setNextItem(loopReg->regBlocks)) {
-       
-       iCode *ic, *dic ;
-
-       /* for all instructions in the blocks do */
-       for ( ic = lBlock->sch ; ic ; ic = ic->next ) {
-           
-           operand *aSym ;
-           unsigned long litValue ;
-           induction *ip;
-           iCode *indIc;
-           eBBlock *owner = NULL;
-           int nexits;
-
-           /* look for assignments of the form */
-           /*   symbolVar := iTempNN */
-           if ( ic->op != '=')
-               continue ;
-
-           if (!IS_TRUE_SYMOP(IC_RESULT(ic)) &&
-               !OP_SYMBOL(IC_RESULT(ic))->isreqv)
-               continue ;
-
-           if (isOperandGlobal(IC_RESULT(ic)))
-               continue ;
-
-           if (!IS_ITEMP(IC_RIGHT(ic)))
-               continue ;
-
-           /* if it has multiple assignments within the loop then skip */
-           if (assignmentsToSym (loopReg->regBlocks,IC_RESULT(ic)) > 1 )
-               continue ;
-           
-           /* if the address of this was taken inside the loop then continue */
-           if (addressTaken (loopReg->regBlocks,IC_RESULT(ic)))
-               continue ;
-
-           /* find the definition for the result in the block */
-           if (! (dic = findDefInRegion (setFromSet(loopReg->regBlocks),
-                                         IC_RIGHT(ic),&owner)))
-               continue ;
-
-           /* if not +/- continue */
-           if (dic->op != '+' && dic->op != '-')
-               continue ;
-
-           /* make sure definition is of the form  a +/- c */
-           if (!IS_OP_LITERAL(IC_LEFT(dic)) && !IS_OP_LITERAL(IC_RIGHT(dic)))
-               continue ;
-           
-           aSym = (IS_OP_LITERAL(IC_RIGHT(dic)) ? 
-                   (litValue = operandLitValue(IC_RIGHT(dic)),IC_LEFT(dic)) : 
-                   (litValue = operandLitValue(IC_LEFT(dic)),IC_RIGHT(dic)));
-           
-           if (!isOperandEqual(IC_RESULT(ic),aSym) &&
-               !isOperandEqual(IC_RIGHT(ic),aSym)) {
-               iCode *ddic ;
-               /* find the definition for this and check */
-               if (!(ddic = findDefInRegion (setFromSet(loopReg->regBlocks),
-                                             aSym,&owner)))                
-                   continue ;
-
-               if (ddic->op != '=')
-                   continue ;
-
-               if (!isOperandEqual(IC_RESULT(ddic),aSym) ||
-                   !isOperandEqual(IC_RIGHT(ddic),IC_RESULT(ic)))
-                   continue ;
-           }
+  eBBlock *lBlock;
+  set *indVars = NULL;
+
+  /* i.e. all assignments of the form a := a +/- const */
+  /* for all blocks within the loop do */
+  for (lBlock = setFirstItem (loopReg->regBlocks); lBlock;
+       lBlock = setNextItem (loopReg->regBlocks))
+    {
+
+      iCode *ic, *dic;
+
+      /* for all instructions in the blocks do */
+      for (ic = lBlock->sch; ic; ic = ic->next)
+       {
+
+         operand *aSym;
+         unsigned long litValue;
+         induction *ip;
+         iCode *indIc;
+         eBBlock *owner = NULL;
+         int nexits;
+
+         /* look for assignments of the form */
+         /*   symbolVar := iTempNN */
+         if (ic->op != '=')
+           continue;
+
+         if (!IS_TRUE_SYMOP (IC_RESULT (ic)) &&
+             !OP_SYMBOL (IC_RESULT (ic))->isreqv)
+           continue;
+
+         if (isOperandGlobal (IC_RESULT (ic)))
+           continue;
+
+         if (!IS_ITEMP (IC_RIGHT (ic)))
+           continue;
+
+         /* if it has multiple assignments within the loop then skip */
+         if (assignmentsToSym (loopReg->regBlocks, IC_RESULT (ic)) > 1)
+           continue;
+
+         /* if the address of this was taken inside the loop then continue */
+         if (addressTaken (loopReg->regBlocks, IC_RESULT (ic)))
+           continue;
+
+         /* find the definition for the result in the block */
+         if (!(dic = findDefInRegion (setFromSet (loopReg->regBlocks),
+                                      IC_RIGHT (ic), &owner)))
+           continue;
+
+         /* if not +/- continue */
+         if (dic->op != '+' && dic->op != '-')
+           continue;
+
+         /* make sure definition is of the form  a +/- c */
+         if (!IS_OP_LITERAL (IC_LEFT (dic)) && !IS_OP_LITERAL (IC_RIGHT (dic)))
+           continue;
+
+         aSym = (IS_OP_LITERAL (IC_RIGHT (dic)) ?
+             (litValue = (unsigned long) operandLitValue (IC_RIGHT (dic)), IC_LEFT (dic)) :
+             (litValue = (unsigned long) operandLitValue (IC_LEFT (dic)), IC_RIGHT (dic)));
+
+         if (!isOperandEqual (IC_RESULT (ic), aSym) &&
+             !isOperandEqual (IC_RIGHT (ic), aSym))
+           {
+             iCode *ddic;
+             /* find the definition for this and check */
+             if (!(ddic = findDefInRegion (setFromSet (loopReg->regBlocks),
+                                           aSym, &owner)))
+               continue;
 
-           /* if the right hand side has more than one usage then
-              don't make it an induction (will have to think some more) */
-           if (bitVectnBitsOn(OP_USES(IC_RIGHT(ic))) > 1)
-                   continue;
+             if (ddic->op != '=')
+               continue;
 
-           /* if the definition is volatile then it cannot be
-              an induction object */
-           if (isOperandVolatile(IC_RIGHT(ic),FALSE) ||
-               isOperandVolatile(IC_RESULT(ic),FALSE))
+             if (!isOperandEqual (IC_RESULT (ddic), aSym) ||
+                 !isOperandEqual (IC_RIGHT (ddic), IC_RESULT (ic)))
                continue;
+           }
 
-           /* whew !! that was a lot of work to find the definition */
-           /* create an induction object */
-           indIc = newiCode('=',NULL,IC_RESULT(ic));
-           indIc->lineno = ic->lineno;
-           IC_RESULT(indIc) = operandFromOperand(IC_RIGHT(ic));
-           IC_RESULT(indIc)->isaddr = 0;
-           OP_SYMBOL(IC_RESULT(indIc))->isind = 1;
-           ip = newInduction (IC_RIGHT(ic),dic->op,litValue,indIc,NULL);
-           
-           /* replace the inducted variable by the iTemp */
-           replaceSymBySym (loopReg->regBlocks,IC_RESULT(ic),IC_RIGHT(ic));
-
-           /* if it has only one exit then remove it from here
-              and put it in the exit block */
-           nexits = elementsInSet (loopReg->exits);
-           if (nexits == 1) {
-               eBBlock *exit = setFirstItem(loopReg->exits);
-
-               /* if it is the same block then there is no
-                  need to move it about */
-               if ( exit != lBlock) {
-                   iCode *saveic = ic->prev;
-                   /* remove it */
-                   remiCodeFromeBBlock(lBlock,ic);
-                   /* clear the definition */
-                   bitVectUnSetBit(lBlock->defSet,ic->key);
-                   /* add it to the exit */
-                   addiCodeToeBBlock(exit,ic,NULL);
-                   /* set the definition bit */
-                   exit->defSet = bitVectSetBit(exit->defSet,ic->key);
-                   ic = saveic ;
+         /* if the right hand side has more than one usage then
+            don't make it an induction (will have to think some more) */
+         if (bitVectnBitsOn (OP_USES (IC_RIGHT (ic))) > 1)
+           continue;
+
+         /* if the definition is volatile then it cannot be
+            an induction object */
+         if (isOperandVolatile (IC_RIGHT (ic), FALSE) ||
+             isOperandVolatile (IC_RESULT (ic), FALSE))
+           continue;
+
+         /* whew !! that was a lot of work to find the definition */
+         /* create an induction object */
+         indIc = newiCode ('=', NULL, IC_RESULT (ic));
+         indIc->lineno = ic->lineno;
+         IC_RESULT (indIc) = operandFromOperand (IC_RIGHT (ic));
+         IC_RESULT (indIc)->isaddr = 0;
+         OP_SYMBOL (IC_RESULT (indIc))->isind = 1;
+         ip = newInduction (IC_RIGHT (ic), dic->op, litValue, indIc, NULL);
+
+         /* replace the inducted variable by the iTemp */
+         replaceSymBySym (loopReg->regBlocks, IC_RESULT (ic), IC_RIGHT (ic));
+
+         /* if it has only one exit then remove it from here
+            and put it in the exit block */
+         nexits = elementsInSet (loopReg->exits);
+         if (nexits == 1)
+           {
+             eBBlock *exit = setFirstItem (loopReg->exits);
+
+             /* if it is the same block then there is no
+                need to move it about */
+             if (exit != lBlock)
+               {
+                 iCode *saveic = ic->prev;
+                 /* remove it */
+                 remiCodeFromeBBlock (lBlock, ic);
+                 /* clear the definition */
+                 bitVectUnSetBit (lBlock->defSet, ic->key);
+                 /* add it to the exit */
+                 addiCodeToeBBlock (exit, ic, NULL);
+                 /* set the definition bit */
+                 exit->defSet = bitVectSetBit (exit->defSet, ic->key);
+                 ic = saveic;
                }
            }
-           
-           /* if the number of exits is greater than one then
-              we use another trick ; we will create an intersection
-              of succesors of the exits, then take those that are not
-              part of the loop and have dfNumber greater loop entry
-              and insert a new definition in them */
-           if ( nexits > 1) {
-
-               bitVect *loopSuccs = intersectLoopSucc (loopReg->exits,ebbs);
-               
-               /* loopSuccs now contains intersection
-                  of all the loops successors */
-               if (loopSuccs) {
-                   int i;
-                   for (i = 0 ; i < loopSuccs->size; i++) {
-                       if (bitVectBitValue(loopSuccs,i)) {
-
-                           eBBlock *eblock = ebbs[i];
-                           
-                           /* if the successor does not belong to the loop
-                              and will be executed after the loop : then
-                              add a definition to the block */
-                           if ( !isinSet(loopReg->regBlocks,eblock) &&
-                                eblock->dfnum > loopReg->entry->dfnum) {
-                               /* create the definition */
-                               iCode *newic = newiCode('=',NULL,
-                                                       operandFromOperand(IC_RIGHT(ic)));
-                               IC_RESULT(newic) = operandFromOperand(IC_RESULT(ic));
-                               OP_DEFS(IC_RESULT(newic)) = 
-                                   bitVectSetBit(OP_DEFS(IC_RESULT(newic)),newic->key);
-                               OP_USES(IC_RIGHT(newic)) = 
-                                   bitVectSetBit(OP_USES(IC_RIGHT(newic)),newic->key);
-                               /* and add it */
-                               if (eblock->sch && eblock->sch->op == LABEL)
-                                   addiCodeToeBBlock(eblock,newic,eblock->sch->next);
-                               else
-                                   addiCodeToeBBlock(eblock,newic,eblock->sch);
-                               /* set the definition bit */
-                               eblock->defSet = bitVectSetBit(eblock->defSet,ic->key); 
+
+         /* if the number of exits is greater than one then
+            we use another trick ; we will create an intersection
+            of succesors of the exits, then take those that are not
+            part of the loop and have dfNumber greater loop entry
+            and insert a new definition in them */
+         if (nexits > 1)
+           {
+
+             bitVect *loopSuccs = intersectLoopSucc (loopReg->exits, ebbs);
+
+             /* loopSuccs now contains intersection
+                of all the loops successors */
+             if (loopSuccs)
+               {
+                 int i;
+                 for (i = 0; i < loopSuccs->size; i++)
+                   {
+                     if (bitVectBitValue (loopSuccs, i))
+                       {
+
+                         eBBlock *eblock = ebbs[i];
+
+                         /* if the successor does not belong to the loop
+                            and will be executed after the loop : then
+                            add a definition to the block */
+                         if (!isinSet (loopReg->regBlocks, eblock) &&
+                             eblock->dfnum > loopReg->entry->dfnum)
+                           {
+                             /* create the definition */
+                             iCode *newic = newiCode ('=', NULL,
+                                       operandFromOperand (IC_RIGHT (ic)));
+                             IC_RESULT (newic) = operandFromOperand (IC_RESULT (ic));
+                             OP_DEFS(IC_RESULT (newic))=
+                               bitVectSetBit (OP_DEFS (IC_RESULT (newic)), newic->key);
+                             OP_USES(IC_RIGHT (newic))=
+                               bitVectSetBit (OP_USES (IC_RIGHT (newic)), newic->key);
+                             /* and add it */
+                             if (eblock->sch && eblock->sch->op == LABEL)
+                               addiCodeToeBBlock (eblock, newic, eblock->sch->next);
+                             else
+                               addiCodeToeBBlock (eblock, newic, eblock->sch);
+                             /* set the definition bit */
+                             eblock->defSet = bitVectSetBit (eblock->defSet, ic->key);
                            }
                        }
                    }
                }
            }
-              
-           addSet (&indVars,ip);
+
+         addSet (&indVars, ip);
        }
 
-    } /* end of all blocks for basic induction variables */
-    
-    return indVars;
+    }                          /* end of all blocks for basic induction variables */
+
+  return indVars;
 }
-    
+
 /*-----------------------------------------------------------------*/
 /* loopInduction - remove induction variables from a loop          */
 /*-----------------------------------------------------------------*/
- int loopInduction( region *loopReg, eBBlock **ebbs, int count)
-{   
-    int change = 0 ;
-    eBBlock *lBlock, *lastBlock = NULL;
-    set *indVars = NULL ;
-    set *basicInd=NULL ;
-
-    if (loopReg->entry->preHeader == NULL)
-       return 0;
-
-    /* we first determine the basic Induction variables */
-    basicInd = setFromSet(indVars = basicInduction(loopReg, ebbs,count));
-
-    /* find other induction variables : by other we mean definitions of */
-    /* the form x := y (* | / ) <constant> .. we will move  this one to */
-    /* beginning of the loop and reduce strength i.e. replace with +/-  */
-    /* these expensive expressions: OH! and y must be induction too     */
-    for ( lBlock = setFirstItem(loopReg->regBlocks), lastBlock = lBlock; 
-         lBlock && indVars;
-         lBlock = setNextItem(loopReg->regBlocks)) {
-       
-       iCode *ic, *indIc;
-       induction *ip;
-
-       /* last block is the one with the highest block 
-          number */
-       if (lastBlock->bbnum < lBlock->bbnum )
-           lastBlock = lBlock;
-
-       for ( ic = lBlock->sch ; ic ; ic = ic->next ) {
-           operand *aSym ;       
-           unsigned long litVal ;
-           int lr = 0;
-
-           /* consider only * & / */
-           if (ic->op != '*' && ic->op != '/')
-               continue ;
-
-           /* if the result has more definitions then */
-           if (assignmentsToSym(loopReg->regBlocks,IC_RESULT(ic)) > 1)
-               continue ;
-
-           /* check if the operands are what we want */
-           /* i.e. one of them an symbol the other a literal */
-           if (! ( (IS_SYMOP(IC_LEFT(ic)) && IS_OP_LITERAL(IC_RIGHT(ic))) ||
-                   (IS_OP_LITERAL(IC_LEFT(ic)) && IS_SYMOP(IC_RIGHT(ic))) ))
-               continue ;
-
-           aSym = (IS_SYMOP(IC_LEFT(ic)) ? 
-                   (lr = 1, litVal = operandLitValue(IC_RIGHT(ic)), IC_LEFT(ic) ) : 
-                   (litVal= operandLitValue(IC_LEFT(ic)), IC_RIGHT(ic) ) ) ;     
-
-           ip = NULL ;
-           /* check if this is an induction variable */
-           if (! applyToSetFTrue (basicInd,findInduction,aSym,&ip))
-               continue ;                  
-           
-           /* if this is a division then the remainder should be zero 
-              for it to be inducted */
-           if (ic->op == '/' && (ip->cval % litVal))
-               continue ;
-
-           /* create the iCode to be placed in the loop header */
-           /* and create the induction object */
-           
-           /* create an instruction */
-           /* this will be put on the loop header */
-           indIc = newiCode(ic->op,
-                            operandFromOperand(aSym),
-                            operandFromLit(litVal));
-           indIc->lineno = ic->lineno;
-           IC_RESULT(indIc) = operandFromOperand(IC_RESULT(ic));
-           OP_SYMBOL(IC_RESULT(indIc))->isind = 1;
-
-           /* keep track of the inductions */
-           litVal = (ic->op == '*' ? (litVal * ip->cval) :
-                     (ip->cval / litVal));         
-
-           addSet (&indVars,
-                   newInduction (IC_RESULT(ic),ip->op,litVal,indIc,NULL)); 
-
-           /* now change this instruction */
-           ic->op = ip->op;
-           if (lr) {
-               IC_LEFT(ic) = operandFromOperand(IC_RESULT(ic));
-               IC_RIGHT(ic) = operandFromLit(litVal);
-           } else {
-               IC_RIGHT(ic) = operandFromOperand(IC_RESULT(ic));
-               IC_LEFT(ic) = operandFromLit(litVal);
+int 
+loopInduction (region * loopReg, eBBlock ** ebbs, int count)
+{
+  int change = 0;
+  eBBlock *lBlock, *lastBlock = NULL;
+  set *indVars = NULL;
+  set *basicInd = NULL;
+
+  if (loopReg->entry->preHeader == NULL)
+    return 0;
+
+  /* we first determine the basic Induction variables */
+  basicInd = setFromSet (indVars = basicInduction (loopReg, ebbs, count));
+
+  /* find other induction variables : by other we mean definitions of */
+  /* the form x := y (* | / ) <constant> .. we will move  this one to */
+  /* beginning of the loop and reduce strength i.e. replace with +/-  */
+  /* these expensive expressions: OH! and y must be induction too     */
+  for (lBlock = setFirstItem (loopReg->regBlocks), lastBlock = lBlock;
+       lBlock && indVars;
+       lBlock = setNextItem (loopReg->regBlocks))
+    {
+
+      iCode *ic, *indIc;
+      induction *ip;
+
+      /* last block is the one with the highest block
+         number */
+      if (lastBlock->bbnum < lBlock->bbnum)
+       lastBlock = lBlock;
+
+      for (ic = lBlock->sch; ic; ic = ic->next)
+       {
+         operand *aSym;
+         unsigned long litVal;
+         int lr = 0;
+
+         /* consider only * & / */
+         if (ic->op != '*' && ic->op != '/')
+           continue;
+
+         /* if the result has more definitions then */
+         if (assignmentsToSym (loopReg->regBlocks, IC_RESULT (ic)) > 1)
+           continue;
+
+         /* check if the operands are what we want */
+         /* i.e. one of them an symbol the other a literal */
+         if (!((IS_SYMOP (IC_LEFT (ic)) && IS_OP_LITERAL (IC_RIGHT (ic))) ||
+               (IS_OP_LITERAL (IC_LEFT (ic)) && IS_SYMOP (IC_RIGHT (ic)))))
+           continue;
+
+         aSym = (IS_SYMOP (IC_LEFT (ic)) ?
+         (lr = 1, litVal = (unsigned long) operandLitValue (IC_RIGHT (ic)), IC_LEFT (ic)) :
+                 (litVal = (unsigned long) operandLitValue (IC_LEFT (ic)), IC_RIGHT (ic)));
+
+         ip = NULL;
+         /* check if this is an induction variable */
+         if (!applyToSetFTrue (basicInd, findInduction, aSym, &ip))
+           continue;
+
+         /* ask port for size not worth if native instruction
+            exist for multiply & divide */
+         if (getSize (operandType (IC_LEFT (ic))) <= (unsigned long) port->support.muldiv ||
+         getSize (operandType (IC_RIGHT (ic))) <= (unsigned long) port->support.muldiv)
+           continue;
+
+         /* if this is a division then the remainder should be zero
+            for it to be inducted */
+         if (ic->op == '/' && (ip->cval % litVal))
+           continue;
+
+         /* create the iCode to be placed in the loop header */
+         /* and create the induction object */
+
+         /* create an instruction */
+         /* this will be put on the loop header */
+         indIc = newiCode (ic->op,
+                           operandFromOperand (aSym),
+                           operandFromLit (litVal));
+         indIc->lineno = ic->lineno;
+         IC_RESULT (indIc) = operandFromOperand (IC_RESULT (ic));
+         OP_SYMBOL (IC_RESULT (indIc))->isind = 1;
+
+         /* keep track of the inductions */
+         litVal = (ic->op == '*' ? (litVal * ip->cval) :
+                   (ip->cval / litVal));
+
+         addSet (&indVars,
+               newInduction (IC_RESULT (ic), ip->op, litVal, indIc, NULL));
+
+         /* now change this instruction */
+         ic->op = ip->op;
+         if (lr)
+           {
+             IC_LEFT (ic) = operandFromOperand (IC_RESULT (ic));
+             IC_RIGHT (ic) = operandFromLit (litVal);
            }
-           
-           /* we need somemore initialisation code */
-           /* we subtract the litVal from itself if increment */
-           if ( ic->op == '+' ) {
-               indIc = newiCode('-',
-                                operandFromOperand(IC_RESULT(ic)),
-                                operandFromLit(litVal));
-               indIc->lineno = ic->lineno;
-               IC_RESULT(indIc) = operandFromOperand(IC_RESULT(ic));
-               
-               addSet (&indVars,
-                       newInduction (IC_RESULT(ic),ip->op,litVal,indIc,NULL)); 
+         else
+           {
+             IC_RIGHT (ic) = operandFromOperand (IC_RESULT (ic));
+             IC_LEFT (ic) = operandFromLit (litVal);
+           }
+
+         /* we need somemore initialisation code */
+         /* we subtract the litVal from itself if increment */
+         if (ic->op == '+')
+           {
+             indIc = newiCode ('-',
+                               operandFromOperand (IC_RESULT (ic)),
+                               operandFromLit (litVal));
+             indIc->lineno = ic->lineno;
+             IC_RESULT (indIc) = operandFromOperand (IC_RESULT (ic));
+
+             addSet (&indVars,
+               newInduction (IC_RESULT (ic), ip->op, litVal, indIc, NULL));
            }
        }
-    }    
-
-    /* if we have some induction variables then */
-    if ( indVars ) {
-       eBBlock *preHdr = loopReg->entry->preHeader ;
-       iCode *icFirst = NULL , *icLast = NULL ;
-       induction *ip;  
-       bitVect *indVect = NULL;
-       
-       /* create an iCode chain from it */
-       for (ip = setFirstItem(indVars); 
-            ip ; 
-            ip = setNextItem(indVars)) {
-
-           indVect = bitVectSetBit(indVect,ip->ic->key);
-           ip->ic->lineno = preHdr->ech->lineno;
-           if (!icFirst)
-               icFirst = ip->ic;
-           if (icLast) {
-               icLast->next = ip->ic;
-               ip->ic->prev = icLast;
-               icLast = ip->ic ;
-           } else 
-               icLast = ip->ic;        
-           change++ ;      
+    }
+
+  /* if we have some induction variables then */
+  if (indVars)
+    {
+      eBBlock *preHdr = loopReg->entry->preHeader;
+      iCode *icFirst = NULL, *icLast = NULL;
+      induction *ip;
+      bitVect *indVect = NULL;
+
+      /* create an iCode chain from it */
+      for (ip = setFirstItem (indVars);
+          ip;
+          ip = setNextItem (indVars))
+       {
+
+         indVect = bitVectSetBit (indVect, ip->ic->key);
+         ip->ic->lineno = preHdr->ech->lineno;
+         if (!icFirst)
+           icFirst = ip->ic;
+         if (icLast)
+           {
+             icLast->next = ip->ic;
+             ip->ic->prev = icLast;
+             icLast = ip->ic;
+           }
+         else
+           icLast = ip->ic;
+         change++;
        }
-       
-       /* add the instruction chain to the end of the */
-       /* preheader for this loop                     */
-       preHdr->ech->next = icFirst ;
-       icFirst->prev = preHdr->ech ;
-       preHdr->ech = icLast;
-       icLast->next = NULL;            
-       
-       /* add the induction variable vector to the last
-          block in the loop */
-       lastBlock->isLastInLoop = 1;
-       lastBlock->linds = indVect;
+
+      /* add the instruction chain to the end of the */
+      /* preheader for this loop                     */
+      preHdr->ech->next = icFirst;
+      icFirst->prev = preHdr->ech;
+      preHdr->ech = icLast;
+      icLast->next = NULL;
+
+      /* add the induction variable vector to the last
+         block in the loop */
+      lastBlock->isLastInLoop = 1;
+      lastBlock->linds = bitVectUnion(lastBlock->linds,indVect);
     }
-    
-    setToNull ((void **)&indVars);
-    return change ;    
+
+  setToNull ((void **) &indVars);
+  return change;
 }
 
 /*-----------------------------------------------------------------*/
 /* mergeRegions - will merge region with same entry point           */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(mergeRegions)
+DEFSETFUNC (mergeRegions)
 {
-    region *theLoop = item;
-    V_ARG(set*,allRegion) ;
-    region *lp ;
-    
-    /* if this has already been merged then do nothing */
-    if (theLoop->merged)
-       return 0;
-
-    /* go thru all the region and check if any of them have the */
-    /* entryPoint as the Loop                                  */
-    for (lp = setFirstItem(allRegion); lp ; lp = setNextItem(allRegion)) {
-
-       if (lp == theLoop)
-           continue ;
-
-       if (lp->entry == theLoop->entry) {
-           theLoop->regBlocks = unionSets (theLoop->regBlocks,
-                                            lp->regBlocks,THROW_BOTH);
-           lp->merged = 1;
+  region *theLoop = item;
+  V_ARG (set *, allRegion);
+  region *lp;
+
+  /* if this has already been merged then do nothing */
+  if (theLoop->merged)
+    return 0;
+
+  /* go thru all the region and check if any of them have the */
+  /* entryPoint as the Loop                                  */
+  for (lp = setFirstItem (allRegion); lp; lp = setNextItem (allRegion))
+    {
+
+      if (lp == theLoop)
+       continue;
+
+      if (lp->entry == theLoop->entry)
+       {
+         theLoop->regBlocks = unionSets (theLoop->regBlocks,
+                                         lp->regBlocks, THROW_DEST);
+         lp->merged = 1;
        }
     }
 
-    return 1;
+  return 1;
 }
 
 /*-----------------------------------------------------------------*/
 /* ifMerged - return 1 if the merge flag is 1                      */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(ifMerged)
+DEFSETFUNC (ifMerged)
 {
-    region *lp = item;
+  region *lp = item;
 
-    return lp->merged ;
+  return lp->merged;
 }
 
 /*-----------------------------------------------------------------*/
 /* mergeInnerLoops - will merge into body when entry is present    */
 /*-----------------------------------------------------------------*/
-DEFSETFUNC(mergeInnerLoops)
+DEFSETFUNC (mergeInnerLoops)
 {
-    region *theLoop = item;
-    V_ARG(set *,allRegion);
-    V_ARG(int *,maxDepth);
-    region *lp;
-    
-    /* check if the entry point is present in the body of any */
-    /* loop then put the body of this loop into the outer loop*/
-    for (lp = setFirstItem(allRegion); lp ; lp = setNextItem(allRegion)) {
-
-       if ( lp == theLoop )
-           continue ;
-
-       if (isinSet(lp->regBlocks, theLoop->entry)) {
-           lp->containsLoops += theLoop->containsLoops + 1 ;
-           if ( lp->containsLoops > (*maxDepth))
-               *maxDepth = lp->containsLoops;
-
-           lp->regBlocks = unionSets (lp->regBlocks,
-                                       theLoop->regBlocks,THROW_DEST);
+  region *theLoop = item;
+  V_ARG (set *, allRegion);
+  V_ARG (int *, maxDepth);
+  region *lp;
+
+  /* check if the entry point is present in the body of any */
+  /* loop then put the body of this loop into the outer loop */
+  for (lp = setFirstItem (allRegion); lp; lp = setNextItem (allRegion))
+    {
+
+      if (lp == theLoop)
+       continue;
+
+      if (isinSet (lp->regBlocks, theLoop->entry))
+       {
+         lp->containsLoops += theLoop->containsLoops + 1;
+         if (lp->containsLoops > (*maxDepth))
+           *maxDepth = lp->containsLoops;
+
+         lp->regBlocks = unionSets (lp->regBlocks,
+                                    theLoop->regBlocks, THROW_DEST);
        }
     }
 
-    return 1;
+  return 1;
 }
 
 
 /*-----------------------------------------------------------------*/
 /* createLoopRegions - will detect and create a set of natural loops */
 /*-----------------------------------------------------------------*/
-hTab *createLoopRegions (eBBlock **ebbs , int count  )
-{   
-    set *allRegion  = NULL; /* set of all loops */       
-    hTab *orderedLoops = NULL ;
-    set *bEdges = NULL;
-    int maxDepth = 0;
-    region *lp;
-
-    /* get all the back edges in the graph */
-    if (! applyToSet(graphEdges,backEdges,&bEdges))
-       return 0 ; /* found no loops */
-
-    /* for each of these back edges get the blocks that */
-    /* constitute the loops                             */
-    applyToSet(bEdges,createLoop,&allRegion);
-
-    /* now we will create regions from these loops               */
-    /* loops with the same entry points are considered to be the */
-    /* same loop & they are merged. If the entry point of a loop */
-    /* is found in the body of another loop then , all the blocks*/
-    /* in that loop are added to the loops containing the header */    
-    applyToSet(allRegion, mergeRegions , allRegion);
-
-    /* delete those already merged */
-    deleteItemIf (&allRegion, ifMerged);
-
-    applyToSet(allRegion, mergeInnerLoops, allRegion, &maxDepth);
-    maxDepth++;
-    /* now create all the exits .. also */
-    /* create an ordered set of loops   */
-    /* i.e. we process loops in the inner to outer order */    
-    for (lp = setFirstItem(allRegion) ; lp ; lp = setNextItem(allRegion)) {
-       applyToSet (lp->regBlocks,addToExitsMarkDepth,
-                   lp->regBlocks,&lp->exits,
-                   (maxDepth - lp->containsLoops),lp);
-       
-       hTabAddItem (&orderedLoops,lp->containsLoops,lp);       
+hTab *
+createLoopRegions (eBBlock ** ebbs, int count)
+{
+  set *allRegion = NULL;       /* set of all loops */
+  hTab *orderedLoops = NULL;
+  set *bEdges = NULL;
+  int maxDepth = 0;
+  region *lp;
+
+  /* get all the back edges in the graph */
+  if (!applyToSet (graphEdges, backEdges, &bEdges))
+    return 0;                  /* found no loops */
+
+  /* for each of these back edges get the blocks that */
+  /* constitute the loops                             */
+  applyToSet (bEdges, createLoop, &allRegion, ebbs,count);
+
+  /* now we will create regions from these loops               */
+  /* loops with the same entry points are considered to be the */
+  /* same loop & they are merged. If the entry point of a loop */
+  /* is found in the body of another loop then , all the blocks */
+  /* in that loop are added to the loops containing the header */
+  applyToSet (allRegion, mergeRegions, allRegion);
+
+  /* delete those already merged */
+  deleteItemIf (&allRegion, ifMerged);
+
+  applyToSet (allRegion, mergeInnerLoops, allRegion, &maxDepth);
+  maxDepth++;
+
+  /* now create all the exits .. also */
+  /* create an ordered set of loops   */
+  /* i.e. we process loops in the inner to outer order */
+  for (lp = setFirstItem (allRegion); lp; lp = setNextItem (allRegion))
+    {
+      applyToSet (lp->regBlocks, addToExitsMarkDepth,
+                 lp->regBlocks, &lp->exits,
+                 (maxDepth - lp->containsLoops), lp);
+
+      hTabAddItem (&orderedLoops, lp->containsLoops, lp);
 
     }
-    return orderedLoops ;
+  return orderedLoops;
 }
 
 /*-----------------------------------------------------------------*/
 /* loopOptimizations - identify region & remove invariants & ind   */
 /*-----------------------------------------------------------------*/
-int loopOptimizations (hTab *orderedLoops, eBBlock **ebbs, int count)
+int 
+loopOptimizations (hTab * orderedLoops, eBBlock ** ebbs, int count)
 {
-    region *lp ;
-    int change = 0 ;
-    int k;
-
-
-    /* if no loop optimizations requested */
-    if (! optimize.loopInvariant &&
-       ! optimize.loopInduction )
-       return 0;
-
-    /* now we process the loops inner to outer order */
-    /* this is essential to maintain data flow information */
-    /* the other choice is an ugly iteration for the depth */
-    /* of the loops would hate that */
-    for ( lp = hTabFirstItem(orderedLoops,&k); lp ; 
-         lp = hTabNextItem(orderedLoops,&k)) {
-       
-       if (optimize.loopInvariant) 
-           change += loopInvariants(lp, ebbs, count);       
-       
-       if (optimize.loopInduction)
-           change += loopInduction(lp, ebbs, count);
+  region *lp;
+  int change = 0;
+  int k;
+
+  /* if no loop optimizations requested */
+  if (!optimize.loopInvariant &&
+      !optimize.loopInduction)
+    return 0;
+
+  /* now we process the loops inner to outer order */
+  /* this is essential to maintain data flow information */
+  /* the other choice is an ugly iteration for the depth */
+  /* of the loops would hate that */
+  for (lp = hTabFirstItem (orderedLoops, &k); lp;
+       lp = hTabNextItem (orderedLoops, &k))
+    {
+
+      if (optimize.loopInvariant)
+       change += loopInvariants (lp, ebbs, count);
+
+      if (optimize.loopInduction)
+       change += loopInduction (lp, ebbs, count);
     }
-        
-    return change;
+
+  return change;
 }