* as/z80/z80mch.c: fixed bug #1704376: missing as-z80 errors
[fw/sdcc] / src / SDCCloop.c
index f12fe1b35f4065a1b1bc699567615304875a88a9..6400d1a4f411f7b92dae13451ab6cf31d93bb52a 100644 (file)
@@ -33,9 +33,9 @@ STACK_DCL (regionStack, eBBlock *, MAX_NEST_LEVEL * 10);
 /*-----------------------------------------------------------------*/
 /* newInduction - creates a new induction variable                 */
 /*-----------------------------------------------------------------*/
-induction *
+static induction *
 newInduction (operand * sym, unsigned int op,
-             long constVal, iCode * ic, operand * asym)
+              long constVal, iCode * ic, operand * asym)
 {
   induction *ip;
 
@@ -53,7 +53,7 @@ newInduction (operand * sym, unsigned int op,
 /*-----------------------------------------------------------------*/
 /* newRegion - allocate & returns a loop structure                 */
 /*-----------------------------------------------------------------*/
-region *
+static region *
 newRegion ()
 {
   region *lp;
@@ -64,9 +64,11 @@ newRegion ()
 }
 
 
+#if 0
 /*-----------------------------------------------------------------*/
 /* pinduction - prints induction                                   */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (pinduction)
 {
   induction *ip = item;
@@ -83,6 +85,7 @@ DEFSETFUNC (pinduction)
 /*-----------------------------------------------------------------*/
 /* pregion - prints loop information                                */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (pregion)
 {
   region *lp = item;
@@ -99,10 +102,12 @@ DEFSETFUNC (pregion)
   printf ("\n");
   return 0;
 }
+#endif
 
 /*-----------------------------------------------------------------*/
 /* backEdges - returns a list of back edges                        */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (backEdges)
 {
   edge *ep = item;
@@ -138,13 +143,12 @@ intersectLoopSucc (set * lexits, eBBlock ** ebbs)
        exit = setNextItem (lexits))
     {
       succVect = bitVectIntersect (succVect,
-                                  exit->succVect);
+                                   exit->succVect);
     }
 
   return succVect;
 }
 
-
 /*-----------------------------------------------------------------*/
 /* loopInsert will insert a block into the loop set                */
 /*-----------------------------------------------------------------*/
@@ -161,6 +165,7 @@ loopInsert (set ** regionSet, eBBlock * block)
 /*-----------------------------------------------------------------*/
 /* insertIntoLoop - insert item into loop                          */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (insertIntoLoop)
 {
   eBBlock *ebp = item;
@@ -173,6 +178,7 @@ DEFSETFUNC (insertIntoLoop)
 /*-----------------------------------------------------------------*/
 /* isNotInBlocks - will return 1 if not is blocks                  */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (isNotInBlocks)
 {
   eBBlock *ebp = item;
@@ -184,11 +190,12 @@ DEFSETFUNC (isNotInBlocks)
   return 0;
 }
 
+#if 0
 /*-----------------------------------------------------------------*/
 /* hasIncomingDefs - has definitions coming into the loop. i.e.    */
 /* check to see if the preheaders outDefs has any definitions      */
 /*-----------------------------------------------------------------*/
-int
+static int
 hasIncomingDefs (region * lreg, operand * op)
 {
   eBBlock *preHdr = lreg->entry->preHeader;
@@ -202,7 +209,7 @@ hasIncomingDefs (region * lreg, operand * op)
 /* findLoopEndSeq - will return the sequence number of the last    */
 /* iCode with the maximum dfNumber in the region                   */
 /*-----------------------------------------------------------------*/
-int
+static int
 findLoopEndSeq (region * lreg)
 {
   eBBlock *block;
@@ -212,16 +219,18 @@ findLoopEndSeq (region * lreg)
        block = setNextItem (lreg->regBlocks))
     {
       if (block != lblock && block->lSeq > lblock->lSeq)
-       lblock = block;
+        lblock = block;
     }
 
   return lblock->lSeq;
 }
+#endif
 
 /*-----------------------------------------------------------------*/
 /* addToExitsMarkDepth - will add the the exitSet all blocks that  */
 /* have exits, will also update the depth field in the blocks      */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (addToExitsMarkDepth)
 {
   eBBlock *ebp = item;
@@ -254,6 +263,7 @@ DEFSETFUNC (addToExitsMarkDepth)
 /*-----------------------------------------------------------------*/
 /* createLoop - will create a set of region                        */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (createLoop)
 {
   edge *ep = item;
@@ -274,7 +284,7 @@ DEFSETFUNC (createLoop)
       block = STACK_POP (regionStack);
       /* if block != entry */
       if (block != ep->to)
-       applyToSet (block->predList, insertIntoLoop, &aloop->regBlocks);
+        applyToSet (block->predList, insertIntoLoop, &aloop->regBlocks);
     }
 
   aloop->entry = ep->to;
@@ -287,6 +297,7 @@ DEFSETFUNC (createLoop)
 /*-----------------------------------------------------------------*/
 /* dominatedBy - will return 1 if item is dominated by block       */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (dominatedBy)
 {
   eBBlock *ebp = item;
@@ -298,6 +309,7 @@ DEFSETFUNC (dominatedBy)
 /*-----------------------------------------------------------------*/
 /* addDefInExprs - adds an expression into the inexpressions       */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (addDefInExprs)
 {
   eBBlock *ebp = item;
@@ -312,7 +324,7 @@ DEFSETFUNC (addDefInExprs)
 /*-----------------------------------------------------------------*/
 /* assignmentsToSym - for a set of blocks determine # time assigned */
 /*-----------------------------------------------------------------*/
-int
+static int
 assignmentsToSym (set * sset, operand * sym)
 {
   eBBlock *ebp;
@@ -322,13 +334,11 @@ assignmentsToSym (set * sset, operand * sym)
   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;
@@ -338,7 +348,7 @@ assignmentsToSym (set * sset, operand * sym)
 /*-----------------------------------------------------------------*/
 /* isOperandInvariant - determines if an operand is an invariant   */
 /*-----------------------------------------------------------------*/
-int
+static int
 isOperandInvariant (operand * op, region * theLoop, set * lInvars)
 {
   int opin = 0;
@@ -350,19 +360,19 @@ isOperandInvariant (operand * op, region * theLoop, set * lInvars)
   if (op)
     {
       if (IS_OP_LITERAL (op))
-       opin = 1;
+        opin = 1;
       else if (IS_SYMOP (op) &&
-              OP_SYMBOL (op)->addrtaken)
-       opin = 0;
+               OP_SYMBOL (op)->addrtaken)
+        opin = 0;
       else if (ifDefSymIs (theLoop->entry->inExprs, op))
-       opin = 1;
+        opin = 1;
       else if (ifDefSymIs (lInvars, op))
-       opin = 1;
+        opin = 1;
       else if (IS_SYMOP (op) &&
-              !IS_OP_GLOBAL (op) &&
-              !IS_OP_VOLATILE (op) &&
-              assignmentsToSym (theLoop->regBlocks, op) == 0)
-       opin = 1;
+               !IS_OP_GLOBAL (op) &&
+               !IS_OP_VOLATILE (op) &&
+               assignmentsToSym (theLoop->regBlocks, op) == 0)
+        opin = 1;
     }
   else
     opin++;
@@ -373,6 +383,7 @@ isOperandInvariant (operand * op, region * theLoop, set * lInvars)
 /*-----------------------------------------------------------------*/
 /* pointerAssigned - will return 1 if pointer set found            */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (pointerAssigned)
 {
   eBBlock *ebp = item;
@@ -399,6 +410,7 @@ DEFSETFUNC (pointerAssigned)
 /*-----------------------------------------------------------------*/
 /* hasNonPtrUse - returns true if operand has non pointer usage    */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (hasNonPtrUse)
 {
   eBBlock *ebp = item;
@@ -415,7 +427,7 @@ DEFSETFUNC (hasNonPtrUse)
 /*-----------------------------------------------------------------*/
 /* loopInvariants - takes loop invariants out of region            */
 /*-----------------------------------------------------------------*/
-int
+static int
 loopInvariants (region * theLoop, ebbIndex * ebbi)
 {
   eBBlock ** ebbs = ebbi->dfOrder;
@@ -444,144 +456,159 @@ loopInvariants (region * theLoop, ebbIndex * ebbi)
 
       /* mark the dominates all exits flag */
       domsAllExits = (applyToSet (theLoop->exits, dominatedBy, lBlock) ==
-                     elementsInSet (theLoop->exits));
+                      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++;
-       }
-      }
+      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;
-
-         /* iTemp assignment from a literal may be invariant, but it
-            will needlessly increase register pressure if the
-            iCode(s) that use this iTemp are not also invariant */
-         if (ic->op=='=' && IS_ITEMP (IC_RESULT (ic))
-             && IS_OP_LITERAL (IC_RIGHT (ic)))
-           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;
+        {
+
+          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;
+                }
+
+              /* Bug 1717943, 
+               * if this is an assignment to a global */
+              if (ic->op=='=' && isOperandGlobal(IC_RESULT(ic)))
+                {
+                  continue;
+                }
+            }
+
+          if (SKIP_IC (ic) || POINTER_SET (ic) || ic->op == IFX)
+            continue;
+
+          /* iTemp assignment from a literal may be invariant, but it
+             will needlessly increase register pressure if the
+             iCode(s) that use this iTemp are not also invariant */
+          if (ic->op=='=' && IS_ITEMP (IC_RESULT (ic))
+              && IS_OP_LITERAL (IC_RIGHT (ic)))
+            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 */
+                  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 exits */
-             /* 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;
-               }
-
-             /* now make sure this is the only definition */
-             for (sBlock = setFirstItem (lSet); sBlock;
-                  sBlock = setNextItem (lSet))
-               {
+          /* 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 exits */
+              /* 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;
+                }
+
+              /* now make sure this is the only definition */
+              for (sBlock = setFirstItem (lSet); sBlock;
+                   sBlock = setNextItem (lSet))
+                {
                   iCode *ic2;
                   int used;
                   int defDominates;
 
-                 /* 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;
+                  /* 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;
 
                   /* Check that this definition is not assigned */
                   /* any other value in this block. Also check */
@@ -618,37 +645,37 @@ loopInvariants (region * theLoop, ebbIndex * ebbi)
                     }
                   if (ic2) /* found another definition or a usage before the definition */
                     break;
-               }
+                }
+
+              if (sBlock)
+                continue;       /* another definition present in the block */
 
-             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;
+              /* 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                */
+              /* 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;
               SPIL_LOC (IC_RESULT (ic)) = NULL;
-             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, ebbi);
-             addSet (&lInvars, ivar);
-           }
-       }
-    }                          /* for all loop blocks */
+              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, ebbi);
+              addSet (&lInvars, ivar);
+            }
+        }
+    }                           /* for all loop blocks */
 
   /* if we have some invariants then */
   if (lInvars)
