From 5bdeb1a432430dcb265cef77fa9365eceddce7d7 Mon Sep 17 00:00:00 2001 From: Gabe Black Date: Thu, 27 Sep 2018 01:16:42 -0700 Subject: systemc: Modify reportifyException to keep sc_reports in scope. reportifyException was sometimes indirectly creating temporary sc_report objects which would go out of scope when they were returned. The later code which tried to print them would then read garbage. Change-Id: I0a744eb029c62bf2ffee83db0a0e6dcbe3e60f7d Reviewed-on: https://gem5-review.googlesource.com/c/13190 Reviewed-by: Gabe Black Maintainer: Gabe Black --- src/systemc/core/scheduler.cc | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) (limited to 'src/systemc/core/scheduler.cc') diff --git a/src/systemc/core/scheduler.cc b/src/systemc/core/scheduler.cc index fe1d7c2e4..d57505517 100644 --- a/src/systemc/core/scheduler.cc +++ b/src/systemc/core/scheduler.cc @@ -418,11 +418,10 @@ Scheduler::schedulePause() } void -Scheduler::throwToScMain(const ::sc_core::sc_report *r) +Scheduler::throwToScMain() { - if (!r) - r = reportifyException(); - _throwToScMain = r; + ::sc_core::sc_report report = reportifyException(); + _throwToScMain = &report; status(StatusOther); scMain->run(); } @@ -462,7 +461,7 @@ throwingReportHandler(const ::sc_core::sc_report &r, } // anonymous namespace -const ::sc_core::sc_report * +const ::sc_core::sc_report reportifyException() { ::sc_core::sc_report_handler_proc old_handler = @@ -488,7 +487,7 @@ reportifyException() } } catch (const ::sc_core::sc_report &r) { ::sc_core::sc_report_handler::set_handler(old_handler); - return &r; + return r; } panic("No exception thrown in reportifyException."); } -- cgit v1.2.3