On 24/01/2024 09:15, Kyrylo Tkachov wrote: > 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.
Heh, definitely seems against the spirit of the rule. > Anyway, this is ok. Thanks for working through the regressions so diligently. Thanks! Pushed as g:da9647e98aa289ba3aba41cf5bbe14d0f5f27e77. I'll keep an eye on gcc-bugs for any further fallout. Alex > Kyrill > > > Thanks, > > Alex > > > > gcc/ChangeLog: > > > > * config/aarch64/aarch64.opt (-mearly-ldp-fusion): Set default > > to 1. > > (-mlate-ldp-fusion): Likewise.
