systemc: Switch to using predefined messages for utils.
authorGabe Black <gabeblack@google.com>
Sun, 7 Oct 2018 10:02:56 +0000 (03:02 -0700)
committerGabe Black <gabeblack@google.com>
Tue, 16 Oct 2018 01:11:44 +0000 (01:11 +0000)
Create and use predefined messages for utils which match the ones
Accellera uses.

Change-Id: I932b7206fc16181d01a0d5b7441ce617b30e5365
Reviewed-on: https://gem5-review.googlesource.com/c/13328
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

20 files changed:
src/systemc/dt/bit/sc_bit.cc
src/systemc/dt/bit/sc_lv_base.cc
src/systemc/dt/fx/sc_fxval.cc
src/systemc/dt/fx/scfx_rep.cc
src/systemc/dt/int/sc_int_base.cc
src/systemc/dt/int/sc_signed.cc
src/systemc/dt/int/sc_uint_base.cc
src/systemc/dt/int/sc_unsigned.cc
src/systemc/ext/dt/bit/sc_bit_proxies.hh
src/systemc/ext/dt/fx/sc_fxdefs.hh
src/systemc/ext/utils/_using.hh
src/systemc/ext/utils/_utils.hh
src/systemc/ext/utils/messages.hh [new file with mode: 0644]
src/systemc/ext/utils/sc_report_handler.hh
src/systemc/ext/utils/sc_vector.hh
src/systemc/utils/SConscript
src/systemc/utils/messages.cc [new file with mode: 0644]
src/systemc/utils/sc_report.cc
src/systemc/utils/sc_report_handler.cc
src/systemc/utils/sc_vector.cc

index 8382c880cb44cafe038e9a1df0105884ad8b1c7f..e2e157905c03107f18c73dda9d62d81e090ca939 100644 (file)
@@ -60,6 +60,7 @@
 
 #include "systemc/ext/dt/bit/sc_bit.hh"
 #include "systemc/ext/dt/bit/sc_logic.hh"
+#include "systemc/ext/utils/messages.hh"
 #include "systemc/ext/utils/sc_report_handler.hh"
 
 namespace sc_dt
@@ -119,7 +120,7 @@ sc_deprecated_sc_bit()
     static bool warn_sc_bit_deprecated = true;
     if (warn_sc_bit_deprecated) {
         warn_sc_bit_deprecated = false;
-        SC_REPORT_INFO("(I804) /IEEE_Std_1666/deprecated",
+        SC_REPORT_INFO(sc_core::SC_ID_IEEE_1666_DEPRECATION_,
             "sc_bit is deprecated, use bool instead");
     }
 }
index 8ffdbe23d8cdd20563b0336f4cae4aa94de2eec6..de2d655db740a4673724a162b6fdc79ea214f4dd 100644 (file)
@@ -51,6 +51,7 @@
 #include <sstream>
 
 #include "systemc/ext/dt/bit/sc_lv_base.hh"
+#include "systemc/ext/utils/messages.hh"
 
 namespace sc_dt
 {
@@ -67,7 +68,7 @@ sc_proxy_out_of_bounds(const char *msg, int64 val)
         ss << msg;
     if (val != 0)
         ss << val;
-    SC_REPORT_ERROR("(E5) out of bounds", ss.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, ss.str().c_str());
 }
 
 // ----------------------------------------------------------------------------
index 45d7d573507f5e9e0ffb1cd48aca5ad8dcde8775..3b3b1e031100f7dc738c356ef85db15ce0cf522c 100644 (file)
@@ -51,6 +51,7 @@
 #include <cstdlib>
 
 #include "systemc/ext/dt/fx/sc_fxval.hh"
