Re: [PATCH] Fix PR48648: Handle CLAST assignments.

2011-07-29 Thread Sebastian Pop
Hi Tobi, On Thu, Jul 28, 2011 at 12:13, Tobias Grosser wrote: >> +  struct clast_user_stmt *body >> +    = clast_get_body_of_loop ((struct clast_stmt *) stmt); > > I am not a big fan of using clast_get_body_of_loop as it is buggy. > Introducing new uses of it, is nothing what I would support. Do

Re: [PATCH] Fix PR48648: Handle CLAST assignments.

2011-07-28 Thread Tobias Grosser
On 07/28/2011 06:56 PM, Sebastian Pop wrote: Hi Tobi, On Thu, Jul 28, 2011 at 12:13, Tobias Grosser wrote: + struct clast_user_stmt *body += clast_get_body_of_loop ((struct clast_stmt *) stmt); I am not a big fan of using clast_get_body_of_loop as it is buggy. Introducing new uses of it

Re: [PATCH] Fix PR48648: Handle CLAST assignments.

2011-07-28 Thread Tobias Grosser
On 07/23/2011 12:01 AM, Sebastian Pop wrote: The CLAST produced by CLooG-ISL contains an assignment and GCC chokes on it. The exact CLAST contains an assignment followed by an if: scat_1 = max(0,ceild(T_4-7,8)); if (scat_1<= min(1,floord(T_4-1,8))) { S7(scat_1); } This is equivalent to a lo

Re: [PATCH] Fix PR48648: Handle CLAST assignments.

2011-07-26 Thread Sebastian Pop
On Sat, Jul 23, 2011 at 04:59, Richard Guenther wrote: > On Sat, Jul 23, 2011 at 1:01 AM, Sebastian Pop wrote: >> The CLAST produced by CLooG-ISL contains an assignment and GCC chokes >> on it.  The exact CLAST contains an assignment followed by an if: >> >> scat_1 = max(0,ceild(T_4-7,8)); >> if

Re: [PATCH] Fix PR48648: Handle CLAST assignments.

2011-07-23 Thread Richard Guenther
On Sat, Jul 23, 2011 at 1:01 AM, Sebastian Pop wrote: > The CLAST produced by CLooG-ISL contains an assignment and GCC chokes > on it.  The exact CLAST contains an assignment followed by an if: > > scat_1 = max(0,ceild(T_4-7,8)); > if (scat_1 <= min(1,floord(T_4-1,8))) { >  S7(scat_1); > } > > Thi

[PATCH] Fix PR48648: Handle CLAST assignments.

2011-07-22 Thread Sebastian Pop
The CLAST produced by CLooG-ISL contains an assignment and GCC chokes on it. The exact CLAST contains an assignment followed by an if: scat_1 = max(0,ceild(T_4-7,8)); if (scat_1 <= min(1,floord(T_4-1,8))) { S7(scat_1); } This is equivalent to a loop that iterates only once, and so CLooG genera