[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