diff mbox

[graphite] Reduce the number of params in a scop to 3

Message ID 1438204470-15164-1-git-send-email-hiraditya@msn.com
State New
Headers show

Commit Message

Aditya K July 29, 2015, 9:14 p.m. UTC 
More than 3 params consumes too much memory while bootstrapping gcc
with graphite enabled.

BOOT_CFLAGS="-g -O2 -fgraphite-identity -floop-block -floop-interchange -floop-strip-mine"
---
 gcc/params.def | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Sebastian Pop July 29, 2015, 9:19 p.m. UTC | #1 
Aditya Kumar wrote:
> More than 3 params consumes too much memory while bootstrapping gcc
> with graphite enabled.

Ok.  I will commit the patch.
Thanks for fixing bootstrap with graphite enabled.

Sebastian
Sebastian Pop July 29, 2015, 9:24 p.m. UTC | #2 
Sebastian Pop wrote:
> Aditya Kumar wrote:
> > More than 3 params consumes too much memory while bootstrapping gcc
> > with graphite enabled.
> 
> Ok.  I will commit the patch.
> Thanks for fixing bootstrap with graphite enabled.

We will increase the max when we will use ISL's mechanism to count the number of
polyhedral operations.  Until then I think we'd better lower the max nb_params
to avoid long compile times and reduce memory requirements to bootstrap with
graphite.
diff mbox

Patch

diff --git a/gcc/params.def b/gcc/params.def
index 0ca3451..1f6e40e 100644
--- a/gcc/params.def
+++ b/gcc/params.def
@@ -850,7 +850,7 @@  DEFPARAM (PARAM_LOOP_UNROLL_JAM_DEPTH,
 DEFPARAM (PARAM_GRAPHITE_MAX_NB_SCOP_PARAMS,
 	  "graphite-max-nb-scop-params",
 	  "maximum number of parameters in a SCoP",
-	  10, 0, 0)
+	  3, 0, 0)
 
 /* Maximal number of basic blocks in the functions analyzed by Graphite.  */