Re: [PATCH, PR68953] Fix pdr accesses order

2016-04-09 Thread Richard Biener
On April 9, 2016 6:07:19 AM GMT+02:00, Sebastian Pop wrote: >On Fri, Apr 8, 2016 at 2:03 AM, Tom de Vries >wrote: >> pdr_0 (read >> in gimple stmt: _9 = yu[_8][0]; >> data accesses: { S_4[i1, i2] -> [1, 0, 1 + i1] } > >data access should be { S_4[i1, i2] -> [1, 1 + i1, 0] } > >> subscript sizes:

Re: [PATCH, PR68953] Fix pdr accesses order

2016-04-08 Thread Sebastian Pop
On Fri, Apr 8, 2016 at 2:03 AM, Tom de Vries wrote: > pdr_0 (read > in gimple stmt: _9 = yu[_8][0]; > data accesses: { S_4[i1, i2] -> [1, 0, 1 + i1] } data access should be { S_4[i1, i2] -> [1, 1 + i1, 0] } > subscript sizes: { [1, i1, 0] : i1 >= 0 and i1 <= 3 } > ) [...] > I'm not really sure h

[PATCH, PR68953] Fix pdr accesses order

2016-04-08 Thread Tom de Vries
Hi, this patch fixes wrong-code PR68953, a graphite 6 regression. I. Consider test.c: ... int yu[4][1] = { { 1 }, { 2 }, { 3 }, { 4 } }; int main (void) { int zh, ro; for (zh = 0; zh < 2; ++zh) for (ro = 0; ro < 3; ++ro) yu[ro][0] = yu[zh + 1][0]; return yu[0][0]; } ... The