+#include "systemc/ext/utils/messages.hh"
 
 namespace sc_dt
 {
@@ -383,7 +384,8 @@ print_other(scfx_string &s, const scfx_ieee_double &id, sc_numrep numrep,
         step = 4;
         break;
       default:
-        SC_REPORT_FATAL("assertion failed", "unexpected sc_numrep");
+        SC_REPORT_FATAL(sc_core::SC_ID_ASSERTION_FAILED_,
+                "unexpected sc_numrep");
         sc_core::sc_abort();
     }
 
index ae19fabe0380a4c990dbf9669c447ca2045379fd..f94c5eae073fcfe1dc5773ba799794edf83dbc90 100644 (file)
@@ -71,6 +71,7 @@
 #include "systemc/ext/dt/fx/scfx_rep.hh"
 #include "systemc/ext/dt/fx/scfx_utils.hh"
 #include "systemc/ext/utils/endian.hh"
+#include "systemc/ext/utils/messages.hh"
 
 namespace sc_dt
 {
@@ -1078,7 +1079,8 @@ print_other(scfx_string &s, const scfx_rep &a, sc_numrep numrep, int w_prefix,
         step = 4;
         break;
       default:
-        SC_REPORT_FATAL("assertion failed", "unexpected sc_numrep");
+        SC_REPORT_FATAL(sc_core::SC_ID_ASSERTION_FAILED_,
+                "unexpected sc_numrep");
         sc_core::sc_abort();
     }
 
index 6094086d85e7b2c6921af72b2372bc5017b106a4..69b62d128f3f43bb6a420db57fd178cd9174f2a7 100644 (file)
@@ -71,6 +71,7 @@
 #include "systemc/ext/dt/int/sc_uint_base.hh"
 #include "systemc/ext/dt/int/sc_unsigned.hh"
 #include "systemc/ext/dt/misc/sc_concatref.hh"
+#include "systemc/ext/utils/messages.hh"
 
 // explicit template instantiations
 namespace sc_core
@@ -92,7 +93,7 @@ sc_int_concref_invalid_length(int length)
     std::stringstream msg;
     msg << "sc_int_concref<T1,T2> initialization: length = " << length <<
            "violates 1 <= length <= " << SC_INTWIDTH;
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -361,7 +362,7 @@ sc_int_base::invalid_length() const
     std::stringstream msg;
     msg << "sc_int[_base] initialization: length = " << m_len <<
            " violates 1 <= length <= " << SC_INTWIDTH;
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -371,7 +372,7 @@ sc_int_base::invalid_index(int i) const
     std::stringstream msg;
     msg << "sc_int[_base] bit selection: index = " << i <<
            " violates 0 <= index <= " << (m_len - 1);
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -382,7 +383,7 @@ sc_int_base::invalid_range(int l, int r) const
     msg << "sc_int[_base] part selection: " <<
            "left = " << l << ", right = " << r << " violates " <<
            (m_len-1) << " >= left >= right >= 0";
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -393,7 +394,7 @@ sc_int_base::check_value() const
     if (m_val < -limit || m_val >= limit) {
         std::stringstream msg;
         msg << "sc_int[_base]: value does not fit into a length of " << m_len;
-        SC_REPORT_WARNING("(E5) out of bounds", msg.str().c_str());
+        SC_REPORT_WARNING(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     }
 }
 
index 74660b371a085ef210bd4f87ce9a89250c6ee8ae..cbb688523980eb78f3dcce3cc8e7fc063cbefd38 100644 (file)
@@ -123,7 +123,7 @@ sc_signed::invalid_index(int i) const
     std::stringstream msg;
     msg << "sc_bigint bit selection: index = " << i << " violates "
            "0 <= index <= " << (nbits - 1);
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -135,7 +135,7 @@ sc_signed::invalid_range(int l, int r) const
            l << ", right = " << r << " \n"
            "  violates either (" << (nbits-1) << " >= left >= 0) or "
            "(" << (nbits-1) << " >= right >= 0)";
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
index 9d28f778c9e642e26e430e1f27c2ec44e651a6ad..a6053813f424f6603f2f3c97c242defbd650b5e9 100644 (file)
@@ -91,7 +91,7 @@ sc_uint_concref_invalid_length(int length)
     std::stringstream msg;
     msg << "sc_uint_concref<T1,T2> initialization: length = " << length <<
            "violates 1 <= length <= " << SC_INTWIDTH;
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -347,7 +347,7 @@ sc_uint_base::invalid_length() const
     std::stringstream msg;
     msg << "sc_uint[_base] initialization: length = " << m_len <<
            " violates 1 <= length <= " << SC_INTWIDTH;
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here}
 }
 
