On Thu, May 26, 2011 at 03:11, Alexey Kravets <kayr...@ispras.ru> wrote: > Here is the patch for this issue. > It is caused by the bug in opencl_get_perfect_nested_loop_depth. > This functions ignores -fgraphite-opencl-ignore-dep flag while > opencl_transform_stmt_list considers this flag. > > This patch adds a new function, which should be used to check for > dependencies in graphite-opencl. I tested it on PolyKernels and on the > block-0.c test from graphite testsuite. > > Is this patch OK for graphite branch? If it is, could you commit it?
The patch looks good. Committed to the graphite branch after make -k check RUNTESTFLAGS=graphite.exp Thanks for fixing this. > > gcc: > 2011-05-26 Alexey Kravets <kayr...@ispras.ru> > > RP middle-end/49147 > * graphite-opencl-codegen.c (opencl_get_perfect_nested_loop_depth): > Use enabled_dependency_in_clast_loop_p instead of > dependency_in_clast_loop_p. > graphite-opencl.c (opencl_transform_stmt_list): Ditto. > (enabled_dependency_in_clast_loop_p): New. > graphite-opencl.h (enabled_dependency_in_clast_loop_p): Export. > > testsuite: > 2011-05-26 Alexey Kravets <kayr...@ispras.ru> > > RP middle-end/49147 > * gcc.dg/graphite/pr49147.c: New test. To increase the number of tests we perform on the graphite-opencl code gen, I was thinking to add the existing block-*.c files from the testsuite to be compiled with the graphite-opencl flags as part of the make check. Sebastian
