derek/gem5
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

systemc: Add some error checks to sc_set_default_time_unit.

Browse Source 
Change-Id: I1d21c56d3b39044d91c96c98d242a571c099707c
Reviewed-on: https://gem5-review.googlesource.com/c/12463
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>
This commit is contained in:
Gabe Black
2018-09-01 17:49:20 -07:00
parent 902d2a598d
commit f8217428d5
1 changed files with 28 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

29
src/systemc/core/sc_time.cc
View File

@@ -371,7 +371,7 @@ const sc_time SC_ZERO_TIME;
void
sc_set_time_resolution(double d, sc_time_unit tu)
{
if (d < 0.0) {
if (d <= 0.0) {
SC_REPORT_ERROR("(E514) set time resolution failed",
"value not positive");
}
@@ -426,7 +426,34 @@ sc_max_time()
void
sc_set_default_time_unit(double d, sc_time_unit tu)
{
if (d < 0.0) {
SC_REPORT_ERROR("(E515) set default time unit failed",
"value not positive");
}
double dummy;
if (modf(log10(d), &dummy) != 0.0) {
SC_REPORT_ERROR("(E515) set default time unit failed",
"value not a power of ten");
}
if (sc_is_running()) {
SC_REPORT_ERROR("(E515) set default time unit failed",
"simulation running");
}
static bool specified = false;
if (specified) {
SC_REPORT_ERROR("(E515) set default time unit failed",
"already specified");
}
// This won't detect the timescale being fixed outside of systemc, but
// it's at least some protection.
if (timeFixed) {
SC_REPORT_ERROR("(E515) set default time unit failed",
"sc_time object(s) constructed");
}
// Normalize d to seconds.
defaultUnit = d * TimeUnitScale[tu];
specified = true;
}
sc_time