Re: [PATCH] OpenMP: Disable GPU threads when only teams are used

2020-07-03 Thread Jakub Jelinek via Gcc-patches
On Thu, Jul 02, 2020 at 10:16:25PM +0100, Andrew Stubbs wrote: > On 02/07/2020 18:00, Jakub Jelinek wrote: > > On Thu, Jul 02, 2020 at 05:15:20PM +0100, Andrew Stubbs wrote: > > > This patch, originally by Kwok, auto-adjusts the default OpenMP target > > > arguments to set num_threads(1) when there

Re: [PATCH] OpenMP: Disable GPU threads when only teams are used

2020-07-02 Thread Andrew Stubbs
On 02/07/2020 18:00, Jakub Jelinek wrote: On Thu, Jul 02, 2020 at 05:15:20PM +0100, Andrew Stubbs wrote: This patch, originally by Kwok, auto-adjusts the default OpenMP target arguments to set num_threads(1) when there are no parallel regions. There may still be multiple teams in this case. The

Re: [PATCH] OpenMP: Disable GPU threads when only teams are used

2020-07-02 Thread Jakub Jelinek via Gcc-patches
On Thu, Jul 02, 2020 at 05:15:20PM +0100, Andrew Stubbs wrote: > This patch, originally by Kwok, auto-adjusts the default OpenMP target > arguments to set num_threads(1) when there are no parallel regions. There > may still be multiple teams in this case. > > The result is that libgomp will not at

[PATCH] OpenMP: Disable GPU threads when only teams are used

2020-07-02 Thread Andrew Stubbs
This patch, originally by Kwok, auto-adjusts the default OpenMP target arguments to set num_threads(1) when there are no parallel regions. There may still be multiple teams in this case. The result is that libgomp will not attempt to launch GPU threads that will never get used. OK to commit?