systemc: Wrap some report maps in functions.
authorGabe Black <gabeblack@google.com>
Wed, 7 Nov 2018 01:48:58 +0000 (17:48 -0800)
committerGabe Black <gabeblack@google.com>
Fri, 9 Nov 2018 01:25:15 +0000 (01:25 +0000)
By declaring the map as a static variable in that function and then
returning it, we can guarantee that it's initialized relative to other
static initializers so that we don't try to use a data structure that
isn't constructed yet. This will let us get rid of the dependence on
python for setting up that mapping.

Change-Id: I031ce2039de8f5f79fbb9d76cf1363f15207b64b
Reviewed-on: https://gem5-review.googlesource.com/c/13975
Maintainer: Gabe Black <gabeblack@google.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
src/systemc/utils/report.cc
src/systemc/utils/report.hh
src/systemc/utils/sc_report.cc
src/systemc/utils/sc_report_handler.cc

index 87671f18146c325c96761110f6471117287b1e21..4a4521582f899d19ef445dc3bb854859e4d76a63 100644 (file)
@@ -49,8 +49,19 @@ ReportSevInfo reportSevInfos[sc_core::SC_MAX_SEVERITY] =
     [sc_core::SC_FATAL] = ReportSevInfo(sc_core::SC_DEFAULT_FATAL_ACTIONS)
 };
 
-std::map<std::string, ReportMsgInfo> reportMsgInfoMap;
-std::map<int, std::string> reportIdToMsgMap;
+std::map<std::string, ReportMsgInfo> &
+reportMsgInfoMap()
+{
+    static std::map<std::string, ReportMsgInfo> m;
+    return m;
+}
+
+std::map<int, std::string> &
+reportIdToMsgMap()
+{
+    static std::map<int, std::string> m;
+    return m;
+}
 
 int reportVerbosityLevel = sc_core::SC_MEDIUM;
 
index a0840c65b307fb3ad9bb83b5db49238b3503e6e9..70716cb91b377afafc3964aca07450359f03bbd4 100644 (file)
@@ -93,8 +93,9 @@ struct ReportSevInfo
 
 extern const char *reportSeverityNames[sc_core::SC_MAX_SEVERITY];
 extern ReportSevInfo reportSevInfos[sc_core::SC_MAX_SEVERITY];
-extern std::map<std::string, ReportMsgInfo> reportMsgInfoMap;
-extern std::map<int, std::string> reportIdToMsgMap;
+
+std::map<std::string, ReportMsgInfo> &reportMsgInfoMap();
+std::map<int, std::string> &reportIdToMsgMap();
 
 extern int reportVerbosityLevel;
 
