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: Integration of ISL code generator into Graphite

2014-04-30 Thread Mircea Namolaru
there are a lot of things that show you how to work with isl ast and may considerably help you. Mircea - Original Message - From: Roman Gareev gareevro...@gmail.com To: Mircea Namolaru mircea.namol...@inria.fr, Tobias Grosser tob...@grosser.es Cc: Albert Cohen albert.co

Re: Integration of ISL code generator into Graphite

2014-03-27 Thread Mircea Namolaru
time) Mircea - Original Message - From: Tobias Grosser tob...@grosser.es To: Mircea Namolaru mircea.namol...@inria.fr, Roman Gareev gareevro...@gmail.com Cc: gcc@gcc.gnu.org, Albert Cohen albert.co...@inria.fr Sent: Monday, March 24, 2014 2:11:49 PM Subject: Re: Integration

Re: Integration of ISL code generator into Graphite

2014-03-24 Thread Mircea Namolaru
- From: Tobias Grosser tob...@grosser.es To: Roman Gareev gareevro...@gmail.com Cc: gcc@gcc.gnu.org, Albert Cohen albert.co...@inria.fr, Mircea Namolaru mircea.namol...@inria.fr Sent: Friday, March 21, 2014 12:51:26 PM Subject: Re: Integration of ISL code generator into Graphite On 03/21/2014

Re: Integration of ISL code generator into Graphite

2014-03-17 Thread Mircea Namolaru
the ISL AST (with possible addition of new attributes/transformations) ? Regards, Mircea - Original Message - From: Roman Gareev gareevro...@gmail.com To: gcc@gcc.gnu.org Cc: Tobias Grosser tob...@grosser.es, Albert Cohen albert.co...@inria.fr, Mircea Namolaru mircea.namol

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

Re: Some thoughts about steerring commitee work

2007-06-25 Thread Mircea Namolaru
As for C++, I think we need more OO language specific optimizations. I don't know what the status of devirtualizion which was reported on the previous summit. Sorry for the late replay. The devirtualization is on hold. Currently GCC is lacking the necessary infrastructure needed by C++

Re: CSE not combining equivalent expressions.

2007-01-17 Thread Mircea Namolaru
Thanks. Another question I have is that, in this case, will the following http://gcc.gnu.org/wiki/Sign_Extension_Removal help in removal of the sign / zero extension ? First, it seems to me that in your case: (1) a = a | 1 /* a |= 1 */ (2) a = a | 1 /* a |= 1 */ the expressions a | 1 in

Re: CSE not combining equivalent expressions.

2007-01-17 Thread Mircea Namolaru
Thanks. Another question I have is that, in this case, will the following http://gcc.gnu.org/wiki/Sign_Extension_Removal help in removal of the sign / zero extension ? First, it seems to me that in your case: (1) a = a | 1 /* a |= 1 */ (2) a = a | 1 /* a |= 1 */ the expressions a | 1 in

Re: CSE not combining equivalent expressions.

2007-01-17 Thread Mircea Namolaru
[EMAIL PROTECTED] (Richard Kenner) wrote on 17/01/2007 18:04:20: First, it seems to me that in your case: (1) a = a | 1 /* a |= 1 */ (2) a = a | 1 /* a |= 1 */ the expressions a | 1 in (1) and (2) are different as the a is not the same. So there is nothing to do for CSE. It's

Re: Disabling -fsee at -O3

2006-05-15 Thread Mircea Namolaru
@item -fsee @opindex fsee Eliminates redundant extension instructions and move the non redundant ones to optimal placement using LCM. Enabled at level @option{-O3}. Would you mind adjusting this as well Thanks. I've updated doc/invoke.texi correspondingly. Mircea

Disabling -fsee at -O3

2006-05-07 Thread Mircea Namolaru
Committed according to http://gcc.gnu.org/ml/gcc/2006-05/msg00185.html Mircea 2006-05-07 Mircea Namolaru [EMAIL PROTECTED] * opts.c (flag_see): remove its setting at -O3. Index: opts.c === --- opts.c (revision

Re: Status of SEE and Autovectorization patches?

2006-05-05 Thread Mircea Namolaru
That certainly does suggest a bug in the SEE patches. They needn't do anything useful on IA32/AMD64, but they should presumably either (a) not cause a bootstrap failure on these architectures, or (b) be disabled on these architectures. Agree. I will check the bootstrapping on x86. (a) seems

Re: Status of SEE and Autovectorization patches?

2006-05-04 Thread Mircea Namolaru
The patches for SEE have been committed today. The minor style corrections requested by you in the final review approval will be in a follow-up patch to be submitted the next week. Mircea