Re: [GRAPHITE, PATCH] Ping: Loop unroll and jam optimization

2014-11-17 Thread Mircea Namolaru
New optimization flags and new params need documentation in gcc/doc/invoke.texi. Thanks. Added description in invoke.texi. The patch is in trunk. The description of the --params suggest they provide fixed values - is there no way to autodetect sensible values with a cost-model? I hardly

Re: [GRAPHITE, PATCH] Ping: Loop unroll and jam optimization

2014-11-15 Thread Mircea Namolaru
, black_box); PBB_TRANSFORMED (pbb) = NULL; 2014-11-12 Mircea Namolaru mircea.namol...@inria.fr * common.opt (flag_loop_unroll_and_jam): New flag for unroll and jam. * params.def (PARAM_LOOP_UNROLL_JAM_SIZE) (PARAM_LOOP_UNROLL_JAM_DEPTH): Parameters for unroll and jam flag

Re: [GRAPHITE, PATCH] Loop unroll and jam optimization

2014-11-13 Thread Mircea Namolaru
...@kotnet.org To: Mircea Namolaru mircea.namol...@inria.fr Cc: gcc-patches@gcc.gnu.org, Tobias Grosser tob...@grosser.es, Richard Biener richard.guent...@gmail.com, Albert Cohen albert.co...@inria.fr Sent: Wednesday, November 12, 2014 12:24:40 AM Subject: Re: [GRAPHITE, PATCH] Loop unroll

Re: [GRAPHITE, PATCH] Loop unroll and jam optimization

2014-11-12 Thread Mircea Namolaru
options. Mircea 2014-11-12 Mircea Namolaru mircea.namol...@inria.fr * common.opt (flag_loop_unroll_and_jam): New flag for unroll and jam. * params.def (PARAM_LOOP_UNROLL_JAM_SIZE) (PARAM_LOOP_UNROLL_JAM_DEPTH): Parameters for unroll and jam flag. * graphite

Re: [GRAPHITE, PATCH] Loop unroll and jam optimization

2014-11-11 Thread Mircea Namolaru
of unrolled loop for unroll-and-jam, + 2, 0, 0) + + /* Maximal number of parameters that we allow in a SCoP. */ DEFPARAM (PARAM_GRAPHITE_MAX_NB_SCOP_PARAMS, 2014-11-11 Mircea Namolaru mircea.namol...@inria.fr * common.opt (flag_loop_unroll_and_jam): New flag for unroll and jam

Re: [GRAPHITE, PATCH] Loop unroll and jam optimization

2014-11-11 Thread Mircea Namolaru
) don't succeed to remove the ifs from the loops and generate two loop structures (this would have been highly desirable). As the stage 1 is going to close soon, quick feedback to this patch is greatly appreciated. Many thanks, Mircea Namolaru int f1(int v[1024][1024], int HEIGHT, int LENGTH

[GRAPHITE, PATCH] Loop unroll and jam optimization

2014-11-07 Thread Mircea Namolaru
loop structures (this would have been highly desirable). As the stage 1 is going to close soon, quick feedback to this patch is greatly appreciated. Many thanks, Mircea Namolaru 2014-11-7 Mircea Namolaru mircea.namol...@inria.fr * common.opt (flag_loop_unroll_jam): New flag

Re: Replacement of isl_int by isl_val

2014-08-06 Thread Mircea Namolaru
On 08/03/14 17:44, Mircea Namolaru wrote: 2014-08-03 Mircea Namolarumircea.namol...@inria.fr Replacement of isl-int by isl_val * graphite-clast-to-gimple.c: include isl/val.h, isl/val_gmp.h (compute_bounds_for_param): use isl_val instead of isl_int

Replacement of isl_int by isl_val

2014-08-03 Thread Mircea Namolaru
= isl_set_add_constraint (scop-context, c); 2014-08-03 Mircea Namolaru mircea.namol...@inria.fr Replacement of isl-int by isl_val * graphite-clast-to-gimple.c: include isl/val.h, isl/val_gmp.h (compute_bounds_for_param): use isl_val instead of isl_int

[GRAPHITE] Replacement of isl_int by isl_val

2014-07-06 Thread Mircea Namolaru
); + isl_val_get_num_gmp (isl_value, up); + isl_val_free (isl_value); isl_set_free (domain); isl_aff_free (dimension); } 2014-07-06 Mircea Namolaru mircea.namol...@inria.fr Replacement of isl-int by isl_val * graphite-clast-to-gimple.c: include isl/val.h, isl/val_gmp.h

Re: Fix PR59586

2014-03-11 Thread Mircea Namolaru
in the future so you don't want to base the correctness of the code on this assumption. Mircea - Original Message - From: Roman Gareev gareevro...@gmail.com To: gcc-patches@gcc.gnu.org Cc: Tobias Grosser tob...@grosser.es, mircea namolaru mircea.namol...@inria.fr Sent: Monday, March 10, 2014 5

Re: [PATCH] Fix bug 59586

2014-03-10 Thread Mircea Namolaru
Hi, Your patch is fine - even without this bug, introducing NULL pointer checks before dereferencing a pointer is a good thing. Mircea

Re: [PATCH,GRAPHITE] Fix for P1 bug 58028

2014-02-28 Thread Mircea Namolaru
Hi, Thanks. Here is the updated patch. 2014-02-26 Tobias Grosser tob...@grosser.es Mircea Namolaru mircea.namol...@inria.fr PR tree-optimization/58028 * graphite-clast-to-gimple.c (set_cloog_options): Don't remove scalar dimensions. Index: gcc/graphite

Re: [PATCH,GRAPHITE] Fix for P1 bug 58028

2014-02-27 Thread Mircea Namolaru
Thanks for comments - updated the patch (fixed my e-mail address too :-)). 2014-02-26 Tobias Grosser tob...@grosser.es Mircea Namolaru mircea.namol...@inria.fr Fix for bug 58028 * graphite-clast-to-gimple.c (set_cloog_options): Don't remove scalar dimensions