Re: [PATCH 1/5] omp-low: introduce omplow_simd_context

2017-01-18 Thread Alexander Monakov
On Wed, 18 Jan 2017, Jakub Jelinek wrote: > > +/* This structure is part of the interface between > > lower_rec_simd_input_clauses > > + and lower_rec_input_clauses. */ > > + > > +struct omplow_simd_context { > > + tree idx; > > + tree lane; > > + int max_vf; > > + bool is_simt; > > Any re

Re: [PATCH 1/5] omp-low: introduce omplow_simd_context

2017-01-18 Thread Jakub Jelinek
On Wed, Jan 18, 2017 at 06:02:06PM +0300, Alexander Monakov wrote: > > > +struct omplow_simd_context { > > > + tree idx; > > > + tree lane; > > > + int max_vf; > > > + bool is_simt; > > > > Any reason ivar and lvar weren't added there too? > > Yes, they are not part of 'persistently live cont

Re: [PATCH 1/5] omp-low: introduce omplow_simd_context

2017-01-18 Thread Jakub Jelinek
On Tue, Jan 17, 2017 at 11:07:41PM +0300, Alexander Monakov wrote: > In preparation to handle new SIMT privatization in > lower_rec_simd_input_clauses > this patch factors out variables common to this and lower_rec_input_clauses to > a new structure. No functional change intended. > > * om

[PATCH 1/5] omp-low: introduce omplow_simd_context

2017-01-17 Thread Alexander Monakov
In preparation to handle new SIMT privatization in lower_rec_simd_input_clauses this patch factors out variables common to this and lower_rec_input_clauses to a new structure. No functional change intended. * omp-low.c (omplow_simd_context): New struct. Use it... (lower_rec_simd_