diff options
author | SiCongLi <sicong.li@arm.com> | 2020-12-11 15:07:53 +0000 |
---|---|---|
committer | SiCong Li <sicong.li@arm.com> | 2020-12-23 10:46:06 +0000 |
commit | 410e21e88db9d98c8144cd93047e506ecd0b7ab4 (patch) | |
tree | 29ad49910db2fe11ee77c31224a61ccb7436a0a5 /SConstruct | |
parent | 2dcffdeda88b59aecc9a4ad6d5fb8b44f9d638b7 (diff) | |
download | ComputeLibrary-410e21e88db9d98c8144cd93047e506ecd0b7ab4.tar.gz |
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>
Diffstat (limited to 'SConstruct')
-rw-r--r-- | SConstruct | 4 |
1 files changed, 4 insertions, 0 deletions
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)]) |