[Bug target/80865] broken compilation on Mac OS X 10.5 / powerpc: unrecognizable insn

iains at gcc dot gnu.org gcc-bugzilla@gcc.gnu.org
Thu Oct 5 06:48:00 GMT 2017


https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80865

--- Comment #9 from Iain Sandoe <iains at gcc dot gnu.org> ---
(In reply to Christian Cornelssen from comment #7)
> I have made the time-consuming effort of building and testing gcc-7.2.0 with
> varying subsets of the four patchsets proposed with attachment 42124
> [details].

Thanks for doing this!

I have building work going on here (to repair water damage to my office) and
temporarily had to relocate - at present, I don't have access to my PPC Darwin
machines.  Hopefully, the works will complete in a month or so, and i'll be
back to normal (please be patient - but I don't believe another 7.x release is
scheduled in that time).

Note that the patches solve individual problems, it would be reasonable to
apply them independently.

> * Patchset 1/4 copies stack alignment changes previously applied to aix.h.
>   This seems to cause 15 additional testsuite failures
>   involving gcc/testsuite/gcc.dg/builtin-apply4.c and
>   gcc/testsuite/gcc.dg/torture/stackalign/builtin-apply-4.c.
>   The failures are related to the builtins around __builtin_apply
>   as defined in gcc/builtins.c.
>   I'd leave that patchset out.

The patch was originally required to allow bootstrap to complete, so clearly
some intervening change has made that unnecessary - but I'd like to identify
what (and if there are other implications).

> * Patchset 2/4 turns out to be necessary and sufficient for the non-Ada
>   build to succeed. If you want a minimal patchset, stick to that one.
>   Given as attachment 42304 [details] now.
> * Patchset 3/4 seems reasonable (marking longjmp noreturn),
>   but does not contribute changes to the test summary.

Are you building Fortran?
IIRC, that patch was required for Fortran bootstrap to complete.

> * Patchset 4/4 looks reasonable (selecting Darwin thread headers),
>   but is Ada-related, thus irrelevant for non-Ada builds.

Well, generally I build for all the supported languages including Ada, Fortran
(and Java where supported) so generally I will keep patches to allow all of
those to succeed.

> 
> Attaching the gcc7 testsuite log diff showing the removal of
> alignment test failures when only patchset 2 (attachment 42304 [details]
> instead of attachment 42124 [details]) is used.


More information about the Gcc-bugs mailing list