@@ -659,28 +686,29 @@ loopInvariants (region * theLoop, ebbIndex * ebbi)
 
       /* 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++;
-       }
+        {
+
+          /* 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->filename = preHdr->ech->filename;
+          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
@@ -697,7 +725,7 @@ loopInvariants (region * theLoop, ebbIndex * ebbi)
 /*-----------------------------------------------------------------*/
 /* addressTaken - returns true if the symbol is found in the addrof */
 /*-----------------------------------------------------------------*/
-int
+static int
 addressTaken (set * sset, operand * sym)
 {
   set *loop;
@@ -709,11 +737,11 @@ addressTaken (set * sset, operand * sym)
       ebp = loop->item;
       loop2 = ebp->addrOf;
       while (loop2)
-       {
-         if (isOperandEqual ((operand *) loop2->item, sym))
-           return 1;
-         loop2 = loop2->next;
-       }
+        {
+          if (isOperandEqual ((operand *) loop2->item, sym))
+            return 1;
+          loop2 = loop2->next;
+        }
     }
 
   return 0;
@@ -723,6 +751,7 @@ addressTaken (set * sset, operand * sym)
 /*-----------------------------------------------------------------*/
 /* findInduction :- returns 1 & the item if the induction is found */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (findInduction)
 {
   induction *ip = item;
@@ -741,7 +770,7 @@ DEFSETFUNC (findInduction)
 /*-----------------------------------------------------------------*/
 /* findDefInRegion - finds the definition within the region        */
 /*-----------------------------------------------------------------*/
-iCode *
+static iCode *
 findDefInRegion (set * regBlocks, operand * defOp, eBBlock ** owner)
 {
   eBBlock *lBlock;
@@ -753,28 +782,210 @@ findDefInRegion (set * regBlocks, operand * defOp, eBBlock ** owner)
 
       /* 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;
-             }
-       }
+        {
+          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;
 }
 
+/*-----------------------------------------------------------------*/
+/* addPostLoopBlock - add a ebblock before the successors of the   */
+/*                    loop exits                                   */
+/*-----------------------------------------------------------------*/
+static void
+addPostLoopBlock (region *loopReg, ebbIndex *ebbi, iCode *ic)
+{
+  bitVect *loopSuccs;
+  int i;
+
+  /* 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 (eblock),
+     insert a new predecessor postLoopBlk before them and add
+     a copy of ic in the new block. The postLoopBlk in between
+     is necessary, because the old successors of the loop exits can
+     be successors of other blocks too: see bug-136564. */
+
+  /* loopSuccs now contains intersection
+      of all the loops successors */
+  loopSuccs = intersectLoopSucc (loopReg->exits, ebbi->dfOrder);
+  if (!loopSuccs)
+    return;
+
+  for (i = 0; i < loopSuccs->size; i++)
+    {
+      eBBlock *eblock = NULL;
+      eBBlock *postLoopBlk;
+      iCode *newic;
+      int j;
+
+      if (!bitVectBitValue (loopSuccs, i))
+        continue;
+
+      /* Need to search for bbnum == i since ebbi->dfOrder is
+         sorted by dfnum; a direct index won't do.  */
+      for (j = 0; j < ebbi->count; j++)
+        if (ebbi->dfOrder[j]->bbnum == i)
+          {
+            eblock = ebbi->dfOrder[j];
+            break;
+          }
+      assert(eblock);
+
+      /* 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))
+        continue;
+      if (eblock->dfnum <= loopReg->entry->dfnum)
+        continue;
+
+      /* look for an existing loopExitBlock */
+      if (strncmp (LOOPEXITLBL,
+                    eblock->entryLabel->name,
+                    sizeof(LOOPEXITLBL) - 1) == 0)
+        {
+          /* reuse the existing one */
+          postLoopBlk = eblock;
+        }
+      else
+        {
+          /* create and insert a new eBBlock.
+              Damn, that's messy ... */
+          int i;
+          set *oldPredList;
+          eBBlock *ebpi;
+
+          postLoopBlk = neweBBlock();
+
+          /* create a loopExit-label */
+          postLoopBlk->entryLabel = newiTempLoopHeaderLabel (0);
+
+          /* increase bbnum for all blocks after
+              (and including) eblock */
+          for (i = 0; i < ebbi->count; i++)
+            {
+              if (ebbi->bbOrder[i]->bbnum >= eblock->bbnum)
+                ++ebbi->bbOrder[i]->bbnum;
+            }
+          /* insert postLoopBlk before bbnum */
+          postLoopBlk->bbnum = eblock->bbnum - 1;
+
+          ++ebbi->count;
+
+          /* these arrays need one more block, which ... */
+          ebbi->bbOrder = Safe_realloc (ebbi->bbOrder,
+                                        (ebbi->count + 1) * sizeof(eBBlock *));
+          /* ... must be initialized with 0 */
+          ebbi->bbOrder[ebbi->count] = NULL;
+          /* move the blocks up ... */
+          memmove (&ebbi->bbOrder[postLoopBlk->bbnum + 1],
+                   &ebbi->bbOrder[postLoopBlk->bbnum],
+                   (ebbi->count - postLoopBlk->bbnum - 1) * sizeof(eBBlock *));
+          /* ... and insert postLoopBlk */
+          ebbi->bbOrder[postLoopBlk->bbnum] = postLoopBlk;
+
+          /* just add postLoopBlk at the end of dfOrder,
+              computeControlFlow() will compute the new dfnum */
+          ebbi->dfOrder = Safe_realloc (ebbi->dfOrder,
+                                        (ebbi->count + 1) * sizeof(eBBlock *));
+          ebbi->dfOrder[ebbi->count] = NULL;
+          ebbi->dfOrder[ebbi->count - 1] = postLoopBlk;
+
+          /* copy loop information from eblock to postLoopBlk */
+          if (eblock->partOfLoop)
+            {
+              postLoopBlk->partOfLoop = eblock->partOfLoop;
+              /* add postLoopBlk to loop region */
+              addSetHead (&postLoopBlk->partOfLoop->regBlocks, postLoopBlk);
+            }
+
+          /* go through loop exits and replace the old exit block eblock
+              with the new postLoopBlk */
+          for (ebpi = setFirstItem (loopReg->exits);
+               ebpi;
+               ebpi = setNextItem (loopReg->exits))
+            {
+              /* replace old label with new label */
+              replaceLabel (ebpi, eblock->entryLabel, postLoopBlk->entryLabel);
+            }
+
+          /* now eblock is replaced by postLoopBlk.
+             It's possible, that eblock has an immediate predecessor
+             (with ebpi->bbnum + 1 == eblock->bbnum), which isn't part of the
+             loop. This predecessor must stay predecessor of eblock, not of
+             postLoopBlk. But now postLoopBlk is in between, therefore a GOTO
+             from this predecessor to eblock must be appended.
+             Example: bug-136564.c */
+
+          /* take all predecessors and subtract the loop exits */
+          oldPredList = subtractFromSet (eblock->predList,
+                                         loopReg->exits,
+                                         THROW_NONE);
+          for (ebpi = setFirstItem (oldPredList);
+               ebpi;
+               ebpi = setNextItem (oldPredList))
+            {
+              /* Is it an immediate predecessor (without GOTO)?
+                 All other predecessors end with a
+                 GOTO, IF, IFX or JUMPTABLE: nothing to to do */
+              if (ebpi->bbnum + 1 == postLoopBlk->bbnum)
+                {
+                  /* insert goto to old predecessor of eblock */
+                  newic = newiCodeLabelGoto (GOTO, eblock->entryLabel);
+                  addiCodeToeBBlock (ebpi, newic, NULL);
+                  break; /* got it, only one is possible */
+                }
+            }
+
+          /* set the label */
+          postLoopBlk->sch =
+          postLoopBlk->ech = newiCodeLabelGoto (LABEL, postLoopBlk->entryLabel);
+        }
+
+      /* create the definition in postLoopBlk */
+      newic = newiCode ('=', NULL, operandFromOperand (IC_RIGHT (ic)));
+      IC_RESULT (newic) = operandFromOperand (IC_RESULT (ic));
+      /* maintain data flow */
+      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 */
+      addiCodeToeBBlock (postLoopBlk, newic, NULL);
+      postLoopBlk->sch->filename =
+      postLoopBlk->ech->filename = eblock->sch->filename;
+      postLoopBlk->sch->lineno =
+      postLoopBlk->ech->lineno = eblock->sch->lineno;
+
+      /* outDefs is needed by computeControlFlow(), anything
+         else will be set up by computeControlFlow() */
+      postLoopBlk->outDefs = bitVectSetBit (postLoopBlk->outDefs, newic->key);
+    } /* for (i = 0; i < loopSuccs->size; i++) */
+
+  /* the postLoopBlk and the induction significantly
+      changed the control flow, recompute it */
+  computeControlFlow (ebbi);
+}
+
 /*-----------------------------------------------------------------*/
 /* basicInduction - finds the basic induction variables in a loop  */
 /*-----------------------------------------------------------------*/
-set *
-basicInduction (region * loopReg, eBBlock ** ebbs, int count)
+static set *
+basicInduction (region * loopReg, ebbIndex * ebbi)
 {
   eBBlock *lBlock;
   set *indVars = NULL;
@@ -789,17 +1000,17 @@ basicInduction (region * loopReg, eBBlock ** ebbs, int count)
 
       /* for all instructions in the blocks do */
       for (ic = lBlock->sch; ic; ic = ic->next)
-       {
-
-         operand *aSym;
-         long litValue;
-         induction *ip;
-         iCode *indIc;
-         eBBlock *owner = NULL;
-         int nexits;
+        {
+
+          operand *aSym;
+          long litValue;
+          induction *ip;
+          iCode *indIc;
+          eBBlock *owner = NULL;
+          int nexits;
           sym_link *optype;
 
-         /* To find an induction variable, we need to */
+          /* To find an induction variable, we need to */
           /* find within the loop three iCodes in this */
           /* general form:                             */
           /*                                           */
@@ -811,29 +1022,29 @@ basicInduction (region * loopReg, eBBlock ** ebbs, int count)
           /*                                           */
           /* (symbolVar may also be an iTemp if it is  */
           /*  register equivalent)                     */
-          
+
           /* look for assignments of the form */
-         /*   symbolVar := iTempNN */
-         if (ic->op != '=')
-           continue;
+          /*   symbolVar := iTempNN */
+          if (ic->op != '=')
+            continue;
 
-         if (!IS_TRUE_SYMOP (IC_RESULT (ic)) &&
-             !OP_SYMBOL (IC_RESULT (ic))->isreqv)
-           continue;
+          if (!IS_TRUE_SYMOP (IC_RESULT (ic)) &&
+              !OP_SYMBOL (IC_RESULT (ic))->isreqv)
+            continue;
 
-         if (isOperandGlobal (IC_RESULT (ic)))
-           continue;
+          if (isOperandGlobal (IC_RESULT (ic)))
+            continue;
 
-         if (!IS_ITEMP (IC_RIGHT (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 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;
+          /* if the address of this was taken inside the loop then continue */
+          if (addressTaken (loopReg->regBlocks, IC_RESULT (ic)))
+            continue;
 
           /* Only consider variables with integral type. */
           /* (2004/12/06 - EEP - ds390 fails regression tests unless */
@@ -844,24 +1055,24 @@ basicInduction (region * loopReg, eBBlock ** ebbs, int count)
           if (!IS_INTEGRAL (optype) && !IS_PTR (optype))
             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;
-          
+          /* 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;
+
           /* make sure the definition found is the only one */
           if (assignmentsToSym (loopReg->regBlocks, IC_RIGHT (ic)) > 1)
-           continue;
+            continue;
 
-         if (IS_OP_LITERAL (IC_RIGHT (dic)))
+          if (IS_OP_LITERAL (IC_RIGHT (dic)))
             {
               aSym = IC_LEFT (dic);
               litValue = (long) operandLitValue (IC_RIGHT (dic));
@@ -877,133 +1088,67 @@ basicInduction (region * loopReg, eBBlock ** ebbs, int count)
               litValue = (long) operandLitValue (IC_LEFT (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;
-           }
-
-         /* 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 = NULL;
-                         int j;
-
-                         /* Need to search for bbnum == i since ebbs is  */
-                         /* sorted by dfnum; a direct index won't do.  */
-                         for (j=0; j<count; j++)
-                           if (ebbs[j]->bbnum == i)
-                             {
-                               eblock = ebbs[j];
-                               break;
-                             }
-                         assert(eblock);
-
-                         /* 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);
-       }
-
-    }                          /* end of all blocks for basic induction variables */
+          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;
+            }
+
+          /* 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->filename = ic->filename;
+          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));
+          /* ic should now be moved to the exit block(s) */
+
+          nexits = elementsInSet (loopReg->exits);
+          if (nexits == 0)
+            {
+              /* this is a endless loop without exits: there's
+                 no need to restore the inducted variable */
+              iCode *saveic = ic->prev;
+
+              /* ic will be removed by killDeadCode() too,
+                 but it's a nice to see a clean dumploop now. */
+              remiCodeFromeBBlock (lBlock, ic);
+              /* clear the definition */
+              bitVectUnSetBit (lBlock->defSet, ic->key);
+              ic = saveic;
+            }
+          else
+            addPostLoopBlock (loopReg, ebbi, ic);
+          addSet (&indVars, ip);
+        } /* for ic */
+    }     /* end of all blocks for basic induction variables */
 
   return indVars;
 }
@@ -1011,11 +1156,9 @@ basicInduction (region * loopReg, eBBlock ** ebbs, int count)
 /*-----------------------------------------------------------------*/
 /* loopInduction - remove induction variables from a loop          */
 /*-----------------------------------------------------------------*/
-int
+static int
 loopInduction (region * loopReg, ebbIndex * ebbi)
 {
-  eBBlock ** ebbs = ebbi->dfOrder;
-  int count = ebbi->count;
   int change = 0;
   eBBlock *lBlock, *owner, *lastBlock = NULL;
   set *indVars = NULL;
@@ -1025,7 +1168,7 @@ loopInduction (region * loopReg, ebbIndex * ebbi)
     return 0;
 
   /* we first determine the basic Induction variables */
-  basicInd = setFromSet (indVars = basicInduction (loopReg, ebbs, count));
+  basicInd = setFromSet (indVars = basicInduction (loopReg, ebbi));
 
   /* find other induction variables : by other we mean definitions of */
   /* the form x := y (* | / ) <constant> .. we will move  this one to */
@@ -1042,32 +1185,32 @@ loopInduction (region * loopReg, ebbIndex * ebbi)
       /* last block is the one with the highest block
          number */
       if (lastBlock->bbnum < lBlock->bbnum)
-       lastBlock = lBlock;
+        lastBlock = lBlock;
 
       for (ic = lBlock->sch; ic; ic = ic->next)
-       {
-         operand *aSym;
-         long litVal;
+        {
+          operand *aSym;
+          long litVal;
 
-         /* consider only * & / */
-         if (ic->op != '*' && ic->op != '/')
-           continue;
+          /* consider only * & / */
+          if (ic->op != '*' && ic->op != '/')
+            continue;
 
           /* only consider variables with integral type */
           if (!IS_INTEGRAL (operandType (IC_RESULT (ic))))
             continue;
-         
+
           /* if the result has more definitions then */
-         if (assignmentsToSym (loopReg->regBlocks, IC_RESULT (ic)) > 1)
-           continue;
+          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;
+          /* 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;
 
-         if (IS_SYMOP (IC_LEFT (ic)))
+          if (IS_SYMOP (IC_LEFT (ic)))
             {
               aSym = IC_LEFT (ic);
               litVal = (long) operandLitValue (IC_RIGHT (ic));
@@ -1081,59 +1224,60 @@ loopInduction (region * loopReg, ebbIndex * ebbi)
               litVal = (long) operandLitValue (IC_LEFT (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));
-
-         /* Replace mul/div with assignment to self; killDeadCode() will */
+          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->filename = ic->filename;
+          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));
+
+          /* Replace mul/div with assignment to self; killDeadCode() will */
           /* clean this up since we can't use remiCodeFromeBBlock() here. */
           ic->op = '=';
           IC_LEFT (ic) = NULL;
           IC_RIGHT (ic) = IC_RESULT (ic);
-          
+
           /* Insert an update of the induction variable just before */
           /* the update of the basic induction variable. */
           indIc = newiCode (ip->op,
                             operandFromOperand (IC_RESULT (ic)),
                             operandFromLit (litVal));
           IC_RESULT (indIc) = operandFromOperand (IC_RESULT (ic));
-         owner = NULL;
+          owner = NULL;
           dic = findDefInRegion (setFromSet (loopReg->regBlocks), aSym, &owner);
           assert (dic);
-         addiCodeToeBBlock (owner, indIc, dic);
-          
-       }
+          addiCodeToeBBlock (owner, indIc, dic);
+
+        }
     }
 
   /* if we have some induction variables then */
@@ -1146,24 +1290,24 @@ loopInduction (region * loopReg, ebbIndex * ebbi)
 
       /* 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++;
-       }
+           ip;
+           ip = setNextItem (indVars))
+        {
+          indVect = bitVectSetBit (indVect, ip->ic->key);
+          ip->ic->filename = preHdr->ech->filename;
+          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                     */
@@ -1185,6 +1329,7 @@ loopInduction (region * loopReg, ebbIndex * ebbi)
 /*-----------------------------------------------------------------*/
 /* mergeRegions - will merge region with same entry point           */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (mergeRegions)
 {
   region *theLoop = item;
@@ -1201,14 +1346,14 @@ DEFSETFUNC (mergeRegions)
     {
 
       if (lp == theLoop)
-       continue;
+        continue;
 
       if (lp->entry == theLoop->entry)
-       {
-         theLoop->regBlocks = unionSets (theLoop->regBlocks,
-                                         lp->regBlocks, THROW_DEST);
-         lp->merged = 1;
-       }
+        {
+          theLoop->regBlocks = unionSets (theLoop->regBlocks,
+                                          lp->regBlocks, THROW_DEST);
+          lp->merged = 1;
+        }
     }
 
   return 1;
@@ -1217,6 +1362,7 @@ DEFSETFUNC (mergeRegions)
 /*-----------------------------------------------------------------*/
 /* ifMerged - return 1 if the merge flag is 1                      */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (ifMerged)
 {
   region *lp = item;
@@ -1227,6 +1373,7 @@ DEFSETFUNC (ifMerged)
 /*-----------------------------------------------------------------*/
 /* mergeInnerLoops - will merge into body when entry is present    */
 /*-----------------------------------------------------------------*/
+static
 DEFSETFUNC (mergeInnerLoops)
 {
   region *theLoop = item;
@@ -1240,17 +1387,17 @@ DEFSETFUNC (mergeInnerLoops)
     {
 
       if (lp == theLoop)
-       continue;
+        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);
-       }
+        {
+          lp->containsLoops += theLoop->containsLoops + 1;
+          if (lp->containsLoops > (*maxDepth))
+            *maxDepth = lp->containsLoops;
+
+          lp->regBlocks = unionSets (lp->regBlocks,
+                                     theLoop->regBlocks, THROW_DEST);
+        }
     }
 
   return 1;
@@ -1263,7 +1410,7 @@ DEFSETFUNC (mergeInnerLoops)
 hTab *
 createLoopRegions (ebbIndex * ebbi)
 {
-  set *allRegion = NULL;       /* set of all loops */
+  set *allRegion = NULL;        /* set of all loops */
   hTab *orderedLoops = NULL;
   set *bEdges = NULL;
   int maxDepth = 0;
@@ -1271,7 +1418,7 @@ createLoopRegions (ebbIndex * ebbi)
 
   /* get all the back edges in the graph */
   if (!applyToSet (graphEdges, backEdges, &bEdges))
-    return 0;                  /* found no loops */
+    return 0;                   /* found no loops */
 
   /* for each of these back edges get the blocks that */
   /* constitute the loops                             */
@@ -1296,8 +1443,8 @@ createLoopRegions (ebbIndex * ebbi)
   for (lp = setFirstItem (allRegion); lp; lp = setNextItem (allRegion))
     {
       applyToSet (lp->regBlocks, addToExitsMarkDepth,
-                 lp->regBlocks, &lp->exits,
-                 (maxDepth - lp->containsLoops), lp);
+                  lp->regBlocks, &lp->exits,
+                  (maxDepth - lp->containsLoops), lp);
 
       hTabAddItem (&orderedLoops, lp->containsLoops, lp);
 
@@ -1329,10 +1476,10 @@ loopOptimizations (hTab * orderedLoops, ebbIndex * ebbi)
     {
 
       if (optimize.loopInvariant)
-       change += loopInvariants (lp, ebbi);
+        change += loopInvariants (lp, ebbi);
 
       if (optimize.loopInduction)
-       change += loopInduction (lp, ebbi);
+        change += loopInduction (lp, ebbi);
     }
 
   return change;