diff options
| author | Luis Machado <luis.machado@linaro.org> | 2018-05-23 16:20:30 +0000 |
|---|---|---|
| committer | Luis Machado <luisgpm@gcc.gnu.org> | 2018-05-23 16:20:30 +0000 |
| commit | 59100dfc42bbe92caff61bca1560da4a30f99906 (patch) | |
| tree | 4e9671a83dcf4509e6aa0d832c7fcf438751424a /gcc/params.def | |
| parent | cf290ea3255625513b6ad5a5c4e189c833a67a92 (diff) | |
| download | gcc-59100dfc42bbe92caff61bca1560da4a30f99906.zip gcc-59100dfc42bbe92caff61bca1560da4a30f99906.tar.gz gcc-59100dfc42bbe92caff61bca1560da4a30f99906.tar.bz2 | |
[Patch 01/02] Introduce prefetch-minimum stride option
This patch adds a new option to control the minimum stride, for a memory
reference, after which the loop prefetch pass may issue software prefetch
hints for. There are two motivations:
* Make the pass less aggressive, only issuing prefetch hints for bigger strides
that are more likely to benefit from prefetching. I've noticed a case in cpu2017
where we were issuing thousands of hints, for example.
* For processors that have a hardware prefetcher, like Falkor, it allows the
loop prefetch pass to defer prefetching of smaller (less than the threshold)
strides to the hardware prefetcher instead. This prevents conflicts between
the software prefetcher and the hardware prefetcher.
I've noticed considerable reduction in the number of prefetch hints and
slightly positive performance numbers. This aligns GCC and LLVM in terms of
prefetch behavior for Falkor.
The default settings should guarantee no changes for existing targets. Those
are free to tweak the settings as necessary.
gcc/ChangeLog:
2018-05-23 Luis Machado <luis.machado@linaro.org>
* config/aarch64/aarch64-protos.h (cpu_prefetch_tune)
<minimum_stride>: New const int field.
* config/aarch64/aarch64.c (generic_prefetch_tune): Update to include
minimum_stride field defaulting to -1.
(exynosm1_prefetch_tune): Likewise.
(thunderxt88_prefetch_tune): Likewise.
(thunderx_prefetch_tune): Likewise.
(thunderx2t99_prefetch_tune): Likewise.
(qdf24xx_prefetch_tune) <minimum_stride>: Set to 2048.
<default_opt_level>: Set to 3.
(aarch64_override_options_internal): Update to set
PARAM_PREFETCH_MINIMUM_STRIDE.
* doc/invoke.texi (prefetch-minimum-stride): Document new option.
* params.def (PARAM_PREFETCH_MINIMUM_STRIDE): New.
* params.h (PARAM_PREFETCH_MINIMUM_STRIDE): Define.
* tree-ssa-loop-prefetch.c (should_issue_prefetch_p): Return false if
stride is constant and is below the minimum stride threshold.
From-SVN: r260617
Diffstat (limited to 'gcc/params.def')
| -rw-r--r-- | gcc/params.def | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/gcc/params.def b/gcc/params.def index 5c4e2c9..6b208bf 100644 --- a/gcc/params.def +++ b/gcc/params.def @@ -795,6 +795,15 @@ DEFPARAM (PARAM_L2_CACHE_SIZE, "The size of L2 cache.", 512, 0, 0) +/* The minimum constant stride beyond which we should use prefetch hints + for. */ + +DEFPARAM (PARAM_PREFETCH_MINIMUM_STRIDE, + "prefetch-minimum-stride", + "The minimum constant stride beyond which we should use prefetch " + "hints for.", + -1, 0, 0) + /* Maximum number of statements in loop nest for loop interchange. */ DEFPARAM (PARAM_LOOP_INTERCHANGE_MAX_NUM_STMTS, |