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.
