diff options
author | Gabe Black <gabeblack@google.com> | 2018-10-07 03:02:56 -0700 |
---|---|---|
committer | Gabe Black <gabeblack@google.com> | 2018-10-16 01:11:44 +0000 |
commit | b366cbcde953e2adddc10a2825e2803b5f8a9bdd (patch) | |
tree | e532fa49c762eb6b293f88f5a0583946d3a88227 | |
parent | 98f40c54a31457078de587b8a29fdd055b9db28b (diff) | |
download | gem5-b366cbcde953e2adddc10a2825e2803b5f8a9bdd.tar.xz |
systemc: Switch to using predefined messages for utils.
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>
-rw-r--r-- | src/systemc/dt/bit/sc_bit.cc | 3 | ||||
-rw-r--r-- | src/systemc/dt/bit/sc_lv_base.cc | 3 | ||||
-rw-r--r-- | src/systemc/dt/fx/sc_fxval.cc | 4 | ||||
-rw-r--r-- | src/systemc/dt/fx/scfx_rep.cc | 4 | ||||
-rw-r--r-- | src/systemc/dt/int/sc_int_base.cc | 11 | ||||
-rw-r--r-- | src/systemc/dt/int/sc_signed.cc | 4 | ||||
-rw-r--r-- | src/systemc/dt/int/sc_uint_base.cc | 10 | ||||
-rw-r--r-- | src/systemc/dt/int/sc_unsigned.cc | 5 | ||||
-rw-r--r-- | src/systemc/ext/dt/bit/sc_bit_proxies.hh | 27 | ||||
-rw-r--r-- | src/systemc/ext/dt/fx/sc_fxdefs.hh | 3 | ||||
-rw-r--r-- | src/systemc/ext/utils/_using.hh | 17 | ||||
-rw-r--r-- | src/systemc/ext/utils/_utils.hh | 1 | ||||
-rw-r--r-- | src/systemc/ext/utils/messages.hh | 55 | ||||
-rw-r--r-- | src/systemc/ext/utils/sc_report_handler.hh | 4 | ||||
-rw-r--r-- | src/systemc/ext/utils/sc_vector.hh | 5 | ||||
-rw-r--r-- | src/systemc/utils/SConscript | 1 | ||||
-rw-r--r-- | src/systemc/utils/messages.cc | 85 | ||||
-rw-r--r-- | src/systemc/utils/sc_report.cc | 8 | ||||
-rw-r--r-- | src/systemc/utils/sc_report_handler.cc | 2 | ||||
-rw-r--r-- | src/systemc/utils/sc_vector.cc | 6 |
20 files changed, 213 insertions, 45 deletions
diff --git a/src/systemc/dt/bit/sc_bit.cc b/src/systemc/dt/bit/sc_bit.cc index 8382c880c..e2e157905 100644 --- a/src/systemc/dt/bit/sc_bit.cc +++ b/src/systemc/dt/bit/sc_bit.cc @@ -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"); } } diff --git a/src/systemc/dt/bit/sc_lv_base.cc b/src/systemc/dt/bit/sc_lv_base.cc index 8ffdbe23d..de2d655db 100644 --- a/src/systemc/dt/bit/sc_lv_base.cc +++ b/src/systemc/dt/bit/sc_lv_base.cc @@ -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()); } // ---------------------------------------------------------------------------- diff --git a/src/systemc/dt/fx/sc_fxval.cc b/src/systemc/dt/fx/sc_fxval.cc index 45d7d5735..3b3b1e031 100644 --- a/src/systemc/dt/fx/sc_fxval.cc +++ b/src/systemc/dt/fx/sc_fxval.cc @@ -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(); } diff --git a/src/systemc/dt/fx/scfx_rep.cc b/src/systemc/dt/fx/scfx_rep.cc index ae19fabe0..f94c5eae0 100644 --- a/src/systemc/dt/fx/scfx_rep.cc +++ b/src/systemc/dt/fx/scfx_rep.cc @@ -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(); } diff --git a/src/systemc/dt/int/sc_int_base.cc b/src/systemc/dt/int/sc_int_base.cc index 6094086d8..69b62d128 100644 --- a/src/systemc/dt/int/sc_int_base.cc +++ b/src/systemc/dt/int/sc_int_base.cc @@ -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()); } } diff --git a/src/systemc/dt/int/sc_signed.cc b/src/systemc/dt/int/sc_signed.cc index 74660b371..cbb688523 100644 --- a/src/systemc/dt/int/sc_signed.cc +++ b/src/systemc/dt/int/sc_signed.cc @@ -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 } diff --git a/src/systemc/dt/int/sc_uint_base.cc b/src/systemc/dt/int/sc_uint_base.cc index 9d28f778c..a6053813f 100644 --- a/src/systemc/dt/int/sc_uint_base.cc +++ b/src/systemc/dt/int/sc_uint_base.cc @@ -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()); } } diff --git a/src/systemc/dt/int/sc_unsigned.cc b/src/systemc/dt/int/sc_unsigned.cc index ffea95a68..b358c5cd3 100644 --- a/src/systemc/dt/int/sc_unsigned.cc +++ b/src/systemc/dt/int/sc_unsigned.cc @@ -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 } diff --git a/src/systemc/ext/dt/bit/sc_bit_proxies.hh b/src/systemc/ext/dt/bit/sc_bit_proxies.hh index da57f04ac..206bb0b06 100644 --- a/src/systemc/ext/dt/bit/sc_bit_proxies.hh +++ b/src/systemc/ext/dt/bit/sc_bit_proxies.hh @@ -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; diff --git a/src/systemc/ext/dt/fx/sc_fxdefs.hh b/src/systemc/ext/dt/fx/sc_fxdefs.hh index c72a45afc..f31867ebc 100644 --- a/src/systemc/ext/dt/fx/sc_fxdefs.hh +++ b/src/systemc/ext/dt/fx/sc_fxdefs.hh @@ -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 diff --git a/src/systemc/ext/utils/_using.hh b/src/systemc/ext/utils/_using.hh index 721f40383..2cbb7254f 100644 --- a/src/systemc/ext/utils/_using.hh +++ b/src/systemc/ext/utils/_using.hh @@ -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__ diff --git a/src/systemc/ext/utils/_utils.hh b/src/systemc/ext/utils/_utils.hh index d76caf9ec..6c19ac230 100644 --- a/src/systemc/ext/utils/_utils.hh +++ b/src/systemc/ext/utils/_utils.hh @@ -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 index 000000000..09c8c06d2 --- /dev/null +++ b/src/systemc/ext/utils/messages.hh @@ -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__ diff --git a/src/systemc/ext/utils/sc_report_handler.hh b/src/systemc/ext/utils/sc_report_handler.hh index f65b32cca..a3bf314bb 100644 --- a/src/systemc/ext/utils/sc_report_handler.hh +++ b/src/systemc/ext/utils/sc_report_handler.hh @@ -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); diff --git a/src/systemc/ext/utils/sc_vector.hh b/src/systemc/ext/utils/sc_vector.hh index f005751d5..f5a8f9ccb 100644 --- a/src/systemc/ext/utils/sc_vector.hh +++ b/src/systemc/ext/utils/sc_vector.hh @@ -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; diff --git a/src/systemc/utils/SConscript b/src/systemc/utils/SConscript index 6e3c6fe18..f8de5b0c8 100644 --- a/src/systemc/utils/SConscript +++ b/src/systemc/utils/SConscript @@ -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 index 000000000..a3f23734d --- /dev/null +++ b/src/systemc/utils/messages.cc @@ -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 diff --git a/src/systemc/utils/sc_report.cc b/src/systemc/utils/sc_report.cc index e76dfcb6d..122facc24 100644 --- a/src/systemc/utils/sc_report.cc +++ b/src/systemc/utils/sc_report.cc @@ -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; } diff --git a/src/systemc/utils/sc_report_handler.cc b/src/systemc/utils/sc_report_handler.cc index 46ee84704..1b725cc0f 100644 --- a/src/systemc/utils/sc_report_handler.cc +++ b/src/systemc/utils/sc_report_handler.cc @@ -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; diff --git a/src/systemc/utils/sc_vector.cc b/src/systemc/utils/sc_vector.cc index 985253995..9d086105f 100644 --- a/src/systemc/utils/sc_vector.cc +++ b/src/systemc/utils/sc_vector.cc @@ -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 |