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. */