On Thu, Jan 16, 2014 at 12:29:44PM -0500, Jason Merrill wrote: > >+ if (code == CILK_FOR) > >+ { > >+ top_level_body = push_stmt_list (); > >+ top_body = begin_omp_parallel (); > >+ } > > I wouldn't expect the front end to care that Cilk for is implemented > using a parallel call; can't we bring that in at lowering time?
The gimplifier already cares, so if it shouldn't be added early in the FE, it must be added during genericization. Unless we treat CILK_FOR as implicit parallel in the gimplifier, but I'd say that it is better to have it expressed as parallel with cilk_for nested in it and the combined flag set, so that omp-low.c can then emit it together. Jakub