This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Merging Graphite to trunk


On Fri, Jul 31, 2009 at 12:41, Richard Guenther<rguenther@suse.de> wrote:
>> What about renaming -fgraphite-force-parallel into -floop-parallelize-all?
>> We could have -floop-parallelize for the version with cost model.
>> These would be consistent with the flag names of the other loop transforms
>> exposed from Graphite.
>
> Certainly better than -fgraphite-force-parallel.

Done like this.  Okay for trunk after regstrap?

Sebastian
Index: doc/invoke.texi
===================================================================
--- doc/invoke.texi	(revision 150316)
+++ doc/invoke.texi	(working copy)
@@ -347,7 +347,7 @@ Objective-C and Objective-C++ Dialects}.
 -fno-ira-share-spill-slots -fira-verbose=@var{n} @gol
 -fivopts -fkeep-inline-functions -fkeep-static-consts @gol
 -floop-block -floop-interchange -floop-strip-mine -fgraphite-identity @gol
--fgraphite-force-parallel @gol
+-floop-parallelize-all @gol
 -fmerge-all-constants -fmerge-constants -fmodulo-sched @gol
 -fmodulo-sched-allow-regmoves -fmove-loop-invariants -fmudflap @gol
 -fmudflapir -fmudflapth -fno-branch-count-reg -fno-default-inline @gol
@@ -6602,7 +6602,7 @@ GIMPLE -> GRAPHITE -> GIMPLE transformat
 are also performed by the code generator CLooG, like index splitting and
 dead code elimination in loops.
 
-@item -fgraphite-force-parallel
+@item -floop-parallelize-all
 Use the Graphite data dependence analysis to identify loops that can
 be parallelized.  Parallelize all the loops that can be analyzed to
 not contain loop carried dependences without checking that it is
Index: toplev.c
===================================================================
--- toplev.c	(revision 150316)
+++ toplev.c	(working copy)
@@ -1802,7 +1802,7 @@ process_options (void)
       || flag_loop_interchange
       || flag_loop_strip_mine
       || flag_graphite_identity
-      || flag_graphite_force_parallel)
+      || flag_loop_parallelize_all)
     sorry ("Graphite loop optimizations cannot be used");
 #endif
 
Index: testsuite/gcc.dg/graphite/graphite_autopar/graphite_autopar.exp
===================================================================
--- testsuite/gcc.dg/graphite/graphite_autopar/graphite_autopar.exp	(revision 150316)
+++ testsuite/gcc.dg/graphite/graphite_autopar/graphite_autopar.exp	(working copy)
@@ -48,7 +48,7 @@ set wait_to_run_files [lsort [glob -noco
 
 # Flags using for force-parallel-*.c files.
 set DEFAULT_CFLAGS_FORCE_PARALLEL " -ansi -pedantic-errors -O2 \
--ftree-parallelize-loops=4 -fgraphite-force-parallel \
+-ftree-parallelize-loops=4 -floop-parallelize-all \
 -fdump-tree-parloops-details -fdump-tree-optimized \
 -fno-loop-strip-mine -fdump-tree-graphite-all"
 set force_parallel_files \
Index: tree-ssa-loop.c
===================================================================
--- tree-ssa-loop.c	(revision 150316)
+++ tree-ssa-loop.c	(working copy)
@@ -308,7 +308,7 @@ gate_graphite_transforms (void)
   /* Enable -fgraphite pass if any one of the graphite optimization flags 
      is turned on.  */
   if (flag_loop_block || flag_loop_interchange || flag_loop_strip_mine
-      || flag_graphite_identity || flag_graphite_force_parallel)
+      || flag_graphite_identity || flag_loop_parallelize_all)
     flag_graphite = 1;
 
   if (flag_loop_block)
Index: graphite.c
===================================================================
--- graphite.c	(revision 150316)
+++ graphite.c	(working copy)
@@ -283,7 +283,7 @@ graphite_transform_loops (void)
 	}
     }
 
-  if (flag_graphite_force_parallel)
+  if (flag_loop_parallelize_all)
     mark_loops_parallel (bb_pbb_mapping);
 
   htab_delete (bb_pbb_mapping);
Index: common.opt
===================================================================
--- common.opt	(revision 150316)
+++ common.opt	(working copy)
@@ -577,8 +577,8 @@ Common Report Var(flag_graphite_identity
 Enable Graphite Identity transformation
 
 ; This option is not documented as it will be removed
-fgraphite-force-parallel
-Common Report Var(flag_graphite_force_parallel) Optimization
+floop-parallelize-all
+Common Report Var(flag_loop_parallelize_all) Optimization
 Mark all loops as parallel
 
 floop-strip-mine
Index: graphite-poly.c
===================================================================
--- graphite-poly.c	(revision 150316)
+++ graphite-poly.c	(working copy)
@@ -248,7 +248,7 @@ apply_poly_transforms (scop_p scop)
   if (flag_graphite_identity)
     transform_done = true;
 
-  if (flag_graphite_force_parallel)
+  if (flag_loop_parallelize_all)
     transform_done = true;
 
   if (flag_loop_block)

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]