Re: [PATCH] Add fields to struct gomp_thread for debugging purposes

2017-11-23 Thread Jakub Jelinek
On Wed, Nov 22, 2017 at 08:25:35AM -0700, Kevin Buettner wrote: > On Sat, 4 Nov 2017 21:39:14 -0700 > Kevin Buettner wrote: > > > On Tue, 31 Oct 2017 08:03:22 +0100 > > Jakub Jelinek wrote: > > > > > On Mon, Oct 30, 2017 at 04:06:15PM -0700, Kevin Buettner

Re: [PATCH] Add fields to struct gomp_thread for debugging purposes

2017-11-22 Thread Kevin Buettner
On Sat, 4 Nov 2017 21:39:14 -0700 Kevin Buettner wrote: > On Tue, 31 Oct 2017 08:03:22 +0100 > Jakub Jelinek wrote: > > > On Mon, Oct 30, 2017 at 04:06:15PM -0700, Kevin Buettner wrote: > > > This patch adds a new member named "pthread_id" to the

Re: [PATCH] Add fields to struct gomp_thread for debugging purposes

2017-11-04 Thread Kevin Buettner
On Tue, 31 Oct 2017 08:03:22 +0100 Jakub Jelinek wrote: > On Mon, Oct 30, 2017 at 04:06:15PM -0700, Kevin Buettner wrote: > > This patch adds a new member named "pthread_id" to the gomp_thread > > struct. It is initialized in team.c. > > That part is reasonable, though it

Re: [PATCH] Add fields to struct gomp_thread for debugging purposes

2017-10-31 Thread Jakub Jelinek
On Mon, Oct 30, 2017 at 04:06:15PM -0700, Kevin Buettner wrote: > This patch adds a new member named "pthread_id" to the gomp_thread > struct. It is initialized in team.c. That part is reasonable, though it is unclear how the debugger will query it (through OMPD, or through hardcoded name lookup

Re: [PATCH] Add fields to struct gomp_thread for debugging purposes

2017-10-30 Thread Kevin Buettner
Below is some additional information about the work I've been doing. It may be useful in understanding where I'm going with my libgomp patch and other patches still to come... I've been working on improvements to gdb, gcc, and libgomp which make GDB able to better access variables in an OpenMP

[PATCH] Add fields to struct gomp_thread for debugging purposes

2017-10-30 Thread Kevin Buettner
This patch adds a new member named "pthread_id" to the gomp_thread struct. It is initialized in team.c. It also adds a field named "parent" which is initialized to the thread which created the thread in question. For non-nested parallel regions, this is always the master thread. These new