systemc: "Fix" some error message formatting.

These changes make the output match what Accellera outputs so that the
tests will pass.

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

diff --git a/src/systemc/dt/bit/sc_bit.cc b/src/systemc/dt/bit/sc_bit.cc
index 7572a0e564a33aced78af04ac561d72a826e9cda..8382c880cb44cafe038e9a1df0105884ad8b1c7f 100644 (file)
--- a/src/systemc/dt/bit/sc_bit.cc
+++ b/src/systemc/dt/bit/sc_bit.cc
@@ -77,8 +77,8 @@ void
 sc_bit::invalid_value(char c)
 {
     std::stringstream msg;
-    msg << "sc_bit('" << c << "')";
-    SC_REPORT_ERROR("value is not valid", msg.str().c_str());
+    msg << "sc_bit( '" << c << "' )";
+    SC_REPORT_ERROR("(E204) value is not valid", msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 
@@ -86,8 +86,8 @@ void
 sc_bit::invalid_value(int i)
 {
     std::stringstream msg;
-    msg << "sc_bit(" << i << ")";
-    SC_REPORT_ERROR("value is not valid", msg.str().c_str());
+    msg << "sc_bit( " << i << " )";
+    SC_REPORT_ERROR("(E204) value is not valid", msg.str().c_str());
     sc_core::sc_abort(); // can't recover from here
 }
 

diff --git a/src/systemc/dt/bit/sc_logic.cc b/src/systemc/dt/bit/sc_logic.cc
index 81663fcee66547b95232257b5d8ea14741123b7a..4943d3df19fbe2612ee6a13a8796d17ba2fdb643 100644 (file)
--- a/src/systemc/dt/bit/sc_logic.cc
+++ b/src/systemc/dt/bit/sc_logic.cc
@@ -72,7 +72,7 @@ sc_logic::invalid_value(char c)
 {
     std::stringstream msg;
     msg << "sc_logic('" << c << "')";
-    SC_REPORT_ERROR("value is not valid", msg.str().c_str());
+    SC_REPORT_ERROR("(E204) value is not valid", msg.str().c_str());
 }
 
 void
@@ -80,7 +80,7 @@ sc_logic::invalid_value(int i)
 {
     std::stringstream msg;
     msg << "sc_logic(" << i << ")";
-    SC_REPORT_ERROR("value is not valid", msg.str().c_str());
+    SC_REPORT_ERROR("(E204) value is not valid", msg.str().c_str());
 }
 
 
@@ -88,9 +88,11 @@ void
 sc_logic::invalid_01() const
 {
     if ((int)m_val == Log_Z) {
-        SC_REPORT_WARNING("sc_logic value 'Z' cannot be converted to bool", 0);
+        SC_REPORT_WARNING(
+                "(W211) sc_logic value 'Z' cannot be converted to bool", 0);
     } else {
-        SC_REPORT_WARNING("sc_logic value 'X' cannot be converted to bool", 0);
+        SC_REPORT_WARNING(
+                "(W212) sc_logic value 'X' cannot be converted to bool", 0);
     }
 }
 

diff --git a/src/systemc/dt/int/sc_nbutils.cc b/src/systemc/dt/int/sc_nbutils.cc
index df7cc554dda5b753076aa806d4e7780957f45dfd..5d2c50fc9e18435433a8da5be5c5e1692a0f04ef 100644 (file)
--- a/src/systemc/dt/int/sc_nbutils.cc
+++ b/src/systemc/dt/int/sc_nbutils.cc
@@ -89,7 +89,7 @@ is_valid_base(sc_numrep base)
         std::stringstream msg;
         msg << "is_valid_base( sc_numrep base ) : base = " << base <<
                " is not valid";
-        SC_REPORT_ERROR("value is not valid", msg.str().c_str() );
+        SC_REPORT_ERROR("(E204) value is not valid", msg.str().c_str() );
     }
 }
 

