Re: omp cleanup

2015-10-29 Thread Jakub Jelinek
On Wed, Oct 28, 2015 at 01:26:31PM -0700, Nathan Sidwell wrote: > looking at the next thing to merge, I stumbled on code in lower_omp_target > that appears at least confused. > > we have: > if (offloaded || data_region) > { A } > else if (data_region) > new_body = tgt_body; > if (of

omp cleanup

2015-10-28 Thread Nathan Sidwell
Jakub, looking at the next thing to merge, I stumbled on code in lower_omp_target that appears at least confused. we have: if (offloaded || data_region) { A } else if (data_region) new_body = tgt_body; if (offloaded || data_region) { B } which can clearly be simplified to: