From 710ed8f492aa783933df6551ff98c9a6750fd9f7 Mon Sep 17 00:00:00 2001 From: Nathan Binkert Date: Thu, 9 Sep 2010 14:15:41 -0700 Subject: [PATCH] scons: use code_formatter wherever we can in the build system --- src/SConscript | 406 ++++++++++++++++++++----------------- src/python/m5/SimObject.py | 191 +++++++++-------- src/python/m5/params.py | 200 +++++++++++------- src/sim/System.py | 5 +- 4 files changed, 459 insertions(+), 343 deletions(-) diff --git a/src/SConscript b/src/SConscript index 5879fbc93..67d285016 100644 --- a/src/SConscript +++ b/src/SConscript @@ -259,10 +259,8 @@ for opt in export_vars: env.ConfigFile(opt) def makeTheISA(source, target, env): - f = file(str(target[0]), 'w') - isas = [ src.get_contents() for src in source ] - target = env['TARGET_ISA'] + target_isa = env['TARGET_ISA'] def define(isa): return isa.upper() + '_ISA' @@ -270,16 +268,24 @@ def makeTheISA(source, target, env): return isa[0].upper() + isa[1:].lower() + 'ISA' - print >>f, '#ifndef __CONFIG_THE_ISA_HH__' - print >>f, '#define __CONFIG_THE_ISA_HH__' - print >>f + code = code_formatter() + code('''\ +#ifndef __CONFIG_THE_ISA_HH__ +#define __CONFIG_THE_ISA_HH__ + +''') + for i,isa in enumerate(isas): - print >>f, '#define %s %d' % (define(isa), i + 1) - print >>f - print >>f, '#define THE_ISA %s' % (define(target)) - print >>f, '#define TheISA %s' % (namespace(target)) - print >>f - print >>f, '#endif // __CONFIG_THE_ISA_HH__' + code('#define $0 $1', define(isa), i + 1) + + code(''' + +#define THE_ISA ${{define(target_isa)}} +#define TheISA ${{namespace(target_isa)}} + +#endif // __CONFIG_THE_ISA_HH__''') + + code.write(str(target[0])) env.Command('config/the_isa.hh', map(Value, all_isa_list), makeTheISA) @@ -347,6 +353,7 @@ class DictImporter(object): import m5.SimObject import m5.params +from m5.util import code_formatter m5.SimObject.clear() m5.params.clear() @@ -402,7 +409,7 @@ depends = [ PySource.modules[dep].tnode for dep in module_depends ] def makeDefinesPyFile(target, source, env): build_env, hg_info = [ x.get_contents() for x in source ] - code = m5.util.code_formatter() + code = code_formatter() code(""" import m5.internal import m5.util @@ -418,7 +425,7 @@ for key,val in m5.internal.core.__dict__.iteritems(): _globals[flag] = val del _globals """) - code.write(str(target[0])) + code.write(target[0].abspath) defines_info = [ Value(build_env), Value(env['HG_INFO']) ] # Generate a file with all of the compile options in it @@ -427,11 +434,11 @@ PySource('m5', 'python/m5/defines.py') # Generate python file containing info about the M5 source code def makeInfoPyFile(target, source, env): - f = file(str(target[0]), 'w') + code = code_formatter() for src in source: data = ''.join(file(src.srcnode().abspath, 'r').xreadlines()) - print >>f, "%s = %s" % (src, repr(data)) - f.close() + code('$src = ${{repr(data)}}') + code.write(str(target[0])) # Generate a file that wraps the basic top level files env.Command('python/m5/info.py', @@ -441,12 +448,15 @@ PySource('m5', 'python/m5/info.py') # Generate the __init__.py file for m5.objects def makeObjectsInitFile(target, source, env): - f = file(str(target[0]), 'w') - print >>f, 'from params import *' - print >>f, 'from m5.SimObject import *' + code = code_formatter() + code('''\ +from params import * +from m5.SimObject import * +''') + for module in source: - print >>f, 'from %s import *' % module.get_contents() - f.close() + code('from $0 import *', module.get_contents()) + code.write(str(target[0])) # Generate an __init__.py file for the objects package env.Command('python/m5/objects/__init__.py', @@ -462,43 +472,42 @@ PySource('m5.objects', 'python/m5/objects/__init__.py') def createSimObjectParam(target, source, env): assert len(target) == 1 and len(source) == 1 - hh_file = file(target[0].abspath, 'w') name = str(source[0].get_contents()) obj = sim_objects[name] - print >>hh_file, obj.cxx_decl() - hh_file.close() + code = code_formatter() + obj.cxx_decl(code) + code.write(target[0].abspath) def createSwigParam(target, source, env): assert len(target) == 1 and len(source) == 1 - i_file = file(target[0].abspath, 'w') name = str(source[0].get_contents()) param = all_params[name] - for line in param.swig_decl(): - print >>i_file, line - i_file.close() + code = code_formatter() + param.swig_decl(code) + code.write(target[0].abspath) def createEnumStrings(target, source, env): assert len(target) == 1 and len(source) == 1 - cc_file = file(target[0].abspath, 'w') name = str(source[0].get_contents()) obj = all_enums[name] - print >>cc_file, obj.cxx_def() - cc_file.close() + code = code_formatter() + obj.cxx_def(code) + code.write(target[0].abspath) def createEnumParam(target, source, env): assert len(target) == 1 and len(source) == 1 - hh_file = file(target[0].abspath, 'w') name = str(source[0].get_contents()) obj = all_enums[name] - print >>hh_file, obj.cxx_decl() - hh_file.close() + code = code_formatter() + obj.cxx_decl(code) + code.write(target[0].abspath) # Generate all of the SimObject param struct header files params_hh_files = [] @@ -538,7 +547,6 @@ for name,enum in sorted(all_enums.iteritems()): def buildParams(target, source, env): names = [ s.get_contents() for s in source ] objs = [ sim_objects[name] for name in names ] - out = file(target[0].abspath, 'w') ordered_objs = [] obj_seen = set() @@ -556,82 +564,67 @@ def buildParams(target, source, env): for obj in objs: order_obj(obj) - enums = set() - predecls = [] - pd_seen = set() + code = code_formatter() + code('%module params') - def add_pds(*pds): - for pd in pds: - if pd not in pd_seen: - predecls.append(pd) - pd_seen.add(pd) + code('%{') + for obj in ordered_objs: + code('#include "params/$obj.hh"') + code('%}') for obj in ordered_objs: params = obj._params.local.values() for param in params: - ptype = param.ptype - if issubclass(ptype, m5.params.Enum): - if ptype not in enums: - enums.add(ptype) - pds = param.swig_predecls() - if isinstance(pds, (list, tuple)): - add_pds(*pds) - else: - add_pds(pds) - - print >>out, '%module params' + param.swig_predecls(code) - print >>out, '%{' + enums = set() for obj in ordered_objs: - print >>out, '#include "params/%s.hh"' % obj - print >>out, '%}' - - for pd in predecls: - print >>out, pd - - enums = list(enums) - enums.sort() - for enum in enums: - print >>out, '%%include "enums/%s.hh"' % enum.__name__ - print >>out + params = obj._params.local.values() + for param in params: + ptype = param.ptype + if issubclass(ptype, m5.params.Enum) and ptype not in enums: + enums.add(ptype) + code('%include "enums/$0.hh"', ptype.__name__) + + for obj in ordered_objs: + obj.swig_objdecls(code) + code() for obj in ordered_objs: + continue if obj.swig_objdecls: - for decl in obj.swig_objdecls: - print >>out, decl + obj.swig_objdecls(code) continue class_path = obj.cxx_class.split('::') classname = class_path[-1] namespaces = class_path[:-1] - namespaces.reverse() - code = '' + for ns in namespaces: + code('namespace $ns {') if namespaces: - code += '// avoid name conflicts\n' + code('// avoid name conflicts') sep_string = '_COLONS_' flat_name = sep_string.join(class_path) - code += '%%rename(%s) %s;\n' % (flat_name, classname) + code('%rename($flat_name) $classname;') - code += '// stop swig from creating/wrapping default ctor/dtor\n' - code += '%%nodefault %s;\n' % classname - code += 'class %s ' % classname + code('// stop swig from creating/wrapping default ctor/dtor') + code('%nodefault $classname;') if obj._base: - code += ': public %s' % obj._base.cxx_class - code += ' {};\n' - - for ns in namespaces: - new_code = 'namespace %s {\n' % ns - new_code += code - new_code += '}\n' - code = new_code + code('class $classname : public ${{obj._base.cxx_class}} {};') + else: + code('class $classname {};') - print >>out, code + for ns in reversed(namespaces): + code('/* namespace $ns */ }') + code() - print >>out, '%%include "src/sim/sim_object_params.hh"' % obj + code('%include "src/sim/sim_object_params.hh"') for obj in ordered_objs: - print >>out, '%%include "params/%s.hh"' % obj + code('%include "params/$obj.hh"') + + code.write(target[0].abspath) params_file = File('params/params.i') names = sorted(sim_objects.keys()) @@ -649,16 +642,23 @@ for swig in SwigSource.all: # Generate the main swig init file def makeSwigInit(target, source, env): - f = file(str(target[0]), 'w') - print >>f, 'extern "C" {' + code = code_formatter() + + code('extern "C" {') + code.indent() for module in source: - print >>f, ' void init_%s();' % module.get_contents() - print >>f, '}' - print >>f, 'void initSwig() {' + code('void init_$0();', module.get_contents()) + code.dedent() + code('}') + + code('void initSwig() {') + code.indent() for module in source: - print >>f, ' init_%s();' % module.get_contents() - print >>f, '}' - f.close() + code('init_$0();', module.get_contents()) + code.dedent() + code('}') + + code.write(str(target[0])) env.Command('python/swig/init.cc', map(Value, sorted(s.module for s in SwigSource.all)), @@ -689,55 +689,61 @@ def getFlags(source_flags): # Generate traceflags.py def traceFlagsPy(target, source, env): assert(len(target) == 1) + code = code_formatter() - f = file(str(target[0]), 'w') - allFlags = getFlags(source) - print >>f, 'basic = [' + code('basic = [') + code.indent() for flag, compound, desc in allFlags: if not compound: - print >>f, " '%s'," % flag - print >>f, " ]" - print >>f - - print >>f, 'compound = [' - print >>f, " 'All'," + code("'$flag',") + code(']') + code.dedent() + code() + + code('compound = [') + code.indent() + code("'All',") for flag, compound, desc in allFlags: if compound: - print >>f, " '%s'," % flag - print >>f, " ]" - print >>f + code("'$flag',") + code("]") + code.dedent() + code() - print >>f, "all = frozenset(basic + compound)" - print >>f + code("all = frozenset(basic + compound)") + code() - print >>f, 'compoundMap = {' + code('compoundMap = {') + code.indent() all = tuple([flag for flag,compound,desc in allFlags if not compound]) - print >>f, " 'All' : %s," % (all, ) + code("'All' : $all,") for flag, compound, desc in allFlags: if compound: - print >>f, " '%s' : %s," % (flag, compound) - print >>f, " }" - print >>f - - print >>f, 'descriptions = {' - print >>f, " 'All' : 'All flags'," + code("'$flag' : $compound,") + code('}') + code.dedent() + code() + + code('descriptions = {') + code.indent() + code("'All' : 'All flags',") for flag, compound, desc in allFlags: - print >>f, " '%s' : '%s'," % (flag, desc) - print >>f, " }" + code("'$flag' : '$desc',") + code("}") + code.dedent() - f.close() + code.write(str(target[0])) def traceFlagsCC(target, source, env): assert(len(target) == 1) - f = file(str(target[0]), 'w') - allFlags = getFlags(source) + code = code_formatter() # file header - print >>f, ''' + code(''' /* * DO NOT EDIT THIS FILE! Automatically generated */ @@ -747,70 +753,74 @@ def traceFlagsCC(target, source, env): using namespace Trace; const char *Trace::flagStrings[] = -{''' +{''') + code.indent() # The string array is used by SimpleEnumParam to map the strings # provided by the user to enum values. for flag, compound, desc in allFlags: if not compound: - print >>f, ' "%s",' % flag + code('"$flag",') - print >>f, ' "All",' + code('"All",') for flag, compound, desc in allFlags: if compound: - print >>f, ' "%s",' % flag + code('"$flag",') + code.dedent() - print >>f, '};' - print >>f - print >>f, 'const int Trace::numFlagStrings = %d;' % (len(allFlags) + 1) - print >>f + code('''\ +}; + +const int Trace::numFlagStrings = ${{len(allFlags) + 1}}; + +''') - # # Now define the individual compound flag arrays. There is an array # for each compound flag listing the component base flags. - # all = tuple([flag for flag,compound,desc in allFlags if not compound]) - print >>f, 'static const Flags AllMap[] = {' + code('static const Flags AllMap[] = {') + code.indent() for flag, compound, desc in allFlags: if not compound: - print >>f, " %s," % flag - print >>f, '};' - print >>f + code('$flag,') + code.dedent() + code('};') + code() for flag, compound, desc in allFlags: if not compound: continue - print >>f, 'static const Flags %sMap[] = {' % flag + code('static const Flags ${flag}Map[] = {') + code.indent() for flag in compound: - print >>f, " %s," % flag - print >>f, " (Flags)-1" - print >>f, '};' - print >>f + code('$flag,') + code('(Flags)-1') + code.dedent() + code('};') + code() - # # Finally the compoundFlags[] array maps the compound flags # to their individual arrays/ - # - print >>f, 'const Flags *Trace::compoundFlags[] =' - print >>f, '{' - print >>f, ' AllMap,' + code('const Flags *Trace::compoundFlags[] = {') + code.indent() + code('AllMap,') for flag, compound, desc in allFlags: if compound: - print >>f, ' %sMap,' % flag + code('${flag}Map,') # file trailer - print >>f, '};' + code.dedent() + code('};') - f.close() + code.write(str(target[0])) def traceFlagsHH(target, source, env): assert(len(target) == 1) - f = file(str(target[0]), 'w') - allFlags = getFlags(source) + code = code_formatter() # file header boilerplate - print >>f, ''' + code('''\ /* * DO NOT EDIT THIS FILE! * @@ -822,36 +832,41 @@ def traceFlagsHH(target, source, env): namespace Trace { -enum Flags {''' +enum Flags {''') # Generate the enum. Base flags come first, then compound flags. idx = 0 + code.indent() for flag, compound, desc in allFlags: if not compound: - print >>f, ' %s = %d,' % (flag, idx) + code('$flag = $idx,') idx += 1 numBaseFlags = idx - print >>f, ' NumFlags = %d,' % idx + code('NumFlags = $idx,') + code.dedent() + code() # put a comment in here to separate base from compound flags - print >>f, ''' + code(''' // The remaining enum values are *not* valid indices for Trace::flags. // They are "compound" flags, which correspond to sets of base -// flags, and are used by changeFlag.''' +// flags, and are used by changeFlag.''') - print >>f, ' All = %d,' % idx + code.indent() + code('All = $idx,') idx += 1 for flag, compound, desc in allFlags: if compound: - print >>f, ' %s = %d,' % (flag, idx) + code('$flag = $idx,') idx += 1 numCompoundFlags = idx - numBaseFlags - print >>f, ' NumCompoundFlags = %d' % numCompoundFlags + code('NumCompoundFlags = $numCompoundFlags') + code.dedent() # trailer boilerplate - print >>f, '''\ + code('''\ }; // enum Flags // Array of strings for SimpleEnumParam @@ -865,9 +880,9 @@ extern const Flags *compoundFlags[]; /* namespace Trace */ } #endif // __BASE_TRACE_FLAGS_HH__ -''' +''') - f.close() + code.write(str(target[0])) flags = map(Value, trace_flags.values()) env.Command('base/traceflags.py', flags, traceFlagsPy) @@ -889,7 +904,6 @@ def objectifyPyFile(target, source, env): as just bytes with a label in the data section''' src = file(str(source[0]), 'r').read() - dst = file(str(target[0]), 'w') pysource = PySource.tnodes[source[0]] compiled = compile(src, pysource.abspath, 'exec') @@ -906,15 +920,21 @@ def objectifyPyFile(target, source, env): sym = pysource.symname step = 16 - print >>dst, ".data" - print >>dst, ".globl %s_beg" % sym - print >>dst, ".globl %s_end" % sym - print >>dst, "%s_beg:" % sym + code = code_formatter() + code('''\ +.data +.globl ${sym}_beg +.globl ${sym}_end +${sym}_beg:''') + for i in xrange(0, len(data), step): x = array.array('B', data[i:i+step]) - print >>dst, ".byte", ','.join([str(d) for d in x]) - print >>dst, "%s_end:" % sym - print >>dst, ".long %d" % len(marshalled) + bytes = ','.join([str(d) for d in x]) + code('.byte $bytes') + code('${sym}_end:') + code('.long $0', len(marshalled)) + + code.write(str(target[0])) for source in PySource.all: env.Command(source.assembly, source.tnode, objectifyPyFile) @@ -926,41 +946,49 @@ for source in PySource.all: # the embedded files, and then there's a list of all of the rest that # the importer uses to load the rest on demand. def pythonInit(target, source, env): - dst = file(str(target[0]), 'w') + code = code_formatter() def dump_mod(sym, endchar=','): def c_str(string): if string is None: return "0" return '"%s"' % string - pysource = PySource.symnames[sym] - print >>dst, ' { %s,' % c_str(pysource.arcname) - print >>dst, ' %s,' % c_str(pysource.abspath) - print >>dst, ' %s,' % c_str(pysource.modpath) - print >>dst, ' %s_beg, %s_end,' % (sym, sym) - print >>dst, ' %s_end - %s_beg,' % (sym, sym) - print >>dst, ' *(int *)%s_end }%s' % (sym, endchar) - - print >>dst, '#include "sim/init.hh"' + pysource = PySource.symnames[sym] + arcname = c_str(pysource.arcname) + abspath = c_str(pysource.abspath) + modpath = c_str(pysource.modpath) + code.indent() + code('''\ +{ $arcname, + $abspath, + $modpath, + ${sym}_beg, ${sym}_end, + ${sym}_end - ${sym}_beg, + *(int *)${sym}_end }$endchar +''') + code.dedent() + + code('#include "sim/init.hh"') for sym in source: sym = sym.get_contents() - print >>dst, "extern const char %s_beg[], %s_end[];" % (sym, sym) + code('extern const char ${sym}_beg[], ${sym}_end[];') - print >>dst, "const EmbeddedPyModule embeddedPyImporter = " - dump_mod("PyEMB_importer", endchar=';'); - print >>dst + code('const EmbeddedPyModule embeddedPyImporter = ') + dump_mod("PyEMB_importer", endchar=';') + code() - print >>dst, "const EmbeddedPyModule embeddedPyModules[] = {" + code('const EmbeddedPyModule embeddedPyModules[] = {') for i,sym in enumerate(source): sym = sym.get_contents() if sym == "PyEMB_importer": # Skip the importer since we've already exported it continue dump_mod(sym) - print >>dst, " { 0, 0, 0, 0, 0, 0, 0 }" - print >>dst, "};" + code(' { 0, 0, 0, 0, 0, 0, 0 }') + code('};') + code.write(str(target[0])) env.Command('sim/init_python.cc', map(Value, (s.symname for s in PySource.all)), diff --git a/src/python/m5/SimObject.py b/src/python/m5/SimObject.py index ab0c39543..d26a99051 100644 --- a/src/python/m5/SimObject.py +++ b/src/python/m5/SimObject.py @@ -29,7 +29,7 @@ # Nathan Binkert import sys -from types import FunctionType +from types import FunctionType, MethodType try: import pydot @@ -97,6 +97,46 @@ allClasses = {} # dict to look up SimObjects based on path instanceDict = {} +def default_cxx_predecls(cls, code): + '''A forward class declaration is sufficient since we are + just declaring a pointer.''' + + class_path = cls._value_dict['cxx_class'].split('::') + for ns in class_path[:-1]: + code('namespace $ns {') + code('class $0;', class_path[-1]) + for ns in reversed(class_path[:-1]): + code('/* namespace $ns */ }') + +def default_swig_objdecls(cls, code): + class_path = cls.cxx_class.split('::') + classname = class_path[-1] + namespaces = class_path[:-1] + + for ns in namespaces: + code('namespace $ns {') + + if namespaces: + code('// avoid name conflicts') + sep_string = '_COLONS_' + flat_name = sep_string.join(class_path) + code('%rename($flat_name) $classname;') + + code() + code('// stop swig from creating/wrapping default ctor/dtor') + code('%nodefault $classname;') + code('class $classname') + if cls._base: + code(' : public ${{cls._base.cxx_class}}') + code('{};') + + for ns in reversed(namespaces): + code('/* namespace $ns */ }') + +def public_value(key, value): + return key.startswith('_') or \ + isinstance(value, (FunctionType, MethodType, classmethod, type)) + # The metaclass for SimObject. This class controls how new classes # that derive from SimObject are instantiated, and provides inherited # class behavior (just like a class controls how instances of that @@ -106,9 +146,9 @@ class MetaSimObject(type): init_keywords = { 'abstract' : bool, 'cxx_class' : str, 'cxx_type' : str, - 'cxx_predecls' : list, - 'swig_objdecls' : list, - 'swig_predecls' : list, + 'cxx_predecls' : MethodType, + 'swig_objdecls' : MethodType, + 'swig_predecls' : MethodType, 'type' : str } # Attributes that can be set any time keywords = { 'check' : FunctionType } @@ -127,9 +167,7 @@ class MetaSimObject(type): cls_dict = {} value_dict = {} for key,val in dict.items(): - if key.startswith('_') or isinstance(val, (FunctionType, - classmethod, - type)): + if public_value(key, val): cls_dict[key] = val else: # must be a param/port setting @@ -190,24 +228,16 @@ class MetaSimObject(type): cls._value_dict['cxx_type'] = '%s *' % cls._value_dict['cxx_class'] - if 'cxx_predecls' not in cls._value_dict: - # A forward class declaration is sufficient since we are - # just declaring a pointer. - class_path = cls._value_dict['cxx_class'].split('::') - class_path.reverse() - decl = 'class %s;' % class_path[0] - for ns in class_path[1:]: - decl = 'namespace %s { %s }' % (ns, decl) - cls._value_dict['cxx_predecls'] = [decl] - - if 'swig_predecls' not in cls._value_dict: - # A forward class declaration is sufficient since we are - # just declaring a pointer. - cls._value_dict['swig_predecls'] = \ - cls._value_dict['cxx_predecls'] - - if 'swig_objdecls' not in cls._value_dict: - cls._value_dict['swig_objdecls'] = [] + if 'cxx_predecls' not in cls.__dict__: + m = MethodType(default_cxx_predecls, cls, MetaSimObject) + setattr(cls, 'cxx_predecls', m) + + if 'swig_predecls' not in cls.__dict__: + setattr(cls, 'swig_predecls', getattr(cls, 'cxx_predecls')) + + if 'swig_objdecls' not in cls.__dict__: + m = MethodType(default_swig_objdecls, cls, MetaSimObject) + setattr(cls, 'swig_objdecls', m) # Now process the _value_dict items. They could be defining # new (or overriding existing) parameters or ports, setting @@ -282,7 +312,7 @@ class MetaSimObject(type): # instance of class cls). def __setattr__(cls, attr, value): # normal processing for private attributes - if attr.startswith('_'): + if public_value(attr, value): type.__setattr__(cls, attr, value) return @@ -328,9 +358,12 @@ class MetaSimObject(type): def __str__(cls): return cls.__name__ - def cxx_decl(cls): - code = "#ifndef __PARAMS__%s\n" % cls - code += "#define __PARAMS__%s\n\n" % cls + def cxx_decl(cls, code): + code('''\ +#ifndef __PARAMS__${cls}__ +#define __PARAMS__${cls}__ + +''') # The 'dict' attribute restricts us to the params declared in # the object itself, not including inherited params (which @@ -344,59 +377,57 @@ class MetaSimObject(type): print params raise - # get a list of lists of predeclaration lines - predecls = [] - predecls.extend(cls.cxx_predecls) - for p in params: - predecls.extend(p.cxx_predecls()) - # remove redundant lines - predecls2 = [] - for pd in predecls: - if pd not in predecls2: - predecls2.append(pd) - predecls2.sort() - code += "\n".join(predecls2) - code += "\n\n"; + # get all predeclarations + cls.cxx_predecls(code) + for param in params: + param.cxx_predecls(code) + code() if cls._base: - code += '#include "params/%s.hh"\n\n' % cls._base.type + code('#include "params/${{cls._base.type}}.hh"') + code() for ptype in ptypes: if issubclass(ptype, Enum): - code += '#include "enums/%s.hh"\n' % ptype.__name__ - code += "\n\n" + code('#include "enums/${{ptype.__name__}}.hh"') + code() - code += cls.cxx_struct(cls._base, params) + cls.cxx_struct(code, cls._base, params) # close #ifndef __PARAMS__* guard - code += "\n#endif\n" + code() + code('#endif // __PARAMS__${cls}__') return code - def cxx_struct(cls, base, params): + def cxx_struct(cls, code, base, params): if cls == SimObject: - return '#include "sim/sim_object_params.hh"\n' + code('#include "sim/sim_object_params.hh"') + return # now generate the actual param struct - code = "struct %sParams" % cls + code("struct ${cls}Params") if base: - code += " : public %sParams" % base.type - code += "\n{\n" + code(" : public ${{base.type}}Params") + code("{") if not hasattr(cls, 'abstract') or not cls.abstract: if 'type' in cls.__dict__: - code += " %s create();\n" % cls.cxx_type - decls = [p.cxx_decl() for p in params] - decls.sort() - code += "".join([" %s\n" % d for d in decls]) - code += "};\n" + code(" ${{cls.cxx_type}} create();") - return code + code.indent() + for param in params: + param.cxx_decl(code) + code.dedent() + code('};') + + def swig_decl(cls, code): + code('''\ +%module $cls - def swig_decl(cls): - code = '%%module %s\n' % cls +%{ +#include "params/$cls.hh" +%} - code += '%{\n' - code += '#include "params/%s.hh"\n' % cls - code += '%}\n\n' +''') # The 'dict' attribute restricts us to the params declared in # the object itself, not including inherited params (which @@ -405,32 +436,22 @@ class MetaSimObject(type): params = cls._params.local.values() ptypes = [p.ptype for p in params] - # get a list of lists of predeclaration lines - predecls = [] - predecls.extend([ p.swig_predecls() for p in params ]) - # flatten - predecls = reduce(lambda x,y:x+y, predecls, []) - # remove redundant lines - predecls2 = [] - for pd in predecls: - if pd not in predecls2: - predecls2.append(pd) - predecls2.sort() - code += "\n".join(predecls2) - code += "\n\n"; + # get all predeclarations + for param in params: + param.swig_predecls(code) + code() if cls._base: - code += '%%import "params/%s.i"\n\n' % cls._base.type + code('%import "params/${{cls._base.type}}.i"') + code() for ptype in ptypes: if issubclass(ptype, Enum): - code += '%%import "enums/%s.hh"\n' % ptype.__name__ - code += "\n\n" + code('%import "enums/${{ptype.__name__}}.hh"') + code() - code += '%%import "params/%s_type.hh"\n\n' % cls - code += '%%include "params/%s.hh"\n\n' % cls - - return code + code('%import "params/${cls}_type.hh"') + code('%include "params/${cls}.hh"') # The SimObject class is the root of the special hierarchy. Most of # the code in this class deals with the configuration hierarchy itself @@ -442,7 +463,9 @@ class SimObject(object): type = 'SimObject' abstract = True - swig_objdecls = [ '%include "python/swig/sim_object.i"' ] + @classmethod + def swig_objdecls(cls, code): + code('%include "python/swig/sim_object.i"') # Initialize new instance. For objects with SimObject-valued # children, we need to recursively clone the classes represented diff --git a/src/python/m5/params.py b/src/python/m5/params.py index 38e320b53..00bba636b 100644 --- a/src/python/m5/params.py +++ b/src/python/m5/params.py @@ -80,8 +80,13 @@ class MetaParamValue(type): class ParamValue(object): __metaclass__ = MetaParamValue - cxx_predecls = [] - swig_predecls = [] + @classmethod + def cxx_predecls(cls, code): + pass + + @classmethod + def swig_predecls(cls, code): + pass # default for printing to .ini file is regular string conversion. # will be overridden in some cases @@ -152,14 +157,14 @@ class ParamDesc(object): return value return self.ptype(value) - def cxx_predecls(self): - return self.ptype.cxx_predecls + def cxx_predecls(self, code): + self.ptype.cxx_predecls(code) - def swig_predecls(self): - return self.ptype.swig_predecls + def swig_predecls(self, code): + self.ptype.swig_predecls(code) - def cxx_decl(self): - return '%s %s;' % (self.ptype.cxx_type, self.name) + def cxx_decl(self, code): + code('${{self.ptype.cxx_type}} ${{self.name}};') # Vector-valued parameter description. Just like ParamDesc, except # that the value is a vector (list) of the specified type instead of a @@ -235,20 +240,25 @@ class VectorParamDesc(ParamDesc): else: return VectorParamValue(tmp_list) - def swig_predecls(self): - return ['%%include "%s_vptype.i"' % self.ptype_str] + def swig_predecls(self, code): + code('%include "${{self.ptype_str}}_vptype.i"') - def swig_decl(self): + def swig_decl(self, code): cxx_type = re.sub('std::', '', self.ptype.cxx_type) - vdecl = 'namespace std { %%template(vector_%s) vector< %s >; }' % \ - (self.ptype_str, cxx_type) - return ['%include "std_vector.i"'] + self.ptype.swig_predecls + [vdecl] + code('%include "std_vector.i"') + self.ptype.swig_predecls(code) + code('''\ +namespace std { +%template(vector_${{self.ptype_str}}) vector< $cxx_type >; +} +''') - def cxx_predecls(self): - return ['#include '] + self.ptype.cxx_predecls + def cxx_predecls(self, code): + code('#include ') + self.ptype.cxx_predecls(code) - def cxx_decl(self): - return 'std::vector< %s > %s;' % (self.ptype.cxx_type, self.name) + def cxx_decl(self, code): + code('std::vector< ${{self.ptype.cxx_type}} > ${{self.name}};') class ParamFactory(object): def __init__(self, param_desc_class, ptype_str = None): @@ -291,10 +301,14 @@ VectorParam = ParamFactory(VectorParamDesc) # built-in str class. class String(ParamValue,str): cxx_type = 'std::string' - cxx_predecls = ['#include '] - swig_predecls = ['%include "std_string.i"\n' + - '%apply const std::string& {std::string *};'] - swig_predecls = ['%include "std_string.i"' ] + + @classmethod + def cxx_predecls(self, code): + code('#include ') + + @classmethod + def swig_predecls(cls, code): + code('%include "std_string.i"') def getValue(self): return self @@ -350,15 +364,6 @@ class CheckedIntType(MetaParamValue): if name == 'CheckedInt': return - if not cls.cxx_predecls: - # most derived types require this, so we just do it here once - cls.cxx_predecls = ['#include "base/types.hh"'] - - if not cls.swig_predecls: - # most derived types require this, so we just do it here once - cls.swig_predecls = ['%import "stdint.i"\n' + - '%import "base/types.hh"'] - if not (hasattr(cls, 'min') and hasattr(cls, 'max')): if not (hasattr(cls, 'size') and hasattr(cls, 'unsigned')): panic("CheckedInt subclass %s must define either\n" \ @@ -393,6 +398,17 @@ class CheckedInt(NumericParamValue): % type(value).__name__ self._check() + @classmethod + def cxx_predecls(cls, code): + # most derived types require this, so we just do it here once + code('#include "base/types.hh"') + + @classmethod + def swig_predecls(cls, code): + # most derived types require this, so we just do it here once + code('%import "stdint.i"') + code('%import "base/types.hh"') + def getValue(self): return long(self.value) @@ -476,8 +492,6 @@ class MetaRange(MetaParamValue): if name == 'Range': return cls.cxx_type = 'Range< %s >' % cls.type.cxx_type - cls.cxx_predecls = \ - ['#include "base/range.hh"'] + cls.type.cxx_predecls class Range(ParamValue): __metaclass__ = MetaRange @@ -521,9 +535,17 @@ class Range(ParamValue): def __str__(self): return '%s:%s' % (self.first, self.second) + @classmethod + def cxx_predecls(cls, code): + code('#include "base/range.hh"') + cls.type.cxx_predecls(code) + class AddrRange(Range): type = Addr - swig_predecls = ['%include "python/swig/range.i"'] + + @classmethod + def swig_predecls(cls, code): + code('%include "python/swig/range.i"') def getValue(self): from m5.objects.params import AddrRange @@ -535,7 +557,10 @@ class AddrRange(Range): class TickRange(Range): type = Tick - swig_predecls = ['%include "python/swig/range.i"'] + + @classmethod + def swig_predecls(cls, code): + code('%include "python/swig/range.i"') def getValue(self): from m5.objects.params import TickRange @@ -589,8 +614,15 @@ def NextEthernetAddr(): class EthernetAddr(ParamValue): cxx_type = 'Net::EthAddr' - cxx_predecls = ['#include "base/inet.hh"'] - swig_predecls = ['%include "python/swig/inet.i"'] + + @classmethod + def cxx_predecls(cls, code): + code('#include "base/inet.hh"') + + @classmethod + def swig_predecls(cls, code): + code('%include "python/swig/inet.i"') + def __init__(self, value): if value == NextEthernetAddr: self.value = value @@ -661,8 +693,15 @@ def parse_time(value): class Time(ParamValue): cxx_type = 'tm' - cxx_predecls = [ '#include ' ] - swig_predecls = [ '%include "python/swig/time.i"' ] + + @classmethod + def cxx_predecls(cls, code): + code('#include ') + + @classmethod + def swig_predecls(cls, code): + code('%include "python/swig/time.i"') + def __init__(self, value): self.value = parse_time(value) @@ -749,34 +788,44 @@ class MetaEnum(MetaParamValue): # Generate C++ class declaration for this enum type. # Note that we wrap the enum in a class/struct to act as a namespace, # so that the enum strings can be brief w/o worrying about collisions. - def cxx_decl(cls): + def cxx_decl(cls, code): name = cls.__name__ - code = "#ifndef __ENUM__%s\n" % name - code += '#define __ENUM__%s\n' % name - code += '\n' - code += 'namespace Enums {\n' - code += ' enum %s {\n' % name + code('''\ +#ifndef __ENUM__${name}__ +#define __ENUM__${name}__ + +namespace Enums { + enum $name { +''') + code.indent(2) for val in cls.vals: - code += ' %s = %d,\n' % (val, cls.map[val]) - code += ' Num_%s = %d,\n' % (name, len(cls.vals)) - code += ' };\n' - code += ' extern const char *%sStrings[Num_%s];\n' % (name, name) - code += '}\n' - code += '\n' - code += '#endif\n' - return code - - def cxx_def(cls): + code('$val = ${{cls.map[val]}},') + code('Num_$name = ${{len(cls.vals)}},') + code.dedent(2) + code('''\ + }; +extern const char *${name}Strings[Num_${name}]; +} + +#endif // __ENUM__${name}__ +''') + + def cxx_def(cls, code): name = cls.__name__ - code = '#include "enums/%s.hh"\n' % name - code += 'namespace Enums {\n' - code += ' const char *%sStrings[Num_%s] =\n' % (name, name) - code += ' {\n' + code('''\ +#include "enums/${name}.hh" +namespace Enums { + const char *${name}Strings[Num_${name}] = + { +''') + code.indent(2) for val in cls.vals: - code += ' "%s",\n' % val - code += ' };\n' - code += '}\n' - return code + code('"$val",') + code.dedent(2) + code(''' + }; +/* namespace Enums */ } +''') # Base class for enum types. class Enum(ParamValue): @@ -800,9 +849,15 @@ frequency_tolerance = 0.001 # 0.1% class TickParamValue(NumericParamValue): cxx_type = 'Tick' - cxx_predecls = ['#include "base/types.hh"'] - swig_predecls = ['%import "stdint.i"\n' + - '%import "base/types.hh"'] + + @classmethod + def cxx_predecls(cls, code): + code('#include "base/types.hh"') + + @classmethod + def swig_predecls(cls, code): + code('%import "stdint.i"') + code('%import "base/types.hh"') def getValue(self): return long(self.value) @@ -878,9 +933,16 @@ class Frequency(TickParamValue): # An explicit conversion to a Latency or Frequency must be made first. class Clock(ParamValue): cxx_type = 'Tick' - cxx_predecls = ['#include "base/types.hh"'] - swig_predecls = ['%import "stdint.i"\n' + - '%import "base/types.hh"'] + + @classmethod + def cxx_predecls(cls, code): + code('#include "base/types.hh"') + + @classmethod + def swig_predecls(cls, code): + code('%import "stdint.i"') + code('%import "base/types.hh"') + def __init__(self, value): if isinstance(value, (Latency, Clock)): self.ticks = value.ticks diff --git a/src/sim/System.py b/src/sim/System.py index 5cf46ad75..eec7b4ae5 100644 --- a/src/sim/System.py +++ b/src/sim/System.py @@ -37,7 +37,10 @@ class MemoryMode(Enum): vals = ['invalid', 'atomic', 'timing'] class System(SimObject): type = 'System' - swig_objdecls = [ '%include "python/swig/system.i"' ] + + @classmethod + def swig_objdecls(cls, code): + code('%include "python/swig/system.i"') physmem = Param.PhysicalMemory(Parent.any, "physical memory") mem_mode = Param.MemoryMode('atomic', "The mode the memory system is in") -- 2.30.2