Re: [ptx] debug info

2016-03-21 Thread Thomas Schwinge
Hi! On Thu, 10 Mar 2016 16:14:34 +0300 (MSK), Alexander Monakov wrote: > On Thu, 10 Mar 2016, Nathan Sidwell wrote: > > Hm, something must have changed since I found that sorry neccessary. > > As I already said in my opening sentence (not quoted in your response), you >

Re: [ptx] debug info

2016-03-10 Thread Alexander Monakov
On Thu, 10 Mar 2016, Nathan Sidwell wrote: > Hm, something must have changed since I found that sorry neccessary. As I already said in my opening sentence (not quoted in your response), you removed the unnecessary override. This is exactly what lets toplevel code see requested debug format now,

Re: [ptx] debug info

2016-03-10 Thread Nathan Sidwell
On 03/10/16 07:52, Alexander Monakov wrote: On Wed, 9 Mar 2016, Nathan Sidwell wrote: Furthermore, this is not useful without support in libgomp/plugin-nvptx.c and nvptx-none-run.c (PTX JIT does not propagate lineinfo by default). Would you like me to submit patches for those? please.

Re: [ptx] debug info

2016-03-10 Thread Nathan Sidwell
On 03/09/16 11:41, Alexander Monakov wrote: :|x86_64-pc-linux-gnu-accel-nvptx-none-gcc -xc - -S -o- -gstabs :1:0: sorry, unimplemented: stabs debug format not supported cc1: error: target system does not support the ‘stabs’ debug format :|x86_64-pc-linux-gnu-accel-nvptx-none-gcc -xc - -S -o-

Re: [ptx] debug info

2016-03-10 Thread Alexander Monakov
On Wed, 9 Mar 2016, Nathan Sidwell wrote: > > Furthermore, this is not useful without support in libgomp/plugin-nvptx.c > > and nvptx-none-run.c (PTX JIT does not propagate lineinfo by default). > > Would you like me to submit patches for those? > > please. Here's the pull request for

Re: [ptx] debug info

2016-03-09 Thread Alexander Monakov
On Wed, 9 Mar 2016, Nathan Sidwell wrote: > On 03/09/16 09:55, Alexander Monakov wrote: > > The preceding code special-casing response to -gstabs can also be removed > > after this patch. Should I submit the (trivial) removal patch? > > No. I found that necessary to stop the testsuite testing

Re: [ptx] debug info

2016-03-09 Thread Nathan Sidwell
On 03/09/16 09:55, Alexander Monakov wrote: Hello Nathan, On Wed, 9 Mar 2016, Nathan Sidwell wrote: I've committed this to trunk, to remove the squashing of debug information. It appears to function correctly. I'd had this patch for a while, but forgot to commit it. The preceding code

Re: [ptx] debug info

2016-03-09 Thread Alexander Monakov
Hello Nathan, On Wed, 9 Mar 2016, Nathan Sidwell wrote: > I've committed this to trunk, to remove the squashing of debug information. > It appears to function correctly. > > I'd had this patch for a while, but forgot to commit it. The preceding code special-casing response to -gstabs can also