diff options
| author | Sang-Hoon Park <sang-hoon.park@arm.com> | 2021-01-14 14:50:25 +0000 |
|---|---|---|
| committer | Sang-Hoon Park <sang-hoon.park@arm.com> | 2021-01-15 12:32:54 +0000 |
| commit | 18fbb92a5e0190c071274d69b3341e9d58ac178e (patch) | |
| tree | 6d6957ea84e36d7a20ccf0c2abca26288ddea9a9 | |
| parent | d05d56da900c5c1226e94a2941fb7ca51a6f4d45 (diff) | |
| download | ComputeLibrary-18fbb92a5e0190c071274d69b3341e9d58ac178e.tar.gz | |
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 <sang-hoon.park@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4860
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
| -rw-r--r-- | SConscript | 7 | ||||
| -rw-r--r-- | 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<T> complex_mul_and_reduce(const SimpleTensor<T> &input, const Simpl output_shape.set(2, Co); SimpleTensor<T> 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) { |