more multiple int i.
authorGengbin Zheng <gzheng@illinois.edu>
Sat, 30 Apr 2005 02:27:48 +0000 (02:27 +0000)
committerGengbin Zheng <gzheng@illinois.edu>
Sat, 30 Apr 2005 02:27:48 +0000 (02:27 +0000)
src/ck-ldb/RefineTopoLB.C
src/ck-ldb/TopoLB.C

index 3ebed17641142ee7af197e0fa04866c811723e29..44ce569834a86a9d1082fb65df93018501df717d 100644 (file)
@@ -41,6 +41,7 @@ CmiBool RefineTopoLB::QueryBalanceNow (int _step)
 
 void RefineTopoLB :: work(CentralLB::LDStats *stats,int count)
 {
+  int i, j;
   if (_lb_args.debug() >= 2) 
   {
     CkPrintf("In TopoLB Strategy...\n");
@@ -102,13 +103,13 @@ void RefineTopoLB :: work(CentralLB::LDStats *stats,int count)
   initDataStructures(stats,count,newmap);
   if(_lb_debug_on)
     CkPrintf("After initizlizing dataStructures...\n");
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
     assign[i]=i;
   if(_lb_debug_on)
     printDataStructures(count, stats->n_objs,newmap);
   /***************** Perform RefineMent *************************/
   bool *swapdone=new bool[count];
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
     swapdone[i]=false;
 
     
@@ -117,12 +118,12 @@ void RefineTopoLB :: work(CentralLB::LDStats *stats,int count)
  // CkPrintf(" Before Mapping Original   hopBytes : %lf  Avg comm hops: %lf\n", hbval,hbval/total_comm);
   //Perform ith swap
   double totalGain=0;
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     //select the cpart which is most communicating and hasn't been moved yet
     int cpart=-1;
     double maxComm=-1;
-    for(int j=0;j<count;j++)
+    for(j=0;j<count;j++)
     {
       if(swapdone[j]) continue;
       if(commUA[j]>maxComm)
@@ -138,7 +139,7 @@ void RefineTopoLB :: work(CentralLB::LDStats *stats,int count)
     double gainMax=-1;
     double gain=-1;;
     double orig_value=getInterMedHopBytes(stats,count,newmap);
-    for(int j=0;j<count;j++)
+    for(j=0;j<count;j++)
     {
       if(j==cpart)
         continue;
@@ -171,7 +172,7 @@ void RefineTopoLB :: work(CentralLB::LDStats *stats,int count)
     //CkPrintf("Swap# %d:  %d and %d\n",i+1,cpart,swapcpart);
   }
   /******************* Assign mapping and print Stats*********/
-  for(int i=0;i<stats->n_objs;i++)
+  for(i=0;i<stats->n_objs;i++)
   {
     stats->to_proc[i]= assign[newmap[i]];
   }
index f08c6f37203a8bc1608a1e465f23110f1d7100a4..6bfd26f471edc0b54ec9dc1c5dd8d02b155e0ae6 100644 (file)
@@ -62,21 +62,22 @@ void TopoLB::freeDataStructures(int count)
 
 void TopoLB::allocateDataStructures(int count )
 {
+  int i;
   //Allocating in separate loop to have somewhat contiguous memory allocation
   hopBytes=new double*[count];
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     hopBytes[i]=new double[count+2];
   }
 
   dist=new double*[count];
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     dist[i]=new double[count+1];
   }
 
   comm=new double*[count];
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     comm[i]=new double[count];
   }
@@ -291,12 +292,13 @@ void TopoLB::computePartitions(CentralLB::LDStats *stats,int count,int *newmap)
 
 void TopoLB::initDataStructures(CentralLB::LDStats *stats,int count,int *newmap)
 {
+  int i;
   //init dist
   if(_lb_debug_on)
     CkPrintf("Before initing dist\n");
  
   topo->get_pairwise_hop_count(dist);
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     double totaldist=0;
     for(int j=0;j<count;j++)
@@ -310,7 +312,7 @@ void TopoLB::initDataStructures(CentralLB::LDStats *stats,int count,int *newmap)
   //Init comm,commUA from stats
   if(_lb_debug_on)
     CkPrintf("Before initing comm\n");
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     for(int j=0;j<count;j++)
     {
@@ -319,7 +321,7 @@ void TopoLB::initDataStructures(CentralLB::LDStats *stats,int count,int *newmap)
     commUA[i]=0;
   }
   bool *multicastAdded=new bool[count];
-  for(int i=0;i<stats->n_comm;i++)
+  for(i=0;i<stats->n_comm;i++)
   {
     LDCommData &cdata=stats->commData[i];
     if(!cdata.from_proc() && cdata.receiver.get_type() ==LD_OBJ_MSG)
@@ -402,7 +404,7 @@ void TopoLB::initDataStructures(CentralLB::LDStats *stats,int count,int *newmap)
   //hopBytes[i][j]=hopBytes if partition i is placed at proc j
   if(_lb_debug_on)
     CkPrintf("Before initing hopBytes\n");
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     int hbminIndex=0;
     double hbtotal=0;
@@ -423,7 +425,7 @@ void TopoLB::initDataStructures(CentralLB::LDStats *stats,int count,int *newmap)
   //Init pfree, cfree, assign
   if(_lb_debug_on)
     CkPrintf("Before initing pfree cfree assign\n");
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     pfree[i]=true;
     cfree[i]=true;
@@ -433,6 +435,7 @@ void TopoLB::initDataStructures(CentralLB::LDStats *stats,int count,int *newmap)
 
 void TopoLB :: work(CentralLB::LDStats *stats,int count)
 {
+  int i, j;
   if (_lb_args.debug() >= 2) 
   {
     CkPrintf("In TopoLB Strategy...\n");
@@ -481,7 +484,7 @@ void TopoLB :: work(CentralLB::LDStats *stats,int count)
     computePartitions(stats,count,newmap);
   else
   {
-    for(int i=0;i<stats->n_objs;i++)
+    for(i=0;i<stats->n_objs;i++)
     {
       newmap[i]=stats->from_proc[i];
     }
@@ -503,7 +506,7 @@ void TopoLB :: work(CentralLB::LDStats *stats,int count)
     CkPrintf("before performing mapping...\n");
 
   double *distnew=new double[count];
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     //Assume i-1 partitions placed, 
     //select and place the ith partition
@@ -585,7 +588,7 @@ void TopoLB :: work(CentralLB::LDStats *stats,int count)
 
     //Update hopBytes
     
-    for(int j=0;j<count;j++)
+    for(j=0;j<count;j++)
     {
       if(procs_left>1)
         distnew[j]=(dist[j][count]*procs_left -dist[j][proc_index]) / (procs_left-1);
@@ -648,7 +651,7 @@ void TopoLB :: work(CentralLB::LDStats *stats,int count)
 
     // d[j][count] is the average dist of proc j to unassigned procs
     // Also update commUA[j]
-    for(int j=0;j<count;j++)
+    for(j=0;j<count;j++)
     {
       if(procs_left>1)
         dist[j][count]=(dist[j][count]*procs_left -dist[j][proc_index]) / (procs_left-1);
@@ -660,7 +663,7 @@ void TopoLB :: work(CentralLB::LDStats *stats,int count)
 
   /******************  Fill out final composition Mapping **************************/
 
-  for(int i=0;i<stats->n_objs;i++)
+  for(i=0;i<stats->n_objs;i++)
   {
     stats->to_proc[i]= assign[newmap[i]];
   }
@@ -683,6 +686,7 @@ void TopoLB :: work(CentralLB::LDStats *stats,int count)
 
 void TopoLB::printDataStructures(int count,int n_objs,int *newmap)
 {
+  int i;
   /*
   CkPrintf("Partition Results : \n");
   for(int i=0;i<n_objs;i++)
@@ -692,7 +696,7 @@ void TopoLB::printDataStructures(int count,int n_objs,int *newmap)
   */
 
   CkPrintf("Dist : \n");
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     for(int j=0;j<count+1;j++)
     {
@@ -701,7 +705,7 @@ void TopoLB::printDataStructures(int count,int n_objs,int *newmap)
     CkPrintf("\n");
   }
   CkPrintf("HopBytes: \n");
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     for(int j=0;j<count+2;j++)
     {
@@ -712,11 +716,12 @@ void TopoLB::printDataStructures(int count,int n_objs,int *newmap)
 }
 double TopoLB::getHopBytes(CentralLB::LDStats *stats,int count,CkVec<int>map)
 {
+  int i;
   double **comm1=new double*[count];
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
     comm1[i]=new double[count];
 
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     for(int j=0;j<count;j++)
     {
@@ -725,7 +730,7 @@ double TopoLB::getHopBytes(CentralLB::LDStats *stats,int count,CkVec<int>map)
   }
 
   bool *multicastAdded=new bool[count];
-  for(int i=0;i<stats->n_comm;i++)
+  for(i=0;i<stats->n_comm;i++)
   {
     LDCommData &cdata=stats->commData[i];
     if(!cdata.from_proc() && cdata.receiver.get_type() ==LD_OBJ_MSG)
@@ -753,7 +758,7 @@ double TopoLB::getHopBytes(CentralLB::LDStats *stats,int count,CkVec<int>map)
       
       CmiAssert(sender<stats->n_objs);
 
-      for(int i=0;i<count;i++)
+      for(i=0;i<count;i++)
         multicastAdded[i]=false;
       multicastAdded[send_part]=true;
 
@@ -779,7 +784,7 @@ double TopoLB::getHopBytes(CentralLB::LDStats *stats,int count,CkVec<int>map)
   double totalHB=0;
   int proc1,proc2;
 
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
   {
     proc1=map[i];
     for(int j=0;j<count;j++)
@@ -789,7 +794,7 @@ double TopoLB::getHopBytes(CentralLB::LDStats *stats,int count,CkVec<int>map)
       totalHB+=dist[i][j]*comm1[i][j];
     }
   }
-  for(int i=0;i<count;i++)
+  for(i=0;i<count;i++)
     delete[] comm1[i];
   delete[] comm1;