Re: [PATCH RFC] c++: implement C++17 hardware interference size

[Martin Liška]

On 9/14/21 09:56, Christophe LYON via Gcc-patches wrote:

So adjustment is needed for both arm and aarch64 targets

Hello.

I noticed the same problem and I've got a patch candidate for it.

What do you think about it?
Martin
From 2ecedfe5cc421dd36f5770b04553343ecebd3430 Mon Sep 17 00:00:00 2001
From: Martin Liska <mli...@suse.cz>
Date: Wed, 15 Sep 2021 14:29:52 +0200
Subject: [PATCH] Fix -Werror=interference-size on ARM and Aarch64.

gcc/ChangeLog:

	* config/aarch64/aarch64.c (aarch64_override_options_internal):
	Use default L1 cache line size from params.
	* config/arm/arm.c (arm_option_override): Likewise.
---
 gcc/config/aarch64/aarch64.c | 2 +-
 gcc/config/arm/arm.c         | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/gcc/config/aarch64/aarch64.c b/gcc/config/aarch64/aarch64.c
index 36519ccc5a5..4eb50bdb7d5 100644
--- a/gcc/config/aarch64/aarch64.c
+++ b/gcc/config/aarch64/aarch64.c
@@ -16559,7 +16559,7 @@ aarch64_override_options_internal (struct gcc_options *opts)
 			   256);
       SET_OPTION_IF_UNSET (opts, &global_options_set,
 			   param_construct_interfere_size,
-			   64);
+			   param_l1_cache_line_size);
     }
 
   if (aarch64_tune_params.prefetch->l2_cache_size >= 0)
diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
index 6c6e77fab66..5d14e1e5e9c 100644
--- a/gcc/config/arm/arm.c
+++ b/gcc/config/arm/arm.c
@@ -3688,7 +3688,8 @@ arm_option_override (void)
       SET_OPTION_IF_UNSET (&global_options, &global_options_set,
 			   param_destruct_interfere_size, 64);
       SET_OPTION_IF_UNSET (&global_options, &global_options_set,
-			   param_construct_interfere_size, 64);
+			   param_construct_interfere_size,
+			   param_l1_cache_line_size);
     }
 
   if (current_tune->prefetch.l1_cache_size >= 0)
-- 
2.33.0