From 18fbb92a5e0190c071274d69b3341e9d58ac178e Mon Sep 17 00:00:00 2001 From: Sang-Hoon Park Date: Thu, 14 Jan 2021 14:50:25 +0000 Subject: Fix build error for gcc 10.2+ - Add -Wno flag for graph build environment - Use std::fill_n instead of memset Partially implements: COMPMID-3808 Change-Id: Ibed1f81c2cee8bcb12a6a0aa3fa5030d3a201d69 Signed-off-by: Sang-Hoon Park Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4860 Comments-Addressed: Arm Jenkins Reviewed-by: Georgios Pinitas Tested-by: Arm Jenkins --- SConscript | 7 ++++--- tests/validation/reference/DFT.cpp | 5 +++-- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/SConscript b/SConscript index 229d7d1c26..ffea1b8f69 100644 --- a/SConscript +++ b/SConscript @@ -320,9 +320,10 @@ if env['os'] != 'bare_metal' and not env['standalone']: Depends(arm_compute_so, arm_compute_core_so) Export('arm_compute_so') -arm_compute_graph_env = arm_compute_env.Clone(); -if 'clang++' in cpp_compiler: - arm_compute_graph_env.Append(CXXFLAGS = ['-Wno-pessimizing-move']) +arm_compute_graph_env = arm_compute_env.Clone() + +arm_compute_graph_env.Append(CXXFLAGS = ['-Wno-redundant-move', '-Wno-pessimizing-move']) + arm_compute_graph_a = build_library('arm_compute_graph-static', arm_compute_graph_env, graph_files, static=True, libs = [ arm_compute_a]) Export('arm_compute_graph_a') diff --git a/tests/validation/reference/DFT.cpp b/tests/validation/reference/DFT.cpp index d5f7010a5b..fd126c7d73 100644 --- a/tests/validation/reference/DFT.cpp +++ b/tests/validation/reference/DFT.cpp @@ -268,8 +268,9 @@ SimpleTensor complex_mul_and_reduce(const SimpleTensor &input, const Simpl output_shape.set(2, Co); SimpleTensor dst(output_shape, input.data_type(), input.num_channels()); - // MemSet dst memory to zero - std::memset(dst.data(), 0, dst.size()); + // dst memory to zero + const auto total_element_count = dst.num_channels() * dst.num_elements(); + std::fill_n(dst.data(), total_element_count, 0); for(uint32_t b = 0; b < N; ++b) { -- cgit v1.2.1