Richard, OK for backport to GCC-13?
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. */