[PATCH][AArch64] Enable AUTOPREFETCHER_WEAK with -mcpu=generic
Richard Earnshaw (lists)
Richard.Earnshaw@arm.com
Thu May 4 10:29:00 GMT 2017
On 05/04/17 13:38, Wilco Dijkstra wrote:
> Many supported cores use the AUTOPREFETCHER_WEAK setting which tries
> to order loads and stores to improve streaming performance. Since significant
> gains were reported in http://patchwork.ozlabs.org/patch/534469/ it seems
> like a good idea to enable this setting too for -mcpu=generic. Since the
> weak model only keeps the order if it doesn't make the schedule worse, it
> should not impact performance adversely on cores that don't show a gain.
> Any objections?
>
> ChangeLog:
> 2017-04-05 Wilco Dijkstra <wdijkstr@arm.com>
>
> * gcc/config/aarch64/aarch64.c (generic_tunings): Update prefetch model.
>
OK. The consensus seems to be in favour of this.
R.
> --
> diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
> index 8b729b1b1f87316e940d7fc657f235a935ffa93e..b249ce2b310707c7ded2827d505ce2ddfcfbf976 100644
> --- a/gcc/config/aarch64/aarch64.c
> +++ b/gcc/config/aarch64/aarch64.c
> @@ -547,7 +547,7 @@ static const struct tune_params generic_tunings =
> 2, /* min_div_recip_mul_df. */
> 0, /* max_case_values. */
> 0, /* cache_line_size. */
> - tune_params::AUTOPREFETCHER_OFF, /* autoprefetcher_model. */
> + tune_params::AUTOPREFETCHER_WEAK, /* autoprefetcher_model. */
> (AARCH64_EXTRA_TUNE_NONE) /* tune_flags. */
> };
>
More information about the Gcc-patches
mailing list