Hi Alex, > -----Original Message----- > From: Alex Coplan <alex.cop...@arm.com> > Sent: Wednesday, January 24, 2024 8:34 AM > To: gcc-patches@gcc.gnu.org > Cc: Richard Earnshaw <richard.earns...@arm.com>; Richard Sandiford > <richard.sandif...@arm.com>; Kyrylo Tkachov <kyrylo.tkac...@arm.com>; > Jakub Jelinek <ja...@redhat.com> > 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.
