Re: [graphite] instancewise dependence analysis functions and structures

["Sebastian Pop" <sebpop at gmail dot com>]

On Fri, Feb 15, 2008 at 7:23 AM, Konrad Trifunovic
<konrad.trifunovic@gmail.com> wrote:
> Hi,
>
>  I removed those function that are not needed anymore. I also made a
>  dummy call for those methods that will be called, but at this moment
>  should remain inactive. This should remove any compiler warnings when
>  compiling graphite.c.
>

Thanks for the fix.  Committed the attached patch.

Sebastian

Index: ChangeLog.graphite
===================================================================
--- ChangeLog.graphite	(revision 132318)
+++ ChangeLog.graphite	(working copy)
@@ -1,3 +1,9 @@
+2008-02-15  Konrad Trifunovic  <konrad.trifunovic@inria.fr>
+
+        * graphite.c (build_scop_alpha): Removed.
+	(graphite_transform_loops): Add a dummy call to build_all_rdg_levels
+        and dump_dependence_graph to avoid compiler warnings.
+
 2008-02-14  Konrad Trifunovic  <konrad.trifunovic@inria.fr>
 
         * tree-data-ref.c (dr_may_alias_p, create_rdg_vertices): Extern.
Index: graphite.c
===================================================================
--- graphite.c	(revision 132318)
+++ graphite.c	(working copy)
@@ -1830,41 +1830,6 @@ build_rdg_all_levels (scop_p scop)
   return rdg;  
 }
 
-/* For each basic block in SCOP, build its dynamic schedule.  */
-
-static void
-build_scop_alpha (scop_p scop)
-{
-  unsigned i, j;
-  graphite_bb_p gb;
-  unsigned nb = scop_nb_loops (scop);
-  struct loop *ploop, *loop;
-  struct loop_to_cloog_loop_str tmp, *slot; 
-
-  for (i = 0; VEC_iterate (graphite_bb_p, SCOP_BBS (scop), i, gb); i++)
-    {
-      GBB_ALPHA (gb) = lambda_vector_new (nb);
-      loop = GBB_BB (gb)->loop_father;
-
-      for (j = 0; VEC_iterate (loop_p, loop->superloops, j, ploop); j++)
-	if (loop_in_scop_p (ploop, scop))
-	  {
-	    tmp.loop_num = ploop->num;
-	    slot = (struct loop_to_cloog_loop_str *) 
-	      htab_find (SCOP_LOOP2CLOOG_LOOP(scop), &tmp);
-	    GBB_ALPHA (gb) [slot->loop_position] = 1;
-	  }
-
-      if (loop_in_scop_p (loop, scop))
-	{
-	  tmp.loop_num = loop->num;
-	  slot = (struct loop_to_cloog_loop_str *)
-	    htab_find (SCOP_LOOP2CLOOG_LOOP(scop), &tmp);
-	  GBB_ALPHA (gb) [slot->loop_position] = 1;
-	}
-    }
-}
-
 /* Dumps the dependence graph G to file F.  */
 
 static void
@@ -1915,6 +1880,7 @@ graphite_transform_loops (void)
 {
   unsigned i;
   scop_p scop;
+  struct graph * rdg = NULL;
 
   current_scops = VEC_alloc (scop_p, heap, 3);
 
@@ -1938,6 +1904,10 @@ graphite_transform_loops (void)
 	continue;
 
       build_scop_data_accesses (scop);
+
+      if (0)
+	build_rdg_all_levels (scop);
+
       gloog (scop, find_transform (scop));
     }
 
@@ -1947,6 +1917,8 @@ graphite_transform_loops (void)
       print_scops (dump_file, 2);
       fprintf (dump_file, "\nnumber of SCoPs: %d\n",
 	       VEC_length (scop_p, current_scops));
+      if (0)
+	dump_dependence_graph (dump_file, rdg);
     }
 
   free_scops (current_scops);