index 122facc2484659a29f7eca2ba79cbb9b333f7f74..a0491d3d5f7625db633b14fe5beba2e79ef83876 100644 (file)
@@ -89,8 +89,8 @@ sc_report::what() const throw()
 const char *
 sc_report::get_message(int id)
 {
-    auto it = sc_gem5::reportIdToMsgMap.find(id);
-    if (it == sc_gem5::reportIdToMsgMap.end())
+    auto it = sc_gem5::reportIdToMsgMap().find(id);
+    if (it == sc_gem5::reportIdToMsgMap().end())
         return "unknown id";
     else
         return it->second.c_str();
@@ -99,11 +99,11 @@ sc_report::get_message(int id)
 bool
 sc_report::is_suppressed(int id)
 {
-    auto it = sc_gem5::reportIdToMsgMap.find(id);
-    if (it == sc_gem5::reportIdToMsgMap.end())
+    auto it = sc_gem5::reportIdToMsgMap().find(id);
+    if (it == sc_gem5::reportIdToMsgMap().end())
         return false;
 
-    auto &msgInfo = sc_gem5::reportMsgInfoMap[it->second];
+    auto &msgInfo = sc_gem5::reportMsgInfoMap()[it->second];
 
     return (msgInfo.actions == SC_DO_NOTHING ||
             (msgInfo.sevActions[SC_INFO] == SC_DO_NOTHING &&
@@ -127,31 +127,31 @@ sc_report::register_id(int id, const char *msg)
         SC_REPORT_ERROR(SC_ID_REGISTER_ID_FAILED_, "invalid report message");
         return;
     }
-    auto p = sc_gem5::reportIdToMsgMap.insert(
+    auto p = sc_gem5::reportIdToMsgMap().insert(
             std::pair<int, std::string>(id, msg));
     if (!p.second) {
         SC_REPORT_ERROR(SC_ID_REGISTER_ID_FAILED_, "report id already exists");
     } else {
-        sc_gem5::reportMsgInfoMap[msg].id = id;
+        sc_gem5::reportMsgInfoMap()[msg].id = id;
     }
 }
 
 void
 sc_report::suppress_id(int id, bool suppress)
 {
-    auto it = sc_gem5::reportIdToMsgMap.find(id);
-    if (it == sc_gem5::reportIdToMsgMap.end())
+    auto it = sc_gem5::reportIdToMsgMap().find(id);
+    if (it == sc_gem5::reportIdToMsgMap().end())
         return;
 
     if (suppress) {
-        sc_gem5::reportMsgInfoMap[it->second].
+        sc_gem5::reportMsgInfoMap()[it->second].
             sevActions[SC_INFO] = SC_DO_NOTHING;
-        sc_gem5::reportMsgInfoMap[it->second].
+        sc_gem5::reportMsgInfoMap()[it->second].
             sevActions[SC_WARNING] = SC_DO_NOTHING;
     } else {
-        sc_gem5::reportMsgInfoMap[it->second].
+        sc_gem5::reportMsgInfoMap()[it->second].
             sevActions[SC_INFO] = SC_UNSPECIFIED;
-        sc_gem5::reportMsgInfoMap[it->second].
+        sc_gem5::reportMsgInfoMap()[it->second].
             sevActions[SC_WARNING] = SC_UNSPECIFIED;
     }
 }
index ad0d3b84ed71603f9e14a20635c5ec820e54fc91..151c3c9c30348d2a140675886efdcfd0a183119c 100644 (file)
@@ -70,7 +70,7 @@ sc_report_handler::report(sc_severity severity, const char *msg_type,
         return;
 
     sc_gem5::ReportSevInfo &sevInfo = sc_gem5::reportSevInfos[severity];
-    sc_gem5::ReportMsgInfo &msgInfo = sc_gem5::reportMsgInfoMap[msg_type];
+    sc_gem5::ReportMsgInfo &msgInfo = sc_gem5::reportMsgInfoMap()[msg_type];
 
     sevInfo.count++;
     msgInfo.count++;
@@ -111,7 +111,7 @@ void
 sc_report_handler::report(sc_severity severity, int id, const char *msg,
                           const char *file, int line)
 {
-    std::string &msg_type = sc_gem5::reportIdToMsgMap[id];
+    std::string &msg_type = sc_gem5::reportIdToMsgMap()[id];
 
     if (sc_gem5::reportWarningsAsErrors && severity == SC_WARNING)
         severity = SC_ERROR;
@@ -134,7 +134,7 @@ sc_report_handler::set_actions(const char *msg_type, sc_actions actions)
     if (!msg_type)
         msg_type = SC_ID_UNKNOWN_ERROR_;
 
-    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap[msg_type];
+    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap()[msg_type];
     sc_actions previous = info.actions;
     info.actions = actions;
     return previous;
@@ -147,7 +147,7 @@ sc_report_handler::set_actions(
     if (!msg_type)
         msg_type = SC_ID_UNKNOWN_ERROR_;
 
-    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap[msg_type];
+    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap()[msg_type];
     sc_actions previous = info.sevActions[severity];
     info.sevActions[severity] = actions;
     return previous;
@@ -168,7 +168,7 @@ sc_report_handler::stop_after(const char *msg_type, int limit)
     if (!msg_type)
         msg_type = SC_ID_UNKNOWN_ERROR_;
 
-    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap[msg_type];
+    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap()[msg_type];
     int previous = info.limit;
     info.limit = limit;
     return previous;
@@ -181,7 +181,7 @@ sc_report_handler::stop_after(
     if (!msg_type)
         msg_type = SC_ID_UNKNOWN_ERROR_;
 
-    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap[msg_type];
+    sc_gem5::ReportMsgInfo &info = sc_gem5::reportMsgInfoMap()[msg_type];
     int previous = info.sevLimits[severity];
     info.sevLimits[severity] = limit;
     return previous;
@@ -199,7 +199,7 @@ sc_report_handler::get_count(const char *msg_type)
     if (!msg_type)
         msg_type = SC_ID_UNKNOWN_ERROR_;
 
-    return sc_gem5::reportMsgInfoMap[msg_type].count;
+    return sc_gem5::reportMsgInfoMap()[msg_type].count;
 }
 
 int
@@ -208,7 +208,7 @@ sc_report_handler::get_count(const char *msg_type, sc_severity severity)
     if (!msg_type)
         msg_type = SC_ID_UNKNOWN_ERROR_;
 
-    return sc_gem5::reportMsgInfoMap[msg_type].sevCounts[severity];
+    return sc_gem5::reportMsgInfoMap()[msg_type].sevCounts[severity];
 }
 
 int