Last year I had queued this up for the gomp4.1 merge to mainline, and Jakub said if no one complained it would be OK (or so I recall :-/).

Either way, I believe it's an obvious change.

This is working because sizeof(gomp_thread) is larger than sizeof(goacc_thread), but the extra memory is unnecessary. It's not like goacc_thread has any variable length data at the end.

Committed to trunk.
commit 0c93cf2e4a8fe72b682ceceb344a3e59b9da0a9b
Author: Aldy Hernandez <al...@redhat.com>
Date:   Thu Oct 27 10:27:59 2016 -0700

        * oacc-init.c (goacc_new_thread): Use sizeof of the appropriate
        size when allocating new thread.

diff --git a/libgomp/oacc-init.c b/libgomp/oacc-init.c
index f2325ad..83920b52 100644
--- a/libgomp/oacc-init.c
+++ b/libgomp/oacc-init.c
@@ -325,7 +325,7 @@ acc_shutdown_1 (acc_device_t d)
 static struct goacc_thread *
 goacc_new_thread (void)
 {
-  struct goacc_thread *thr = gomp_malloc (sizeof (struct gomp_thread));
+  struct goacc_thread *thr = gomp_malloc (sizeof (struct goacc_thread));
 
 #if defined HAVE_TLS || defined USE_EMUTLS
   goacc_tls_data = thr;

Reply via email to