| Message ID | CAGWvny=bJYsWW=KFh9OcNoipGRiKegE+nkPM7jJmUrtXkWaapA@mail.gmail.com |
|---|---|
| State | New |
| Headers | show |
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
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
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*-*-*)
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. gcc_GAS_CHECK_FEATURE([explicit relocation support],