summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNathan Binkert <nate@binkert.org>2009-06-05 15:20:09 -0700
committerNathan Binkert <nate@binkert.org>2009-06-05 15:20:09 -0700
commitc76a8b1c15fb6e0a1f9da60cd7ee73efd14200e5 (patch)
tree537e02ec7de7d57182604bbc0ad52244742ea5b5
parenta01437ab03fedd65405f214aacb0416088704505 (diff)
downloadgem5-c76a8b1c15fb6e0a1f9da60cd7ee73efd14200e5.tar.xz
scons: Make it so that the processing of trace flags does not depend on order
-rw-r--r--src/SConscript47
1 files changed, 25 insertions, 22 deletions
diff --git a/src/SConscript b/src/SConscript
index d36f5f244..42e002310 100644
--- a/src/SConscript
+++ b/src/SConscript
@@ -209,13 +209,6 @@ def CompoundFlag(name, flags, desc=None):
raise AttributeError, "Flag %s already specified" % name
compound = tuple(flags)
- for flag in compound:
- if flag not in trace_flags:
- raise AttributeError, "Trace flag %s not found" % flag
- if trace_flags[flag][1]:
- raise AttributeError, \
- "Compound flag can't point to another compound flag"
-
trace_flags[name] = (name, compound, desc)
Export('TraceFlag')
@@ -630,18 +623,34 @@ env.Command('python/swig/init.cc',
makeSwigInit)
Source('python/swig/init.cc')
+def getFlags(source_flags):
+ flagsMap = {}
+ flagsList = []
+ for s in source_flags:
+ val = eval(s.get_contents())
+ name, compound, desc = val
+ flagsList.append(val)
+ flagsMap[name] = bool(compound)
+
+ for name, compound, desc in flagsList:
+ for flag in compound:
+ if flag not in flagsMap:
+ raise AttributeError, "Trace flag %s not found" % flag
+ if flagsMap[flag]:
+ raise AttributeError, \
+ "Compound flag can't point to another compound flag"
+
+ flagsList.sort()
+ return flagsList
+
+
# Generate traceflags.py
def traceFlagsPy(target, source, env):
assert(len(target) == 1)
f = file(str(target[0]), 'w')
-
- allFlags = []
- for s in source:
- val = eval(s.get_contents())
- allFlags.append(val)
-
- allFlags.sort()
+
+ allFlags = getFlags(source)
print >>f, 'basic = ['
for flag, compound, desc in allFlags:
@@ -683,10 +692,7 @@ def traceFlagsCC(target, source, env):
f = file(str(target[0]), 'w')
- allFlags = []
- for s in source:
- val = eval(s.get_contents())
- allFlags.append(val)
+ allFlags = getFlags(source)
# file header
print >>f, '''
@@ -759,10 +765,7 @@ def traceFlagsHH(target, source, env):
f = file(str(target[0]), 'w')
- allFlags = []
- for s in source:
- val = eval(s.get_contents())
- allFlags.append(val)
+ allFlags = getFlags(source)
# file header boilerplate
print >>f, '''