diff --git a/src/systemc/dt/int/sc_signed.cc b/src/systemc/dt/int/sc_signed.cc
index 1cb71123b98fb9fb0c0c05d2655b0240e1eb79f0..74660b371a085ef210bd4f87ce9a89250c6ee8ae 100644 (file)
--- a/src/systemc/dt/int/sc_signed.cc
+++ b/src/systemc/dt/int/sc_signed.cc
@@ -132,7 +132,7 @@ sc_signed::invalid_range(int l, int r) const
 {
     std::stringstream msg;
     msg << "sc_bigint part selection: left = " <<
-           l << ", right = " << r << "\n"
+           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());

diff --git a/src/systemc/dt/int/sc_unsigned.cc b/src/systemc/dt/int/sc_unsigned.cc
index aca49bac61664d17171955eb6d418d86fe772798..ffea95a68684a2ca0df4d34f52770d099db2403d 100644 (file)
--- a/src/systemc/dt/int/sc_unsigned.cc
+++ b/src/systemc/dt/int/sc_unsigned.cc
@@ -140,7 +140,7 @@ sc_unsigned::invalid_range(int l, int r) const
 {
     std::stringstream msg;
     msg << "sc_biguint part selection: left = " <<
-           l << ", right = " << r << "\n"
+           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());

diff --git a/src/systemc/ext/dt/bit/sc_bv_base.hh b/src/systemc/ext/dt/bit/sc_bv_base.hh
index 8aac78d37feb1cdfc934483b14174c88df37ea71..ffb3c5a03c776359e77ca035918c2fbe7ac2c5f7 100644 (file)
--- a/src/systemc/ext/dt/bit/sc_bv_base.hh
+++ b/src/systemc/ext/dt/bit/sc_bv_base.hh
@@ -264,7 +264,7 @@ inline void
 sc_bv_base::set_cword(int /*i*/, sc_digit w)
 {
     if (w) {
-        SC_REPORT_WARNING("sc_bv cannot contain values X and Z", 0);
+        SC_REPORT_WARNING("(W207) sc_bv cannot contain values X and Z", 0);
     }
 }
 

diff --git a/src/systemc/ext/dt/fx/sc_fxdefs.hh b/src/systemc/ext/dt/fx/sc_fxdefs.hh
index 0d76df6e2e65ff0604935f71cf0d7baf6624fce1..8acfabfa58ecb67c57320e22794beed468c96d18 100644 (file)
--- a/src/systemc/ext/dt/fx/sc_fxdefs.hh
+++ b/src/systemc/ext/dt/fx/sc_fxdefs.hh
@@ -243,7 +243,7 @@ const int SC_DEFAULT_MAX_WL_ = SC_BUILTIN_MAX_WL_;
 #define SC_ERROR_IF_(cnd,id) SC_ERROR_IF_IMPL_(cnd, id, 0)
 
 #define SC_CHECK_WL_(wl) SC_ERROR_IF_((wl) <= 0, \
-        "total wordlength <= 0 is not valid")
+        "(E300) total wordlength <= 0 is not valid")
 
 #define SC_CHECK_N_BITS_(n_bits) \
     SC_ERROR_IF_((n_bits) < 0, "number of bits < 0 is not valid")

diff --git a/src/systemc/ext/dt/int/sc_nbutils.hh b/src/systemc/ext/dt/int/sc_nbutils.hh
index 9f7269e8c44b2d999ec28615e82b4e7b53f949ae..0d3a15cc638c28e9d63af987eafc1acb957d120a 100644 (file)
--- a/src/systemc/ext/dt/int/sc_nbutils.hh
+++ b/src/systemc/ext/dt/int/sc_nbutils.hh
@@ -822,7 +822,7 @@ is_bad_double(double v)
 {
     // Windows throws exception.
     if (is_nan(v) || is_inf(v))
-        SC_REPORT_ERROR("value is not valid",
+        SC_REPORT_ERROR("(E204) value is not valid",
                         "is_bad_double(double v) : "
                         "v is not finite - NaN or Inf");
 }