Alex Richardson has uploaded this change for review. ( https://gem5-review.googlesource.com/c/public/gem5/+/54983 )

Change subject: misc: Generate StateMachine debug includes in deterministic order
......................................................................

misc: Generate StateMachine debug includes in deterministic order

Since 3454a4a36e927f483b36fa66baabe2c85ecf3ddc the order of the debug/
includes is non-deterministic which can result in unnecessary rebuilds.

Change-Id: I583d2caf70632e08fa59ac85073786270991edbc
---
M src/mem/slicc/symbols/StateMachine.py
1 file changed, 17 insertions(+), 4 deletions(-)



diff --git a/src/mem/slicc/symbols/StateMachine.py b/src/mem/slicc/symbols/StateMachine.py
index 5d315e9..c2a68ac 100644
--- a/src/mem/slicc/symbols/StateMachine.py
+++ b/src/mem/slicc/symbols/StateMachine.py
@@ -105,9 +105,10 @@
         self.objects = []
         self.TBEType   = None
         self.EntryType = None
-        self.debug_flags = set()
-        self.debug_flags.add('RubyGenerated')
-        self.debug_flags.add('RubySlicc')
+ # Ideally we would use a OrderedSet, but python does not provide that.
+        self.debug_flags = OrderedDict()
+        self.debug_flags['RubyGenerated'] = True
+        self.debug_flags['RubySlicc'] = True

     def __repr__(self):
         return "[StateMachine: %s]" % self.ident
@@ -137,7 +138,7 @@
         self.actions[action.ident] = action

     def addDebugFlag(self, flag):
-        self.debug_flags.add(flag)
+        self.debug_flags[flag] = True

     def addRequestType(self, request_type):
         assert self.table is None

--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/54983
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: I583d2caf70632e08fa59ac85073786270991edbc
Gerrit-Change-Number: 54983
Gerrit-PatchSet: 1
Gerrit-Owner: Alex Richardson <alexrichard...@google.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to