Re: [GSoC] the separate option for all dimensions

2014-08-12 Thread Tobias Grosser
On August 12, 2014 8:15:34 AM CEST, Roman Gareev wrote: >> Checking for this specific AST may cause failures with future >versions of >> isl that choose a different schedule. Could you write a regular >expression >> that checks that there is no if-condition contained in a for loop? I >think >> thi

Re: [GSoC] the separate option for all dimensions

2014-08-11 Thread Roman Gareev
> Checking for this specific AST may cause failures with future versions of > isl that choose a different schedule. Could you write a regular expression > that checks that there is no if-condition contained in a for loop? I think > this best models the issue that was addressed in the original bug r

Re: [GSoC] the separate option for all dimensions

2014-08-06 Thread Tobias Grosser
On 06/08/2014 17:21, Roman Gareev wrote: I've tested the modified version of Graphite using the gcc test suite and haven't found out new failed tests. However, pr35356-2.c is still not suitable for testing. The ISL AST generated from its source code doesn't contain MIN or MAX. if (k <= -1)

Re: [GSoC] the separate option for all dimensions

2014-08-06 Thread Roman Gareev
I've tested the modified version of Graphite using the gcc test suite and haven't found out new failed tests. However, pr35356-2.c is still not suitable for testing. The ISL AST generated from its source code doesn't contain MIN or MAX. if (k <= -1) { for (int c1 = 0; c1 < n; c1 += 1)

Re: [GSoC] the separate option for all dimensions

2014-08-04 Thread Tobias Grosser
On 05/08/2014 06:02, Roman Gareev wrote: I've attached the patch, which sets the separate option for all dimensions. Is it fine for trunk? LGTM. Tobias

[GSoC] the separate option for all dimensions

2014-08-04 Thread Roman Gareev
I've attached the patch, which sets the separate option for all dimensions. Is it fine for trunk? -- Cheers, Roman Gareev. 2014-08-05 Roman Gareev [gcc/] * graphite-isl-ast-to-gimple.c: (set_options): New function. (scop_to_isl_ast