Test the build options

Add new build target: test-buildoptions
Now every build option is tested on the proper targets. These are the native
and the MCU targets in release mode and unittests.
Therefore the USE_COMPILER_DEFAULT_LIBC build option is refactored.
This check is integrated into the precommit process.

JerryScript-DCO-1.0-Signed-off-by: Zsolt Borbély [email protected]

Author: Zsolt Borbély

CMakeLists.txt

@@ -113,9 +113,9 @@ project (Jerry C ASM)
   set(MCU_SCRIPT_GENERATED_HEADER ${CMAKE_BINARY_DIR}/generated.h)
 
 # Should we use external libc?
- if(DEFINED USE_COMPILER_DEFAULT_LIBC AND USE_COMPILER_DEFAULT_LIBC STREQUAL "YES")
+ if(DEFINED COMPILER_DEFAULT_LIBC AND COMPILER_DEFAULT_LIBC STREQUAL "ON")
   if(DEFINED EXTERNAL_LIBC_INTERFACE AND NOT EXTERNAL_LIBC_INTERFACE STREQUAL "UNDEFINED")
-    message(FATAL_ERROR "EXTERNAL_LIBC_INTERFACE='${EXTERNAL_LIBC_INTERFACE}' should not be set in case compiler's default libc is used (USE_COMPILER_DEFAULT_LIBC=YES)")
+    message(FATAL_ERROR "EXTERNAL_LIBC_INTERFACE='${EXTERNAL_LIBC_INTERFACE}' should not be set in case compiler's default libc is used (COMPILER_DEFAULT_LIBC=ON)")
   endif()
 
   set(USE_JERRY_LIBC FALSE)

Makefile

@@ -81,11 +81,15 @@ BUILD_NAME:=
    BUILD_NAME:=$(BUILD_NAME)-ALL_IN_ONE-$(ALL_IN_ONE)
   endif
 
- # Flag, indicating whether to use compiler's default libc (YES / NO)
-  ifneq ($(USE_COMPILER_DEFAULT_LIBC),)
-   CMAKE_DEFINES:=$(CMAKE_DEFINES) -DUSE_COMPILER_DEFAULT_LIBC=$(USE_COMPILER_DEFAULT_LIBC)
+ # Flag, indicating whether to use compiler's default libc (ON / OFF)
+  ifneq ($(COMPILER_DEFAULT_LIBC),)
+   CMAKE_DEFINES:=$(CMAKE_DEFINES) -DCOMPILER_DEFAULT_LIBC=$(COMPILER_DEFAULT_LIBC)
   endif
 
+# For testing build-options
+export BUILD_OPTIONS_TEST_MCU := LTO ALL_IN_ONE
+export BUILD_OPTIONS_TEST_NATIVE := $(BUILD_OPTIONS_TEST_MCU) VALGRIND VALGRIND_FREYA COMPILER_DEFAULT_LIBC
+
 # Directories
 export ROOT_DIR := $(shell pwd)
 export BUILD_DIR_PREFIX := $(ROOT_DIR)/build/obj
@@ -121,6 +125,15 @@ export JERRY_TEST_TARGETS_CP := \
   $(foreach __MODE,$(DEBUG_MODES) $(RELEASE_MODES), \
     $(__MODE).$(NATIVE_SYSTEM)-cp)
 
+# Build-options test targets
+export JERRY_BUILD_OPTIONS_TEST_TARGETS := \
+  $(foreach __MODE,$(RELEASE_MODES), \
+    $(__MODE).$(NATIVE_SYSTEM) \
+      $(foreach __SYSTEM,$(MCU_SYSTEMS), \
+        $(__MODE).mcu_$(__SYSTEM)-$(firstword $(MCU_MODS))))
+
+JERRY_BUILD_OPTIONS_TEST_TARGETS += unittests
+
 # JS test suites (in the format of id:path)
 export JERRY_TEST_SUITE_J := j:$(ROOT_DIR)/tests/jerry
 export JERRY_TEST_SUITE_JTS := jts:$(ROOT_DIR)/tests/jerry-test-suite
@@ -304,6 +317,10 @@ check-vera:
 check-cppcheck:
 	$(Q) $(call SHLOG,./tools/check-cppcheck.sh,$(OUT_DIR)/cppcheck.log,Cppcheck)
 
+.PHONY: test-buildoptions
+test-buildoptions:
+	$(Q) $(call SHLOG,./tools/test-buildoptions.sh "$(JERRY_BUILD_OPTIONS_TEST_TARGETS)" "$(BUILD_OPTIONS_TEST_NATIVE)" "$(BUILD_OPTIONS_TEST_MCU)",$(OUT_DIR)/test-buildoptions.log,Buildoptions test)
+
 .PHONY: build
 build: build.$(NATIVE_SYSTEM) $(foreach __SYSTEM,$(MCU_SYSTEMS),build.mcu_$(__SYSTEM))
 
@@ -332,6 +349,8 @@ precommit: prerequisites
 	$(Q)+$(MAKE) --no-print-directory clean
 	$(Q) echo "Running checks..."
 	$(Q)+$(MAKE) --no-print-directory check-signed-off check-vera check-cppcheck
+	$(Q) echo "...testing build-options..."
+	$(Q)+$(MAKE) --no-print-directory test-buildoptions
 	$(Q) echo "...building engine..."
 	$(Q)+$(MAKE) --no-print-directory build
 	$(Q) echo "...building and running unit tests..."

targets/mbedk64f/Makefile.mbedk64f

@@ -41,7 +41,7 @@ jerry:
 	 -DENABLE_LTO=OFF \
 	 -DENABLE_VALGRIND=OFF \
 	 -DCMAKE_TOOLCHAIN_FILE=build/configs/toolchain_external.cmake \
-	 -DUSE_COMPILER_DEFAULT_LIBC=YES \
+	 -DCOMPILER_DEFAULT_LIBC=ON \
 	 -DEXTERNAL_CMAKE_SYSTEM_PROCESSOR=arm7-m \
 	 -DEXTERNAL_CMAKE_C_COMPILER=arm-none-eabi-gcc \
 	 -DEXTERNAL_COMPILE_FLAGS="$(EXT_CFLAGS)" \

tools/test-buildoptions.sh

@@ -0,0 +1,40 @@
+#!/bin/bash
+
+# Copyright 2016 Samsung Electronics Co., Ltd.
+# Copyright 2016 University of Szeged
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+TARGETS=$1
+shift
+
+BUILD_OPTIONS_NATIVE=$1
+shift
+
+BUILD_OPTIONS_MCU=$1
+shift
+
+for target in $TARGETS
+do
+  if [[ "$target" =~ "mcu" ]]
+  then
+    current_options=$BUILD_OPTIONS_MCU
+  else
+    current_options=$BUILD_OPTIONS_NATIVE
+  fi
+
+  for option in $current_options
+  do
+    make $option=ON $target || exit 1
+  done
+done