[PATCH] gcc/configure test for AIX DWARF

[PATCH] gcc/configure test for AIX DWARF

[David Edelsohn]

AIX7 has added support for DWARF to XCOFF, but complete and correct
support did not occur with a single update and the initial release of
AIX7.  The initial support defined a subset of common DWARF debug
sections.  A later update added most of the remaining sections for
location lists and frames, but the AIX Assembler did not correctly
handle references to labels generated by GCC.

This patch updates the gcc/configure test for the extended DWARF
support to ensure that the AIX toolchain correctly handles the label
reference.

Bootstrapped on powerpc-ibm-aix7.1.2.0 with and without the corrected assembler.

Okay?

Thanks, David

* configure.ac (gcc_cv_as_dwloc): Test support for debug frame section
label reference.
* configure: Regenerate.

Index: configure.ac
===================================================================
--- configure.ac        (revision 232532)
+++ configure.ac        (working copy)
@@ -4384,7 +4384,7 @@

     case $target in
       *-*-aix*)
-       gcc_GAS_CHECK_FEATURE([.ref support],
+       gcc_GAS_CHECK_FEATURE([AIX .ref support],
          gcc_cv_as_aix_ref, [2,21,0],,
          [     .csect stuff[[rw]]
             stuff:
@@ -4395,19 +4395,17 @@
          [AC_DEFINE(HAVE_AS_REF, 1,
            [Define if your assembler supports .ref])])
        ;;
-    esac

-    case $target in
-      *-*-aix*)
-       gcc_GAS_CHECK_FEATURE([dwarf location lists section support],
+       gcc_GAS_CHECK_FEATURE([AIX DWARF location lists section support],
          gcc_cv_as_aix_dwloc, [2,21,0],,
-         [     .dwsect 0xB0000
+         [     .dwsect 0xA0000
+       Lframe..0:
+               .vbyte 4,Lframe..0:
          ],,
          [AC_DEFINE(HAVE_XCOFF_DWARF_EXTRAS, 1,
-           [Define if your assembler supports .dwsect 0xB0000])])
+           [Define if your assembler supports AIX debug frame section
label reference.])])
        ;;
     esac
-    ;;

   mips*-*-*)
     gcc_GAS_CHECK_FEATURE([explicit relocation support],

Re: [PATCH] gcc/configure test for AIX DWARF

[Bernd Schmidt]

On 01/18/2016 08:30 PM, David Edelsohn wrote:
> Bootstrapped on powerpc-ibm-aix7.1.2.0 with and without the corrected assembler.
>
> Okay?

The changes seem to be in *-*-aix blocks, so as far as I'm concerned you 
are the maintainer and can check this in. One question though:

>         ;;
>      esac
> -    ;;
>
>    mips*-*-*)
>      gcc_GAS_CHECK_FEATURE([explicit relocation support],

Did you intend to remove this line? This looks odd.


Bernd

Re: [PATCH] gcc/configure test for AIX DWARF

[David Edelsohn]

On Thu, Jan 21, 2016 at 12:47 PM, Bernd Schmidt <bschmidt@redhat.com> wrote:
> On 01/18/2016 08:30 PM, David Edelsohn wrote:
>>
>> Bootstrapped on powerpc-ibm-aix7.1.2.0 with and without the corrected
>> assembler.
>>
>> Okay?
>
>
> The changes seem to be in *-*-aix blocks, so as far as I'm concerned you are
> the maintainer and can check this in. One question though:
>
>>         ;;
>>      esac
>> -    ;;
>>
>>    mips*-*-*)
>>      gcc_GAS_CHECK_FEATURE([explicit relocation support],
>
>
> Did you intend to remove this line? This looks odd.

The ";;" were mis-matched in the patch I sent.  It is correct on trunk.

- David