Gabe Black has submitted this change and it was merged. (
https://gem5-review.googlesource.com/c/public/gem5/+/13315 )
Change subject: systemc: Manually correct the golden output for a test.
......................................................................
systemc: Manually correct the golden output for a test.
The golden reference output for the test was to throw an error which
is clearly not at all related to the test (it was about immediate
self notifications while the test never calls any form of notify())
and which would happen significantly before the end of the test,
negating all the other behaviors and checks which would happen after
that point.
Since it strongly looks like the reference output was updated in error,
and because other very similarly structured tests are expected to run
silently except for printing "Success" at the end (which it does when
run under gem5), this change manually updates the golden reference
output to reflect what appears to be correct.
Change-Id: I9cde81c28774049653d60f1ffd37a2fae875b522
Reviewed-on: https://gem5-review.googlesource.com/c/13315
Reviewed-by: Gabe Black <[email protected]>
Maintainer: Gabe Black <[email protected]>
---
M
src/systemc/tests/systemc/1666-2011-compliance/method_with_reset/golden/method_with_reset.log
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
Gabe Black: Looks good to me, approved; Looks good to me, approved
diff --git
a/src/systemc/tests/systemc/1666-2011-compliance/method_with_reset/golden/method_with_reset.log
b/src/systemc/tests/systemc/1666-2011-compliance/method_with_reset/golden/method_with_reset.log
index 0d5dc4e..7ced375 100644
---
a/src/systemc/tests/systemc/1666-2011-compliance/method_with_reset/golden/method_with_reset.log
+++
b/src/systemc/tests/systemc/1666-2011-compliance/method_with_reset/golden/method_with_reset.log
@@ -1,5 +1,3 @@
SystemC Simulation
-Error: (E521) immediate notification is not allowed during update phase or
elaboration
-In file: <removed by verify.pl>
-In process: top.ctrl.m3 @ 520 ns
+Success
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/13315
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: I9cde81c28774049653d60f1ffd37a2fae875b522
Gerrit-Change-Number: 13315
Gerrit-PatchSet: 3
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: Giacomo Travaglini <[email protected]>
Gerrit-Reviewer: Jason Lowe-Power <[email protected]>
Gerrit-Reviewer: Matthias Jung <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
