Re: [patch] Add counter inits to zero_iter_bb in expand_omp_for_init_counts

2015-10-28 Thread Tom de Vries
On 23/10/15 16:27, Thomas Schwinge wrote: Hi Jakub and Tom! On Thu, 1 Oct 2015 15:37:26 +0200, Tom de Vries wrote: On 01/10/15 14:49, Jakub Jelinek wrote: On Thu, Oct 01, 2015 at 02:46:01PM +0200, Tom de Vries wrote: this patch adds initialization in zero_iter_bb of counters introduced in ex

Re: [patch] Add counter inits to zero_iter_bb in expand_omp_for_init_counts

2015-10-23 Thread Thomas Schwinge
Hi Jakub and Tom! On Thu, 1 Oct 2015 15:37:26 +0200, Tom de Vries wrote: > On 01/10/15 14:49, Jakub Jelinek wrote: > > On Thu, Oct 01, 2015 at 02:46:01PM +0200, Tom de Vries wrote: > >> this patch adds initialization in zero_iter_bb of counters introduced in > >> expand_omp_for_init_counts. > >>

Re: [patch] Add counter inits to zero_iter_bb in expand_omp_for_init_counts

2015-10-01 Thread Tom de Vries
On 01/10/15 14:49, Jakub Jelinek wrote: On Thu, Oct 01, 2015 at 02:46:01PM +0200, Tom de Vries wrote: this patch adds initialization in zero_iter_bb of counters introduced in expand_omp_for_init_counts. This removes the need to set TREE_NO_WARNING on those counters. Why do you think it is a g

Re: [patch] Add counter inits to zero_iter_bb in expand_omp_for_init_counts

2015-10-01 Thread Jakub Jelinek
On Thu, Oct 01, 2015 at 02:46:01PM +0200, Tom de Vries wrote: > this patch adds initialization in zero_iter_bb of counters introduced in > expand_omp_for_init_counts. > > This removes the need to set TREE_NO_WARNING on those counters. Why do you think it is a good idea? I'd be afraid it slows th

[patch] Add counter inits to zero_iter_bb in expand_omp_for_init_counts

2015-10-01 Thread Tom de Vries
Hi, this patch adds initialization in zero_iter_bb of counters introduced in expand_omp_for_init_counts. This removes the need to set TREE_NO_WARNING on those counters. Build on x86_64 and reg-tested with gomp.exp and target-libgomp c.exp. OK for trunk, if bootstrap and reg-test on x86_64 su