Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread David Edelsohn
On Thu, Sep 24, 2015 at 8:01 PM, Mike Stump wrote: > On Sep 24, 2015, at 11:40 AM, David Edelsohn wrote: >> AIX added support for the initially missing DWARF sections. >> Bad news: The support is in an AIX service pack whose presence on a >> system requires effort to determine. > > So, we faced t

Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread Mike Stump
On Sep 24, 2015, at 11:40 AM, David Edelsohn wrote: > AIX added support for the initially missing DWARF sections. > Bad news: The support is in an AIX service pack whose presence on a > system requires effort to determine. So, we faced this problem at Apple, and we just required that people apply

Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread David Edelsohn
On Thu, Sep 24, 2015 at 4:05 PM, Richard Henderson wrote: >> If the assembler and linker generate errors, it further >> delays deployment in GCC. > > If the assembler generates errors, then surely it's easy to error out at > configure time, printing a note about the service pack. > > So, assuming

Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread Richard Henderson
On 09/24/2015 12:02 PM, David Edelsohn wrote: > If the assembler and linker pass the sections with the additional numbers > through and older DBX doesn't know about the sections, it doesn't matter. Agreed. In that case we simply emit the data all the time and let capable debuggers pick it up. >

Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread David Edelsohn
On Thu, Sep 24, 2015 at 2:49 PM, Richard Henderson wrote: > On 09/24/2015 11:40 AM, David Edelsohn wrote: >> Good news: AIX added support for the initially missing DWARF sections. > > Yay! > >> Bad news: The support is in an AIX service pack whose presence on a >> system requires effort to determi

Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread Richard Henderson
On 09/24/2015 11:40 AM, David Edelsohn wrote: > Good news: AIX added support for the initially missing DWARF sections. Yay! > Bad news: The support is in an AIX service pack whose presence on a > system requires effort to determine. Boo! Well, we've had worse problems with Solaris in the past.

Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread David Edelsohn
>> + else if (TARGET_XCOFF && GET_CODE (x) == SYMBOL_REF >> + && SYMBOL_REF_TLS_MODEL (x) != 0) >> +{ >> + if (SYMBOL_REF_TLS_MODEL (x) == TLS_MODEL_LOCAL_EXEC) >> + fputs ("@le", file); >> + else if (SYMBOL_REF_TLS_MODEL (x) == TLS_MODEL_INITIAL_EXEC) >> + fputs

Re: [PATCH] DWARF support for AIX v4

2015-09-24 Thread Richard Henderson
On 09/23/2015 07:39 PM, David Edelsohn wrote: > Richard and Richard, > > Appended is the updated version of the DWARF support patch for AIX. I > still can split out the length computation into a separate helper > function, but, as I mentioned, it won't apply to the instance that > uses a delta of

[PATCH] DWARF support for AIX v4

2015-09-23 Thread David Edelsohn
Richard and Richard, Appended is the updated version of the DWARF support patch for AIX. I still can split out the length computation into a separate helper function, but, as I mentioned, it won't apply to the instance that uses a delta of two labels. This version sets have_macinfo to False and