On Fri, Jan 10, 2014 at 12:51:10PM +0100, Thomas Schwinge wrote:
> > > +static bool
> > > +maybe_fold_stmt (gimple_stmt_iterator *gsi)
> > > +{
> > > + bool changed = false;
> > > + struct gimplify_omp_ctx *ctx;
> > > + for (ctx = gimplify_omp_ctxp; ctx; ctx = ctx->outer_context)
> > > + if (ctx->region_type == ORT_TARGET)
> > > + break;
I think better would be to drop changed var altogether, do return false;
instead of break and then you can just do:
return fold_stmt (gsi);
after the loop.
Ok with those changes.
Jakub
