Bobby R. Bruce has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/47342 )

Change subject: tests: Add missing build targets to compiler tests
......................................................................

tests: Add missing build targets to compiler tests

This commit ensures all build targets are tested during the compiler
tests. Previously some missing.

Change-Id: I48674ead876dd528574b518108725493d2b8abc3
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/47342
Tested-by: kokoro <[email protected]>
Reviewed-by: Daniel Carvalho <[email protected]>
Reviewed-by: Jason Lowe-Power <[email protected]>
Maintainer: Jason Lowe-Power <[email protected]>
---
M tests/compiler-tests.sh
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Jason Lowe-Power: Looks good to me, approved; Looks good to me, approved
  Daniel Carvalho: Looks good to me, but someone else must approve
  kokoro: Regressions pass



diff --git a/tests/compiler-tests.sh b/tests/compiler-tests.sh
index df51925..29a4268 100755
--- a/tests/compiler-tests.sh
+++ b/tests/compiler-tests.sh
@@ -37,9 +37,12 @@
 # All build targets in build_opt/ which we want to build using each image.
 builds=("ARM"
         "ARM_MESI_Three_Level"
+        "ARM_MESI_Three_Level_HTM"
+        "ARM_MOESI_hammer"
         "Garnet_standalone"
         "GCN3_X86"
         "MIPS"
+        "NULL"
         "NULL_MESI_Two_Level"
         "NULL_MOESI_CMP_directory"
         "NULL_MOESI_CMP_token"
@@ -48,6 +51,7 @@
         "RISCV"
         "SPARC"
         "X86"
+        "X86_MESI_Two_Level"
         "X86_MOESI_AMD_Base")

 # The optimizations to use for each build target.



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/47342
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: I48674ead876dd528574b518108725493d2b8abc3
Gerrit-Change-Number: 47342
Gerrit-PatchSet: 4
Gerrit-Owner: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to