Hi Alex, > -----Original Message----- > From: Alex Coplan <[email protected]> > Sent: Wednesday, January 24, 2024 8:34 AM > To: [email protected] > Cc: Richard Earnshaw <[email protected]>; Richard Sandiford > <[email protected]>; Kyrylo Tkachov <[email protected]>; > Jakub Jelinek <[email protected]> > Subject: [PATCH] aarch64: Re-enable ldp/stp fusion pass > > Hi, > > Since, to the best of my knowledge, all reported regressions related to > the ldp/stp fusion pass have now been fixed, and PGO+LTO bootstrap with > --enable-languages=all is working again with the passes enabled, this > patch turns the passes back on by default, as agreed with Jakub here: > > https://gcc.gnu.org/pipermail/gcc-patches/2024-January/642478.html > > Bootstrapped/regtested on aarch64-linux-gnu, OK for trunk? >
If we were super-pedantic about the GCC rules we could say that this is a revert of 8ed77a2356c3562f96c64f968e7529065c128c6a and therefore: "Similarly, no outside approval is needed to revert a patch that you checked in." 😊 But that would go against the spirit of the rule. Anyway, this is ok. Thanks for working through the regressions so diligently. Kyrill > Thanks, > Alex > > gcc/ChangeLog: > > * config/aarch64/aarch64.opt (-mearly-ldp-fusion): Set default > to 1. > (-mlate-ldp-fusion): Likewise.
