From 554086389c18b2907a87c3a3c8d136e40574af8d Mon Sep 17 00:00:00 2001 From: "Bobby R. Bruce" Date: Thu, 3 Dec 2020 15:48:10 -0800 Subject: [PATCH] tests,arch-arm: Replaced X86 clang fast build for ARM_MESI_THREE Given ARM_MESI_THREE_Level_HTM is a superset of ARM, it is logical to compile it with clang and with 'fast' compiler optimizations when checking for errors. This increases our test coverage Change-Id: I320291cd543e5db8f543edd81facc91f3f52e17b Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/38315 Reviewed-by: Bobby R. Bruce Maintainer: Bobby R. Bruce Tested-by: kokoro --- tests/jenkins/presubmit.sh | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/tests/jenkins/presubmit.sh b/tests/jenkins/presubmit.sh index 7b59f55fb6..e7274ee870 100755 --- a/tests/jenkins/presubmit.sh +++ b/tests/jenkins/presubmit.sh @@ -65,10 +65,11 @@ docker run -u $UID:$GID --volume $(pwd):$(pwd) -w $(pwd) --rm \ # DOCKER_IMAGE_ALL_DEP compiles gem5.opt with GCC. We run a compilation of # gem5.fast on the Clang compiler to ensure changes are compilable with the -# clang compiler. Due to the costs of compilation, we only compile X86 -# at this point. Further compiler tests are carried out as part of our weekly -# "Compiler Checks" tests: http://jenkins.gem5.org/job/Compiler-Checks. +# clang compiler. Due to the costs of compilation, we only compile +# ARM_MESI_Three_Level_HTM at this point. Further compiler tests are carried +# out as part of our weekly "Compiler Checks" tests: +# http://jenkins.gem5.org/job/Compiler-Checks. rm -rf build docker run -u $UID:$GID --volume $(pwd):$(pwd) -w $(pwd) --rm \ "${DOCKER_IMAGE_CLANG_COMPILE}" /usr/bin/env python3 /usr/bin/scons \ - build/X86/gem5.fast -j4 --no-compress-debug + build/ARM_MESI_Three_Level_HTM/gem5.fast -j4 --no-compress-debug