public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
* r227907 and AIX 5.[23]
@ 2018-07-25 17:16 Albert Chin
  2018-07-25 17:31 ` David Edelsohn
  0 siblings, 1 reply; 4+ messages in thread
From: Albert Chin @ 2018-07-25 17:16 UTC (permalink / raw)
  To: gcc

r227907 had the following change:
Index: aix61.h
===================================================================
--- aix61.h     (revision 227906)
+++ aix61.h     (revision 227907)
@@ -167,7 +167,7 @@
    %{!maix64:\
      %{pthread:%{pg:gcrt0_r%O%s}%{!pg:%{p:mcrt0_r%O%s}%{!p:crt0_r%O%s}}}\
      %{!pthread:%{pg:gcrt0%O%s}%{!pg:%{p:mcrt0%O%s}%{!p:crt0%O%s}}}}}\
-   %{shared:crtcxa_s%O%s;:crtcxa%O%s}"
+   %{shared:crtcxa_s%O%s;:crtcxa%O%s} crtdbase%O%s"
 
 /* AIX V5 typedefs ptrdiff_t as "long" while earlier releases used "int".  */

In trying to build gcc-8.1.0 on AIX 5.3 (cf. PR86553), I looked at how
libgcc_s.a was built and, on AIX 6 and 7, crtdbase was linked in,
providing __gcc_unwind_dbase (and crtcxa_s for __dso_handle). However,
on AIX 5.3, this file is never included because
gcc/config/rs6000/aix53.h has:
  #undef STARTFILE_SPEC
  #define STARTFILE_SPEC "%{!shared:\
     %{maix64:%{pg:gcrt0_64%O%s}%{!pg:%{p:mcrt0_64%O%s}%{!p:crt0_64%O%s}}}\
     %{!maix64:\
       %{pthread:%{pg:gcrt0_r%O%s}%{!pg:%{p:mcrt0_r%O%s}%{!p:crt0_r%O%s}}}\
       %{!pthread:%{pg:gcrt0%O%s}%{!pg:%{p:mcrt0%O%s}%{!p:crt0%O%s}}}}}"

A patch similar to the above must also be made to aix53.h for 8.1.0 to
build successfully on AIX 5.3. At the moment, GCC 5+ doesn't build on
AIX 5.3 because of the above.

-- 
albert chin (china@thewrittenword.com)

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: r227907 and AIX 5.[23]
  2018-07-25 17:16 r227907 and AIX 5.[23] Albert Chin
@ 2018-07-25 17:31 ` David Edelsohn
  2018-07-25 18:58   ` Albert Chin
  2018-07-25 22:42   ` Marc Glisse
  0 siblings, 2 replies; 4+ messages in thread
From: David Edelsohn @ 2018-07-25 17:31 UTC (permalink / raw)
  To: GCC Development

AIX 5.3 no longer is under supported or maintained.

- David

On Wed, Jul 25, 2018 at 1:13 PM Albert Chin
<gcc@mlists.thewrittenword.com> wrote:
>
> r227907 had the following change:
> Index: aix61.h
> ===================================================================
> --- aix61.h     (revision 227906)
> +++ aix61.h     (revision 227907)
> @@ -167,7 +167,7 @@
>     %{!maix64:\
>       %{pthread:%{pg:gcrt0_r%O%s}%{!pg:%{p:mcrt0_r%O%s}%{!p:crt0_r%O%s}}}\
>       %{!pthread:%{pg:gcrt0%O%s}%{!pg:%{p:mcrt0%O%s}%{!p:crt0%O%s}}}}}\
> -   %{shared:crtcxa_s%O%s;:crtcxa%O%s}"
> +   %{shared:crtcxa_s%O%s;:crtcxa%O%s} crtdbase%O%s"
>
>  /* AIX V5 typedefs ptrdiff_t as "long" while earlier releases used "int".  */
>
> In trying to build gcc-8.1.0 on AIX 5.3 (cf. PR86553), I looked at how
> libgcc_s.a was built and, on AIX 6 and 7, crtdbase was linked in,
> providing __gcc_unwind_dbase (and crtcxa_s for __dso_handle). However,
> on AIX 5.3, this file is never included because
> gcc/config/rs6000/aix53.h has:
>   #undef STARTFILE_SPEC
>   #define STARTFILE_SPEC "%{!shared:\
>      %{maix64:%{pg:gcrt0_64%O%s}%{!pg:%{p:mcrt0_64%O%s}%{!p:crt0_64%O%s}}}\
>      %{!maix64:\
>        %{pthread:%{pg:gcrt0_r%O%s}%{!pg:%{p:mcrt0_r%O%s}%{!p:crt0_r%O%s}}}\
>        %{!pthread:%{pg:gcrt0%O%s}%{!pg:%{p:mcrt0%O%s}%{!p:crt0%O%s}}}}}"
>
> A patch similar to the above must also be made to aix53.h for 8.1.0 to
> build successfully on AIX 5.3. At the moment, GCC 5+ doesn't build on
> AIX 5.3 because of the above.
>
> --
> albert chin (china@thewrittenword.com)

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: r227907 and AIX 5.[23]
  2018-07-25 17:31 ` David Edelsohn
@ 2018-07-25 18:58   ` Albert Chin
  2018-07-25 22:42   ` Marc Glisse
  1 sibling, 0 replies; 4+ messages in thread
From: Albert Chin @ 2018-07-25 18:58 UTC (permalink / raw)
  To: GCC Development

On Wed, Jul 25, 2018 at 01:15:44PM -0400, David Edelsohn wrote:
> AIX 5.3 no longer is under supported or maintained.

Ok. Well, we can now build 8.1 with this change so we'll update the
PR and leave it to someone else to decide if the patch should be
merged.

-- 
albert chin (china@thewrittenword.com)

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: r227907 and AIX 5.[23]
  2018-07-25 17:31 ` David Edelsohn
  2018-07-25 18:58   ` Albert Chin
@ 2018-07-25 22:42   ` Marc Glisse
  1 sibling, 0 replies; 4+ messages in thread
From: Marc Glisse @ 2018-07-25 22:42 UTC (permalink / raw)
  To: David Edelsohn; +Cc: GCC Development

On Wed, 25 Jul 2018, David Edelsohn wrote:

> AIX 5.3 no longer is under supported or maintained.

If gcc-5+ fails to build on AIX 5.3 and patches to make it compile are
not welcome, maybe some cleanup removing aix43.h, aix5*.h and whatever
configure bits could help clarify things? Only when someone has the time, 
of course.

-- 
Marc Glisse

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2018-07-25 19:38 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-07-25 17:16 r227907 and AIX 5.[23] Albert Chin
2018-07-25 17:31 ` David Edelsohn
2018-07-25 18:58   ` Albert Chin
2018-07-25 22:42   ` Marc Glisse

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).