Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-14 Thread Tom de Vries
On 14/09/15 11:09, Jakub Jelinek wrote: On Fri, Sep 11, 2015 at 03:28:01PM +0200, Tom de Vries wrote: On 11/09/15 12:57, Jakub Jelinek wrote: On Fri, Sep 11, 2015 at 12:55:00PM +0200, Tom de Vries wrote: Hi, this patch adds a param parloops-schedule=<0-4>, which sets the omp schedule for

Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-14 Thread Tom de Vries
On 14/09/15 16:28, Tom de Vries wrote: On 14/09/15 11:09, Jakub Jelinek wrote: On Fri, Sep 11, 2015 at 03:28:01PM +0200, Tom de Vries wrote: On 11/09/15 12:57, Jakub Jelinek wrote: On Fri, Sep 11, 2015 at 12:55:00PM +0200, Tom de Vries wrote: Hi, this patch adds a param

Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-14 Thread Tom de Vries
On 14/09/15 10:52, Bernd Schmidt wrote: I'm curious why this would be a param rather than a -f option. Hi Bernd, parloops-chunk-size is also a param, so I think it would make sense to have parloops-schedule as a param as well. [ So, in order for parloops to generate: #pragma omp for

Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-14 Thread Jakub Jelinek
On Fri, Sep 11, 2015 at 03:28:01PM +0200, Tom de Vries wrote: > On 11/09/15 12:57, Jakub Jelinek wrote: > >On Fri, Sep 11, 2015 at 12:55:00PM +0200, Tom de Vries wrote: > >>>Hi, > >>> > >>>this patch adds a param parloops-schedule=<0-4>, which sets the omp > >>>schedule > >>>for loops

Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-14 Thread Bernd Schmidt
On 09/11/2015 03:28 PM, Tom de Vries wrote: This patch adds handling of a DEFPARAMENUM macro, which is similar to the DEFPARAM macro, but allows the values to be named. So the definition of param parloop-schedule becomes: ... DEFPARAMENUM PARAM_PARLOOPS_SCHEDULE,

Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-11 Thread Jakub Jelinek
On Fri, Sep 11, 2015 at 12:55:00PM +0200, Tom de Vries wrote: > Hi, > > this patch adds a param parloops-schedule=<0-4>, which sets the omp schedule > for loops paralellized by parloops. > > The <0-4> maps onto . > > Bootstrapped and reg-tested on x86_64. >

Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-11 Thread Tom de Vries
On 11/09/15 15:28, Tom de Vries wrote: So the definition of param parloop-schedule becomes: ... DEFPARAMENUM PARAM_PARLOOPS_SCHEDULE, "parloops-schedule", "Schedule type of omp schedule for loops parallelized by " "parloops (static, dynamic, guided,

[PATCH][PR67476] Add param parloops-schedule

2015-09-11 Thread Tom de Vries
Hi, this patch adds a param parloops-schedule=<0-4>, which sets the omp schedule for loops paralellized by parloops. The <0-4> maps onto . Bootstrapped and reg-tested on x86_64. OK for trunk? Thanks, - Tom Add param parloops-schedule 2015-09-10 Tom de

Re: [PATCH][PR67476] Add param parloops-schedule

2015-09-11 Thread Tom de Vries
On 11/09/15 12:57, Jakub Jelinek wrote: On Fri, Sep 11, 2015 at 12:55:00PM +0200, Tom de Vries wrote: >Hi, > >this patch adds a param parloops-schedule=<0-4>, which sets the omp schedule >for loops paralellized by parloops. > >The <0-4> maps onto . >