Gabe Black has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/12612

Change subject: systemc: Track event sensitivities with a list instead of a set.
......................................................................

systemc: Track event sensitivities with a list instead of a set.

It's totally legal to signal that an event happened to waiting
processes in any order we choose, but to match the order of events
which appears in the Accellera test golden output, we need to do things
in the order they did. This is less efficient, but will reduce the
number of false positives.

Change-Id: Ie2882249ae846991d627f5f688a9e89e629bb300
---
M src/systemc/core/event.hh
1 file changed, 4 insertions(+), 4 deletions(-)



diff --git a/src/systemc/core/event.hh b/src/systemc/core/event.hh
index 57d3f3f..c39053b 100644
--- a/src/systemc/core/event.hh
+++ b/src/systemc/core/event.hh
@@ -30,7 +30,7 @@
 #ifndef __SYSTEMC_CORE_EVENT_HH__
 #define __SYSTEMC_CORE_EVENT_HH__

-#include <set>
+#include <list>
 #include <string>
 #include <vector>

@@ -93,8 +93,8 @@
         return e->_gem5_event;
     }

-    void addSensitivity(Sensitivity *s) const { sensitivities.insert(s); }
-    void delSensitivity(Sensitivity *s) const { sensitivities.erase(s); }
+ void addSensitivity(Sensitivity *s) const { sensitivities.push_back(s); }
+    void delSensitivity(Sensitivity *s) const { sensitivities.remove(s); }

   private:
     sc_core::sc_event *_sc_event;
@@ -107,7 +107,7 @@

     ScEvent delayedNotify;

-    mutable std::set<Sensitivity *> sensitivities;
+    mutable std::list<Sensitivity *> sensitivities;
 };

 extern Events topLevelEvents;

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/12612
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: Ie2882249ae846991d627f5f688a9e89e629bb300
Gerrit-Change-Number: 12612
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <gabebl...@google.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to