From 410e21e88db9d98c8144cd93047e506ecd0b7ab4 Mon Sep 17 00:00:00 2001
From: SiCongLi <sicong.li@arm.com>
Date: Fri, 11 Dec 2020 15:07:53 +0000
Subject: Fix baremetal arm_compute_validation build errors

* Add -C flag to instruct preprocessor not to strip comments. This is to
prevent marker comments like '// fall through' that suppresses certain
warnings from being removed.

* Fix unused variable warnings.

* Add M_PI definition that's missing from certain toolchain standard
libraries.

Resolves COMPMID-4054

Change-Id: I1d641db668685d4b678f3d0efed84bfe9e630b4b
Signed-off-by: SiCongLi <sicong.li@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/4692
Reviewed-by: Georgios Pinitas <georgios.pinitas@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Tested-by: Arm Jenkins <bsgcomp@arm.com>
---
 SConstruct | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'SConstruct')

diff --git a/SConstruct b/SConstruct
index b543bc0ebb..6b20ac2177 100644
--- a/SConstruct
+++ b/SConstruct
@@ -166,6 +166,10 @@ elif 'armclang' in cpp_compiler:
 else:
     env.Append(CXXFLAGS = ['-Wlogical-op','-Wnoexcept','-Wstrict-null-sentinel'])
 
+if cpp_compiler == 'g++':
+    # Don't strip comments that could include markers
+    env.Append(CXXFLAGS = ['-C'])
+
 if env['cppthreads']:
     env.Append(CPPDEFINES = [('ARM_COMPUTE_CPP_SCHEDULER', 1)])
 
-- 
cgit v1.2.1