@@ -357,7 +357,7 @@ sc_uint_base::invalid_index(int i) const
     std::stringstream msg;
     msg << "sc_uint[_base] bit selection: index = " << i <<
            " violates 0 <= index <= " << (m_len - 1);
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -368,7 +368,7 @@ sc_uint_base::invalid_range(int l, int r) const
     msg << "sc_uint[_base] part selection: " <<
            "left = " << l << ", right = " << r << " violates " <<
            (m_len - 1) << " >= left >= right >= 0";
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -380,7 +380,7 @@ sc_uint_base::check_value() const
     if (m_val > limit) {
         std::stringstream msg;
         msg << "sc_uint[_base]: value does not fit into a length of " << m_len;
-        SC_REPORT_WARNING("(E5) out of bounds", msg.str().c_str());
+        SC_REPORT_WARNING(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     }
 }
 
index ffea95a68684a2ca0df4d34f52770d099db2403d..b358c5cd37d27de8deff362aef2dc7449afd7fac 100644 (file)
@@ -89,6 +89,7 @@
 #include "systemc/ext/dt/int/sc_uint_base.hh"
 #include "systemc/ext/dt/int/sc_unsigned.hh"
 #include "systemc/ext/dt/misc/sc_concatref.hh"
+#include "systemc/ext/utils/messages.hh"
 
 // explicit template instantiations
 namespace sc_core
@@ -131,7 +132,7 @@ sc_unsigned::invalid_index(int i) const
     std::stringstream msg;
     msg << "sc_biguint bit selection: index = " << i << " violates "
            "0 <= index <= " << (nbits-2);
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -143,7 +144,7 @@ sc_unsigned::invalid_range(int l, int r) const
            l << ", right = " << r << " \n"
            "  violates either (" << (nbits - 2) << " >= left >= 0) or "
            "(" << (nbits-2) << " >= right >= 0)";
-    SC_REPORT_ERROR("(E5) out of bounds", msg.str().c_str());
+    SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
index da57f04ac20041642ae56ab8f008ca53e03f1f36..206bb0b0658d6c4b0121cc86fdef96aff2962daf 100644 (file)
@@ -31,6 +31,7 @@
 
 #include <iostream>
 
+#include "../../utils/messages.hh"
 #include "sc_proxy.hh"
 
 namespace sc_dt
@@ -1735,7 +1736,7 @@ sc_bitref_r<T>::get_bit(int n) const
     if (n == 0) {
         return m_obj.get_bit(m_index);
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
         return Log_0;
     }
 }
@@ -1747,7 +1748,7 @@ sc_bitref_r<T>::get_word(int n) const
     if (n == 0) {
         return (get_bit(n) & SC_DIGIT_ONE);
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
         return 0;
     }
 }
@@ -1759,7 +1760,7 @@ sc_bitref_r<T>::get_cword(int n) const
     if (n == 0) {
         return ((get_bit(n) & SC_DIGIT_TWO) >> 1);
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
         return 0;
     }
 }
@@ -2068,7 +2069,7 @@ sc_bitref<X>::set_bit(int n, value_type value)
     if (n == 0) {
         this->m_obj.set_bit(this->m_index, value);
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
 }
 
@@ -2084,7 +2085,7 @@ sc_bitref<X>::set_word(int n, sc_digit w)
         temp = (temp & ~(1 << bi)) | ((w & 1) << bi);
         this->m_obj.set_word(wi, temp);
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
 }
 
@@ -2100,7 +2101,7 @@ sc_bitref<X>::set_cword(int n, sc_digit w)
         temp = (temp & ~(1 << bi)) | ((w & 1) << bi);
         this->m_obj.set_cword(wi, temp);
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
 }
 
