commit 77624b2bd8ef1bbf8bdd944304e5abbe8fca97f1
author Gilles Peskine <Gilles.Peskine@arm.com> Tue Oct 19 21:33:32 2021 +0200
committer Gilles Peskine <Gilles.Peskine@arm.com> Tue Oct 19 21:39:45 2021 +0200
tree e0b0dce90b6259eefe37f00a70443ce2b1940d38
parent 13236707a20766cac6537b1c192239d90397fa1f

Always set a build type for cmake when building for testing

Set the build type to Release (-O2) when running CPU-intensive tests (ssl-opt,
or unit tests with debug features). A build type of Check (-Os) would be best
when the main objective of the build is to check for build errors or warnings
and there aren't many tests to run; in this commit there are no such test
cases to change. Only use cmake with no build type (which results in not
passing a -O option, and thus missing some GCC warnings) when exercising cmake
features.

Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>

tests/scripts/all.sh

diff --git a/tests/scripts/all.sh b/tests/scripts/all.sh
index 945a0f4..3e52e6e 100755
--- a/tests/scripts/all.sh
+++ b/tests/scripts/all.sh
@@ -1266,7 +1266,7 @@
     scripts/config.pl set MBEDTLS_PLATFORM_MEMORY
     scripts/config.pl set MBEDTLS_MEMORY_BACKTRACE
     scripts/config.pl set MBEDTLS_MEMORY_DEBUG
-    CC=gcc cmake .
+    CC=gcc cmake -DCMAKE_BUILD_TYPE:String=Release .
     make
 
     msg "test: MBEDTLS_MEMORY_BUFFER_ALLOC_C and MBEDTLS_MEMORY_BACKTRACE"
@@ -1277,7 +1277,7 @@
     msg "build: default config with memory buffer allocator"
     scripts/config.pl set MBEDTLS_MEMORY_BUFFER_ALLOC_C
     scripts/config.pl set MBEDTLS_PLATFORM_MEMORY
-    CC=gcc cmake .
+    CC=gcc cmake -DCMAKE_BUILD_TYPE:String=Release .
     make
 
     msg "test: MBEDTLS_MEMORY_BUFFER_ALLOC_C"