Bobby Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/63371?usp=email ) ( 7 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: tests: Update presubmit.sh to compile ALL/gem5.fast ...................................................................... tests: Update presubmit.sh to compile ALL/gem5.fast This part of the Kokoro presubmit tests was designed to ensure gem5 still compiled sucessfully with Clang and to the '.fast' variant. ARM was chosen arbitarily. Now that ALL exists, it makes more sense to use it for this test. Change-Id: Ia3593f7dd506205da13802a69094f4dd7019ab90 Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/63371 Reviewed-by: Jason Lowe-Power <power.jg@gmail.com> Maintainer: Jason Lowe-Power <power.jg@gmail.com> Tested-by: kokoro <noreply+kokoro@google.com> --- M tests/jenkins/presubmit.sh 1 file changed, 20 insertions(+), 5 deletions(-) Approvals: Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved kokoro: Regressions pass diff --git a/tests/jenkins/presubmit.sh b/tests/jenkins/presubmit.sh index 98c3d17..44ec8cf 100755 --- a/tests/jenkins/presubmit.sh +++ b/tests/jenkins/presubmit.sh @@ -61,12 +61,9 @@ # 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 -# 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. +# clang compiler. 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/ARM_MESI_Three_Level_HTM/gem5.fast -j4 --no-compress-debug \ + build/ALL/gem5.fast -j4 --no-compress-debug \ --ignore-style -- To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/63371?usp=email To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings Gerrit-Project: public/gem5 Gerrit-Branch: develop Gerrit-Change-Id: Ia3593f7dd506205da13802a69094f4dd7019ab90 Gerrit-Change-Number: 63371 Gerrit-PatchSet: 12 Gerrit-Owner: Bobby Bruce <bbruce@ucdavis.edu> Gerrit-Reviewer: Bobby Bruce <bbruce@ucdavis.edu> Gerrit-Reviewer: Jason Lowe-Power <power.jg@gmail.com> Gerrit-Reviewer: kokoro <noreply+kokoro@google.com> Gerrit-MessageType: merged