@@ -2199,7 +2200,7 @@ sc_subref_r<X>::check_bounds()
 {
     int len = m_obj.length();
     if (m_hi < 0 || m_hi >= len || m_lo < 0 || m_lo >= len) {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
         sc_core::sc_abort(); // can't recover from here
     }
     if (reversed()) {
@@ -2668,7 +2669,7 @@ sc_concref_r<X, Y>::get_bit(int n) const
     } else if (n < r_len + m_left.length()) {
         return value_type(m_left.get_bit(n - r_len));
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
         return Log_0;
     }
 }
@@ -2683,7 +2684,7 @@ sc_concref_r<X, Y>::set_bit(int n, value_type v)
     } else if (n < r_len + m_left.length()) {
         m_left.set_bit(n - r_len, typename X::value_type(v));
     } else {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
 }
 
@@ -2692,7 +2693,7 @@ inline sc_digit
 sc_concref_r<X, Y>::get_word(int i) const
 {
     if (i < 0 || i >= size()) {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
     // 0 <= i < size()
     Y &r = m_right;
@@ -2726,7 +2727,7 @@ inline void
 sc_concref_r<X, Y>::set_word(int i, sc_digit w)
 {
     if (i < 0 || i >= size()) {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
     // 0 <= i < size()
     Y &r = m_right;
@@ -2765,7 +2766,7 @@ inline sc_digit
 sc_concref_r<X, Y>::get_cword(int i) const
 {
     if (i < 0 || i >= size()) {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
     // 0 <= i < size()
     Y &r = m_right;
@@ -2799,7 +2800,7 @@ inline void
 sc_concref_r<X, Y>::set_cword(int i, sc_digit w)
 {
     if (i < 0 || i >= size()) {
-        SC_REPORT_ERROR("(E5) out of bounds", 0);
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, 0);
     }
     // 0 <= i < size()
     Y &r = m_right;
index c72a45afc21d4bbfee9808d68edaa7f9d8d019c9..f31867ebcce8cf7fda1bb6a70ce0180b23d79e84 100644 (file)
@@ -47,6 +47,7 @@
 #ifndef __SYSTEMC_EXT_DT_FX_SC_FXDEFS_HH__
 #define __SYSTEMC_EXT_DT_FX_SC_FXDEFS_HH__
 
+#include "../../utils/messages.hh"
 #include "../../utils/sc_report_handler.hh"
 #include "../int/sc_nbutils.hh"
 
@@ -241,7 +242,7 @@ const int SC_DEFAULT_MAX_WL_ = SC_BUILTIN_MAX_WL_;
 
 #ifdef DEBUG_SYSTEMC
 #   define SC_ASSERT_(cnd, msg) \
-    SC_ERROR_IF_IMPL_(!(cnd), "internal error", msg)
+    SC_ERROR_IF_IMPL_(!(cnd), sc_core::SC_ID_INTERNAL_ERROR_, msg)
 #else
 #   define SC_ASSERT_(cnd, msg) (void(0))
 #endif
index 721f40383efea900595c2f458a1c01ade235a3b6..2cbb7254f4b6508dbef39bdcb342a0f3f9446a86 100644 (file)
@@ -98,4 +98,21 @@ using sc_core::sc_release;
 using sc_core::sc_copyright;
 using sc_core::sc_version;
 
+using sc_core::SC_ID_UNKNOWN_ERROR_;
+using sc_core::SC_ID_WITHOUT_MESSAGE_;
+using sc_core::SC_ID_NOT_IMPLEMENTED_;
+using sc_core::SC_ID_INTERNAL_ERROR_;
+using sc_core::SC_ID_ASSERTION_FAILED_;
+using sc_core::SC_ID_OUT_OF_BOUNDS_;
+using sc_core::SC_ID_ABORT_;
+
+using sc_core::SC_ID_REGISTER_ID_FAILED_;
+using sc_core::SC_ID_STRING_TOO_LONG_;
+using sc_core::SC_ID_FRONT_ON_EMPTY_LIST_;
+using sc_core::SC_ID_BACK_ON_EMPTY_LIST_;
+using sc_core::SC_ID_IEEE_1666_DEPRECATION_;
+using sc_core::SC_ID_VECTOR_INIT_CALLED_TWICE_;
+using sc_core::SC_ID_VECTOR_BIND_EMPTY_;
+using sc_core::SC_ID_VECTOR_NONOBJECT_ELEMENTS_;
+
 #endif  //__SYSTEMC_EXT_UTILS__USING_HH__
index d76caf9ecc2a9ef6e23043c8a71b956b6937756f..6c19ac230612d0f33d9e3b6583f8d6013f8f7635 100644 (file)
@@ -31,6 +31,7 @@
 #define __SYSTEMC_EXT_UTIL__UTIL_HH__
 
 #include "functions.hh"
+#include "messages.hh"
 #include "sc_exception.hh"
 #include "sc_report.hh"
 #include "sc_report_handler.hh"
diff --git a/src/systemc/ext/utils/messages.hh b/src/systemc/ext/utils/messages.hh
new file mode 100644 (file)
index 0000000..09c8c06
--- /dev/null
@@ -0,0 +1,55 @@
+/*
+ * Copyright 2018 Google, Inc.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: Gabe Black
+ */
+
+#ifndef __SYSTEMC_EXT_UTILS_MESSAGES_HH__
+#define __SYSTEMC_EXT_UTILS_MESSAGES_HH__
+
+namespace sc_core
+{
+
+extern const char SC_ID_UNKNOWN_ERROR_[];
+extern const char SC_ID_WITHOUT_MESSAGE_[];
+extern const char SC_ID_NOT_IMPLEMENTED_[];
+extern const char SC_ID_INTERNAL_ERROR_[];
+extern const char SC_ID_ASSERTION_FAILED_[];
+extern const char SC_ID_OUT_OF_BOUNDS_[];
+extern const char SC_ID_ABORT_[];
+
+extern const char SC_ID_REGISTER_ID_FAILED_[];
+extern const char SC_ID_STRING_TOO_LONG_[];
+extern const char SC_ID_FRONT_ON_EMPTY_LIST_[];
+extern const char SC_ID_BACK_ON_EMPTY_LIST_[];
+extern const char SC_ID_IEEE_1666_DEPRECATION_[];
+extern const char SC_ID_VECTOR_INIT_CALLED_TWICE_[];
+extern const char SC_ID_VECTOR_BIND_EMPTY_[];
+extern const char SC_ID_VECTOR_NONOBJECT_ELEMENTS_[];
+
+} // namespace sc_core
+
+#endif // __SYSTEMC_EXT_UTILS_MESSAGES_HH__
index f65b32ccaeca30668404a309727cb14471a9293f..a3bf314bbe940f60fdd5929d89051f8ee3c0e006 100644 (file)
@@ -32,6 +32,7 @@
 
 #include <string>
 
+#include "messages.hh"
 #include "sc_report.hh" // for sc_severity
 
 namespace sc_core
@@ -134,7 +135,8 @@ class sc_report_handler
             ::sc_core::SC_FATAL, msg_type, msg, __FILE__, __LINE__)
 
 #define sc_assert(expr) \
-        ((void)((expr) ? 0 : (SC_REPORT_FATAL("assertion failed", #expr), 0)))
+        ((void)((expr) ? 0 : (SC_REPORT_FATAL( \
+            ::sc_core::SC_ID_ASSERTION_FAILED_, #expr), 0)))
 
 void sc_interrupt_here(const char *msg_type, sc_severity);
 void sc_stop_here(const char *msg_type, sc_severity);
index f005751d5d822e340dac1f4135596680c2b3eb97..f5a8f9ccb6289c953cc12d74858816e25f77fd4b 100644 (file)
@@ -57,6 +57,7 @@
 
 #include "../core/sc_module.hh"
 #include "../core/sc_object.hh"
+#include "messages.hh"
 
 namespace sc_gem5
 {
@@ -182,9 +183,7 @@ class sc_vector_base : public sc_object
     sc_object *implicitCast(sc_object *p) const { return p; }
     sc_object *implicitCast(...) const
     {
-        SC_REPORT_ERROR(
-                "(E808) sc_vector::get_elements called for element type "
-                "not derived from sc_object", name());
+        SC_REPORT_ERROR(SC_ID_VECTOR_NONOBJECT_ELEMENTS_, name());
         return nullptr;
     }
     virtual sc_object *objectCast(void *) const = 0;
index 6e3c6fe18b72e0815b20ceaa136b0659e7626180..f8de5b0c8d636627fa2a1afe7bbf2fc6404608a6 100644 (file)
@@ -29,6 +29,7 @@ Import('*')
 
 if env['USE_SYSTEMC']:
     Source('functions.cc')
+    Source('messages.cc')
     Source('report.cc')
     Source('sc_report.cc')
     Source('sc_report_handler.cc')
diff --git a/src/systemc/utils/messages.cc b/src/systemc/utils/messages.cc
new file mode 100644 (file)
index 0000000..a3f2373
--- /dev/null
@@ -0,0 +1,85 @@
+/*
+ * Copyright 2018 Google, Inc.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met: redistributions of source code must retain the above copyright
+ * notice, this list of conditions and the following disclaimer;
+ * redistributions in binary form must reproduce the above copyright
+ * notice, this list of conditions and the following disclaimer in the
+ * documentation and/or other materials provided with the distribution;
+ * neither the name of the copyright holders nor the names of its
+ * contributors may be used to endorse or promote products derived from
+ * this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
+ * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
+ * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
+ * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
+ * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
+ * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ *
+ * Authors: Gabe Black
+ */
+
+#include "systemc/ext/utils/messages.hh"
+#include "systemc/utils/report.hh"
+
+namespace sc_core
+{
+
+const char SC_ID_REGISTER_ID_FAILED_[] = "register_id failed";
+const char SC_ID_UNKNOWN_ERROR_[] = "unknown error";
+const char SC_ID_WITHOUT_MESSAGE_[] = "";
+const char SC_ID_NOT_IMPLEMENTED_[] = "not implemented";
+const char SC_ID_INTERNAL_ERROR_[] = "internal error";
+const char SC_ID_ASSERTION_FAILED_[] = "assertion failed";
+const char SC_ID_OUT_OF_BOUNDS_[] = "out of bounds";
+const char SC_ID_ABORT_[] = "simulation aborted";
+
+const char SC_ID_STRING_TOO_LONG_[] = "string is too long";
+const char SC_ID_FRONT_ON_EMPTY_LIST_[] =
+    "attempt to take front() on an empty list";
+const char SC_ID_BACK_ON_EMPTY_LIST_[] =
+    "attempt to take back() on an empty list";
+const char SC_ID_IEEE_1666_DEPRECATION_[] =
+    "/IEEE_Std_1666/deprecated";
+const char SC_ID_VECTOR_INIT_CALLED_TWICE_[] =
+    "sc_vector::init has already been called";
+const char SC_ID_VECTOR_BIND_EMPTY_[] =
+    "sc_vector::bind called with empty range";
+const char SC_ID_VECTOR_NONOBJECT_ELEMENTS_[] =
+    "sc_vector::get_elements called for element type "
+    "not derived from sc_object";
+
+namespace
+{
+
+sc_gem5::DefaultReportMessages predefinedMessages{
+    {800, SC_ID_REGISTER_ID_FAILED_},
+    {0, SC_ID_UNKNOWN_ERROR_},
+    {1, SC_ID_WITHOUT_MESSAGE_},
+    {2, SC_ID_NOT_IMPLEMENTED_},
+    {3, SC_ID_INTERNAL_ERROR_},
+    {4, SC_ID_ASSERTION_FAILED_},
+    {5, SC_ID_OUT_OF_BOUNDS_},
+
+    {99, SC_ID_ABORT_},
+
+    {801, SC_ID_STRING_TOO_LONG_},
+    {802, SC_ID_FRONT_ON_EMPTY_LIST_},
+    {803, SC_ID_BACK_ON_EMPTY_LIST_},
+    {804, SC_ID_IEEE_1666_DEPRECATION_},
+    {805, SC_ID_VECTOR_INIT_CALLED_TWICE_},
+    {807, SC_ID_VECTOR_BIND_EMPTY_},
+    {808, SC_ID_VECTOR_NONOBJECT_ELEMENTS_}
+};
+
+} // anonymous namespace
+
+} // namespace sc_core
index e76dfcb6d8bdaceca4eec4e80a79e7713201f3f3..122facc2484659a29f7eca2ba79cbb9b333f7f74 100644 (file)
@@ -30,6 +30,7 @@
 #include <cstring>
 
 #include "base/logging.hh"
+#include "systemc/ext/utils/messages.hh"
 #include "systemc/ext/utils/sc_report.hh"
 #include "systemc/ext/utils/sc_report_handler.hh"
 #include "systemc/utils/report.hh"
@@ -119,18 +120,17 @@ void
 sc_report::register_id(int id, const char *msg)
 {
     if (id < 0) {
-        SC_REPORT_ERROR("(E800) register_id failed", "invalid report id");
+        SC_REPORT_ERROR(SC_ID_REGISTER_ID_FAILED_, "invalid report id");
         return;
     }
     if (!msg) {
-        SC_REPORT_ERROR("(E800) register_id failed", "invalid report message");
+        SC_REPORT_ERROR(SC_ID_REGISTER_ID_FAILED_, "invalid report message");
         return;
     }
     auto p = sc_gem5::reportIdToMsgMap.insert(
             std::pair<int, std::string>(id, msg));
     if (!p.second) {
-        SC_REPORT_ERROR("(E800) register_id failed",
-                "report id already exists");
+        SC_REPORT_ERROR(SC_ID_REGISTER_ID_FAILED_, "report id already exists");
     } else {
         sc_gem5::reportMsgInfoMap[msg].id = id;
     }
index 46ee847046b11a0321b83139dab8a364d3194cda..1b725cc0f91046218cacd1d22e0e032d2fe7347c 100644 (file)
@@ -108,8 +108,6 @@ 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];
-    if (msg_type == "")
-        msg_type = "unknown id";
 
     if (sc_gem5::reportWarningsAsErrors && severity == SC_WARNING)
         severity = SC_ERROR;
index 985253995f61a03dd0f02663013f7460af0c03d9..9d086105fec4f7b7dce44eb74dc1a1062cd59c3f 100644 (file)
@@ -50,6 +50,7 @@
 
 #include "base/cprintf.hh"
 #include "systemc/core/object.hh"
+#include "systemc/ext/utils/messages.hh"
 #include "systemc/ext/utils/sc_report_handler.hh"
 #include "systemc/ext/utils/sc_vector.hh"
 
@@ -76,7 +77,7 @@ sc_vector_base::checkIndex(size_type index) const
     if (index >= size()) {
         std::ostringstream ss;
         ccprintf(ss, "%s[%d] >= size() = %d", name(), index, size());
-        SC_REPORT_ERROR("(E5) out of bounds", ss.str().c_str());
+        SC_REPORT_ERROR(sc_core::SC_ID_OUT_OF_BOUNDS_, ss.str().c_str());
         sc_abort();
     }
 }
@@ -107,8 +108,7 @@ sc_vector_base::reportEmpty(const char *kind_, bool empty_dest) const
     else
         ss << "empty destination range given";
 
-    SC_REPORT_WARNING("(W807) sc_vector::bind called with empty range",
-            ss.str().c_str());
+    SC_REPORT_WARNING(SC_ID_VECTOR_BIND_EMPTY_, ss.str().c_str());
 }
 
 } // namespace sc_core