Cherry-picked this to gcc-13. Thanks, Di Zhao
> -----Original Message----- > From: Richard Sandiford <richard.sandif...@arm.com> > Sent: Monday, June 26, 2023 10:28 PM > To: Philipp Tomsich <philipp.toms...@vrull.eu> > Cc: Di Zhao OS via Gcc-patches <gcc-patches@gcc.gnu.org>; Di Zhao OS > <diz...@os.amperecomputing.com> > Subject: Re: [PATCH] Change fma_reassoc_width tuning for ampere1 > > Philipp Tomsich <philipp.toms...@vrull.eu> writes: > > Richard, > > > > OK for backport to GCC-13? > > Yeah, OK for GCC 13 too. > > Thanks, > Richard > > > Thanks, > > Philipp. > > > > On Thu, 22 Jun 2023 at 16:18, Richard Sandiford via Gcc-patches > > <gcc-patches@gcc.gnu.org> wrote: > >> > >> Di Zhao OS via Gcc-patches <gcc-patches@gcc.gnu.org> writes: > >> > This patch enables reassociation of floating-point additions on ampere1. > >> > This brings about 1% overall benefit on spec2017 fprate cases. (There > >> > are minor regressions in 510.parest_r and 508.namd_r, analyzed here: > >> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110279 .) > >> > > >> > Bootstrapped and tested on aarch64-unknown-linux-gnu. Is this OK for > trunk? > >> > > >> > Thanks, > >> > Di Zhao > >> > > >> > gcc/ChangeLog: > >> > > >> > * config/aarch64/aarch64.cc: Change fma_reassoc_width for > ampere1 > >> > >> Thanks, pushed to trunk. > >> > >> Richard > >> > >> > --- > >> > diff --git a/gcc/config/aarch64/aarch64.cc > b/gcc/config/aarch64/aarch64.cc > >> > index d16565b5581..301c9f6c0cd 100644 > >> > --- a/gcc/config/aarch64/aarch64.cc > >> > +++ b/gcc/config/aarch64/aarch64.cc > >> > @@ -1927,7 +1927,7 @@ static const struct tune_params ampere1_tunings = > >> > "32:12", /* loop_align. */ > >> > 2, /* int_reassoc_width. */ > >> > 4, /* fp_reassoc_width. */ > >> > - 1, /* fma_reassoc_width. */ > >> > + 4, /* fma_reassoc_width. */ > >> > 2, /* vec_reassoc_width. */ > >> > 2, /* min_div_recip_mul_sf. */ > >> > 2, /* min_div_recip_mul_df. */
