On Mon, Aug 31, 2015 at 02:00:10PM +0200, Tom de Vries wrote: > --- a/gcc/omp-low.c > +++ b/gcc/omp-low.c > @@ -6885,6 +6885,22 @@ expand_omp_for_static_nochunk (struct omp_region > *region, > } > } >
Please add a function comment. > +static gphi * > +find_phi_with_arg_on_edge (tree arg, edge e) > +{ > + basic_block bb = e->dest; > + > + for (gphi_iterator gpi = gsi_start_phis (bb); > + !gsi_end_p (gpi); > + gsi_next (&gpi)) > + { > + gphi *phi = gpi.phi (); > + if (PHI_ARG_DEF_FROM_EDGE (phi, e) == arg) > + return phi; > + } > + > + return NULL; > +} > --- /dev/null > +++ b/libgomp/testsuite/libgomp.c/autopar-1-chunk-size.c > @@ -0,0 +1,44 @@ > +/* { dg-do run } */ > +/* { dg-additional-options "-ftree-parallelize-loops=4 -ffast-math --param > parloops-chunk-size=100" } */ Similarly to previous patch, just use autopar-2.c as filename and #include "autopar-1.c" rather than duplicating it. Ok with those changes. Jakub