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

Reply via email to