Re: Generalizing existing OpenMP infrastructure (was: Cleanup patches, mostly in the realm of OpenMP)

2013-12-16 Thread Jakub Jelinek
On Mon, Dec 16, 2013 at 05:49:26PM +0100, Thomas Schwinge wrote: > > > --- gcc/gimplify.c > > > +++ gcc/gimplify.c > > > @@ -129,6 +129,8 @@ struct gimplify_ctx > > >bool in_cleanup_point_expr; > > > }; > > > > > > +/* Variable remapping context. */ > > > > Please mention there OpenMP (and

Re: Cleanup patches, mostly in the realm of OpenMP

2013-12-16 Thread Thomas Schwinge
Hi! On Mon, 16 Dec 2013 09:58:07 +0100, Jakub Jelinek wrote: > On Fri, Dec 13, 2013 at 11:12:03AM +0100, Thomas Schwinge wrote: > > OK to apply the following cleanup patches to trunk, assuming that testing > > (still running) is fine? Thanks for the review! > > Reflect reality in comment.

Generalizing existing OpenMP infrastructure (was: Cleanup patches, mostly in the realm of OpenMP)

2013-12-16 Thread Thomas Schwinge
Hi! On Mon, 16 Dec 2013 09:58:07 +0100, Jakub Jelinek wrote: > On Fri, Dec 13, 2013 at 11:12:03AM +0100, Thomas Schwinge wrote: > > --- gcc/gimplify.c > > +++ gcc/gimplify.c > > @@ -129,6 +129,8 @@ struct gimplify_ctx > >bool in_cleanup_point_expr; > > }; > > > > +/* Variable remapping con

Re: Cleanup patches, mostly in the realm of OpenMP

2013-12-16 Thread Jakub Jelinek
On Fri, Dec 13, 2013 at 11:12:03AM +0100, Thomas Schwinge wrote: > OK to apply the following cleanup patches to trunk, assuming that testing > (still running) is fine? > gcc/c/ > * c-parser.c (c_parser_omp_parallel): Fix description. > gcc/cp/ > parser.c (cp_parser_omp_para

Cleanup patches, mostly in the realm of OpenMP

2013-12-13 Thread Thomas Schwinge
Hi! OK to apply the following cleanup patches to trunk, assuming that testing (still running) is fine? commit 68c9deec084dff1e5d8caf6a17ec3eec1f8f9f33 Author: Thomas Schwinge Date: Wed Oct 23 19:40:10 2013 +0200 Fix description of OpenMP parallel directive in the C and C++ front ends.