Re: (Non-)offloading diagnostics

2016-02-26 Thread Martin Jambor
Hi, On Fri, Feb 26, 2016 at 06:51:34PM +0100, Jakub Jelinek wrote: > On Fri, Feb 26, 2016 at 06:18:13PM +0100, Martin Jambor wrote: > > > I'm a proponent of enabling as many useful warnings by default, or if not > > > by default, then with -Wall. -Whsa is enabled by default, and has thus > > > se

Re: (Non-)offloading diagnostics

2016-02-26 Thread Jakub Jelinek
On Fri, Feb 26, 2016 at 06:18:13PM +0100, Martin Jambor wrote: > > I'm a proponent of enabling as many useful warnings by default, or if not > > by default, then with -Wall. -Whsa is enabled by default, and has thus > > set a precedent of doing that. > > I am not sure I'd go as far as "as many as

Re: (Non-)offloading diagnostics

2016-02-26 Thread Martin Jambor
Hi, On Fri, Feb 26, 2016 at 05:46:33PM +0100, Thomas Schwinge wrote: > Hi! > > In light of the -Whsa testsuite patches just posted, I think we first > need to clarify the general policy questions I posted a month ago: > > On Tue, 26 Jan 2016 11:46:14 +0100, I wrote: > > On Thu, 10 Dec 2015 18:51

Re: (Non-)offloading diagnostics

2016-02-26 Thread Thomas Schwinge
Hi! In light of the -Whsa testsuite patches just posted, I think we first need to clarify the general policy questions I posted a month ago: On Tue, 26 Jan 2016 11:46:14 +0100, I wrote: > On Thu, 10 Dec 2015 18:51:48 +0100, Martin Jambor wrote: > > On Mon, Dec 07, 2015 at 12:46:45PM +0100, Jakub

Re: (Non-)offloading diagnostics

2016-01-26 Thread Thomas Schwinge
Hi! On Tue, 26 Jan 2016 14:18:31 +0300 (MSK), Alexander Monakov wrote: > On Tue, 26 Jan 2016, Thomas Schwinge wrote: > > A very similar problem also exists for nvptx offloading (Nathan CCed), > > where we emit similar warnings (enabled by default). As nvptx offloading > > happens during link-ti

Re: (Non-)offloading diagnostics (was: [hsa 0/10] Merge of HSA branch)

2016-01-26 Thread Alexander Monakov
On Tue, 26 Jan 2016, Thomas Schwinge wrote: > A very similar problem also exists for nvptx offloading (Nathan CCed), > where we emit similar warnings (enabled by default). As nvptx offloading > happens during link-time (not compile-time, as with hsa offloading), > these don't affect GCC's compile

(Non-)offloading diagnostics (was: [hsa 0/10] Merge of HSA branch)

2016-01-26 Thread Thomas Schwinge
Hi! On Thu, 10 Dec 2015 18:51:48 +0100, Martin Jambor wrote: > On Mon, Dec 07, 2015 at 12:46:45PM +0100, Jakub Jelinek wrote: > > On Mon, Dec 07, 2015 at 12:17:58PM +0100, Martin Jambor wrote: > > > [...] There are no failing > > > testcases if HSA is not configured. If it is, there are some, a