Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread Paolo Bonzini
Il 13/02/2013 22:12, Steven Bosscher ha scritto: > On Wed, Feb 13, 2013 at 4:54 PM, Jakub Jelinek wrote: >> Hi! >> >> As agreed on in the PR, here is the revertion of 3 commits, so that >> PCH works again for -gstabs and other debug info formats for 4.8 >> release. For 4.9 we should either remove

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread David Edelsohn
On Tue, Feb 19, 2013 at 11:11 AM, Joel Brobecker wrote: > I haven't looked at this part in details, but our recollection is that > it's a matter of configuring GCC with --with-gnu-as and --with-gnu-ld, > together with a small patch to make DWARF the default. AFAIK, the rest > should be working as

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread Joel Brobecker
> Thanks! That's great progress on the Binutils side. You are very welcome. > What is the status of patches for GCC to generate DWARF for AIX XCOFF > and do the AIX assembler and linker recognize, consume and process the > directives produced by GCC correctly? I haven't looked at this part in d

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-19 Thread David Edelsohn
On Thu, Feb 14, 2013 at 7:09 PM, Joel Brobecker wrote: >> >> AFAICT, for gcc+gas it should already work with binutils that include >> >> the AdaCore patch for DWARF support. But this has apparently not been >> >> tested with AIX ld, and there are AdaCore local patches pending. >> >> http://sourcew

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-18 Thread Joel Brobecker
> > >> AFAICT, for gcc+gas it should already work with binutils that include > > >> the AdaCore patch for DWARF support. But this has apparently not been > > >> tested with AIX ld, and there are AdaCore local patches pending. > > >> http://sourceware.org/ml/binutils/2011-04/msg00250.html > > >> htt

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Joel Brobecker
> >> AFAICT, for gcc+gas it should already work with binutils that include > >> the AdaCore patch for DWARF support. But this has apparently not been > >> tested with AIX ld, and there are AdaCore local patches pending. > >> http://sourceware.org/ml/binutils/2011-04/msg00250.html > >> http://www.so

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Tom Tromey
> "Dave" == John David Anglin writes: Dave> I had looked at this a bit in the past. I don't think it's that Dave> difficult to add DWARF2 support Dave> to GCC on hppa. Although we don't support named sections, we can Dave> create named subspaces Dave> for the dwarf info. More of an issue i

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Tristan Gingold
On Feb 14, 2013, at 3:47 PM, Steven Bosscher wrote: > On Thu, Feb 14, 2013 at 2:21 PM, Tristan Gingold wrote: >>> Is there a description for what has to be done in GCC to enable DWARF >>> with AIX as/ld? E.g. is it required to support the ".dwsect" pseudo? >>> Is there already a plan from someone

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Steven Bosscher
On Thu, Feb 14, 2013 at 2:21 PM, Tristan Gingold wrote: >> Is there a description for what has to be done in GCC to enable DWARF >> with AIX as/ld? E.g. is it required to support the ".dwsect" pseudo? >> Is there already a plan from someone to make GCC produce DWARF on >> AIX7? > > Yes, you need to

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread David Edelsohn
On Thu, Feb 14, 2013 at 8:21 AM, Tristan Gingold wrote: > > On Feb 14, 2013, at 12:26 PM, Steven Bosscher wrote: > >> On Wed, Feb 13, 2013 at 10:33 PM, David Edelsohn wrote: >>> The AIX system supports DWARF debugging, but GCC does not generate it >>> on AIX and GDB does not consume it on AIX. >>

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Tristan Gingold
On Feb 14, 2013, at 12:26 PM, Steven Bosscher wrote: > On Wed, Feb 13, 2013 at 10:33 PM, David Edelsohn wrote: >> The AIX system supports DWARF debugging, but GCC does not generate it >> on AIX and GDB does not consume it on AIX. > > Is there a description for what has to be done in GCC to enabl

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Jeff Law
On 02/14/13 03:52, Jakub Jelinek wrote: No, it prevents *reading* of PCH files with -gstabs, so whatever you write into the PCH file is uninteresting. The patch can surely be acompanied by a patch to warn that writing a PCH file is useless in that case, as in your patch, though the spot you cho

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Jeff Law
On 02/13/13 16:17, John David Anglin wrote: On 2013-02-13 3:33 PM, David Edelsohn wrote: Perhaps Dave can explain what would have to be done to move this platform to DWARF2...? I had looked at this a bit in the past. I don't think it's that difficult to add DWARF2 support to GCC on hppa. Al

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Steven Bosscher
On Wed, Feb 13, 2013 at 10:33 PM, David Edelsohn wrote: > The AIX system supports DWARF debugging, but GCC does not generate it > on AIX and GDB does not consume it on AIX. Is there a description for what has to be done in GCC to enable DWARF with AIX as/ld? E.g. is it required to support the ".dw

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Jakub Jelinek
On Thu, Feb 14, 2013 at 11:19:22AM +0100, Steven Bosscher wrote: > On Thu, Feb 14, 2013 at 12:48 AM, Jakub Jelinek wrote: > > On Thu, Feb 14, 2013 at 12:41:30AM +0100, Steven Bosscher wrote: > >> > I agree with David that it might be better to give up pch. > >> > >> That'd be a really a good start.

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Steven Bosscher
On Thu, Feb 14, 2013 at 12:48 AM, Jakub Jelinek wrote: > On Thu, Feb 14, 2013 at 12:41:30AM +0100, Steven Bosscher wrote: >> > I agree with David that it might be better to give up pch. >> >> That'd be a really a good start. > > You can do it with say following patch instead, PCH is just a compile

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Jakub Jelinek
On Thu, Feb 14, 2013 at 12:41:30AM +0100, Steven Bosscher wrote: > > I agree with David that it might be better to give up pch. > > That'd be a really a good start. You can do it with say following patch instead, PCH is just a compile time optimization, so for -gstabs IMHO it is just fine if we j

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Steven Bosscher
On Thu, Feb 14, 2013 at 12:17 AM, John David Anglin wrote: > On 2013-02-13 3:33 PM, David Edelsohn wrote: >> >> Perhaps Dave can explain what would have to be done to move this >> > platform to DWARF2...? >> > > > I had looked at this a bit in the past. I don't think it's that difficult >

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread John David Anglin
On 2013-02-13 3:33 PM, David Edelsohn wrote: Perhaps Dave can explain what would have to be done to move this > platform to DWARF2...? > I had looked at this a bit in the past. I don't think it's that difficult to add DWARF2 support to GCC on hppa. Although we don't support named sect

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Jeff Law
On 02/13/13 13:07, Jakub Jelinek wrote: On Wed, Feb 13, 2013 at 11:23:24AM -0700, Jeff Law wrote: On 02/13/13 08:54, Jakub Jelinek wrote: That looks just like the patch I have here. Yet, I'm still seeing failures: Running target unix/-m32/-gstabs Using /usr/share/dejagnu/baseboards/unix.exp as

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Douglas B Rupp
On 02/13/2013 01:12 PM, Steven Bosscher wrote: * ix86-*-interix* - no solution immediately available, and no-one listed in MAINTAINERS to ask for help, so maybe Doug can say something about this one? Microsoft has pulled the plug on Interix, Windows 7 will be the last version of Windows

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread David Edelsohn
The AIX system supports DWARF debugging, but GCC does not generate it on AIX and GDB does not consume it on AIX. There is no way that I will allow experimental DWARF support to be enabled at the same time STABS support is removed. This is a non-starter. If you want to disable PCH on STABS system

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Steven Bosscher
On Wed, Feb 13, 2013 at 4:54 PM, Jakub Jelinek wrote: > Hi! > > As agreed on in the PR, here is the revertion of 3 commits, so that > PCH works again for -gstabs and other debug info formats for 4.8 > release. For 4.9 we should either remove support for anything non-DWARF, or > hope somebody steps

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Jakub Jelinek
On Wed, Feb 13, 2013 at 11:23:24AM -0700, Jeff Law wrote: > On 02/13/13 08:54, Jakub Jelinek wrote: > That looks just like the patch I have here. Yet, I'm still seeing failures: > > Running target unix/-m32/-gstabs > Using /usr/share/dejagnu/baseboards/unix.exp as board description > file for tar

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Jeff Law
On 02/13/13 08:54, Jakub Jelinek wrote: Hi! As agreed on in the PR, here is the revertion of 3 commits, so that PCH works again for -gstabs and other debug info formats for 4.8 release. For 4.9 we should either remove support for anything non-DWARF, or hope somebody steps up and fixes dbxout.c

[PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-13 Thread Jakub Jelinek
Hi! As agreed on in the PR, here is the revertion of 3 commits, so that PCH works again for -gstabs and other debug info formats for 4.8 release. For 4.9 we should either remove support for anything non-DWARF, or hope somebody steps up and fixes dbxout.c etc. not to emit debug info right away, bu