Re: [GSoC] the separate option for all dimensions

2014-08-12 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

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 gareevro...@gmail.com 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

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-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-05 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 gareevro...@gmail.com [gcc/] * graphite-isl-ast-to-gimple.c: (set_options): New function.