diff --git a/src/systemc/core/module.cc b/src/systemc/core/module.cc index b13b50d287..0c118e0c5d 100644 --- a/src/systemc/core/module.cc +++ b/src/systemc/core/module.cc @@ -32,6 +32,7 @@ #include #include "base/logging.hh" +#include "systemc/ext/utils/sc_report_handler.hh" namespace sc_gem5 { @@ -52,7 +53,14 @@ Module::Module(const char *name) : _name(name), _sc_mod(nullptr), _obj(nullptr) _new_module = this; } -Module::~Module() { allModules.remove(this); } +Module::~Module() +{ + if (_new_module == this) { + // Aborted module construction? + _new_module = nullptr; + } + allModules.remove(this); +} void Module::finish(Object *this_obj) @@ -85,6 +93,17 @@ currentModule() return _modules.back(); } +Module * +newModuleChecked() +{ + if (!_new_module) { + SC_REPORT_ERROR("(E533) module name stack is empty: " + "did you forget to add a sc_module_name parameter to " + "your module constructor?", nullptr); + } + return _new_module; +} + Module * newModule() { diff --git a/src/systemc/core/module.hh b/src/systemc/core/module.hh index e988b9667e..e276dad199 100644 --- a/src/systemc/core/module.hh +++ b/src/systemc/core/module.hh @@ -116,6 +116,7 @@ class Module }; Module *currentModule(); +Module *newModuleChecked(); Module *newModule(); void callbackModule(Module *m); diff --git a/src/systemc/core/sc_module.cc b/src/systemc/core/sc_module.cc index 016b9edbe9..c515aaf098 100644 --- a/src/systemc/core/sc_module.cc +++ b/src/systemc/core/sc_module.cc @@ -168,7 +168,7 @@ sc_module::get_child_events() const } sc_module::sc_module() : - sc_object(sc_gem5::newModule()->name()), + sc_object(sc_gem5::newModuleChecked()->name()), _gem5_module(sc_gem5::currentModule()) {}