Message ID | 20160120144759.GB49431@msticlxl57.ims.intel.com |
---|---|
State | New |
Headers | show |
On Wed, Jan 20, 2016 at 05:47:59PM +0300, Ilya Verbin wrote: > OK for trunk? > > libgomp/ > * task.c (gomp_create_target_task): Set firstprivate_copies to NULL. > > diff --git a/libgomp/task.c b/libgomp/task.c > index 0f45c44..38d4e9b 100644 > --- a/libgomp/task.c > +++ b/libgomp/task.c > @@ -683,6 +683,7 @@ gomp_create_target_task (struct gomp_device_descr *devicep, > ttask->state = state; > ttask->task = task; > ttask->team = team; > + ttask->firstprivate_copies = NULL; > task->fn = NULL; > task->fn_data = ttask; > task->final_task = 0; Ok (though, eventually I'd prefer if free (ttask->firstprivate_copies) is only performed for the shared mem async tasks and not other one. Jakub
diff --git a/libgomp/task.c b/libgomp/task.c index 0f45c44..38d4e9b 100644 --- a/libgomp/task.c +++ b/libgomp/task.c @@ -683,6 +683,7 @@ gomp_create_target_task (struct gomp_device_descr *devicep, ttask->state = state; ttask->task = task; ttask->team = team; + ttask->firstprivate_copies = NULL; task->fn = NULL; task->fn_data = ttask; task->final_task = 0;