sim: Fix the Ticked class constructor's event wrapper.
authorGabe Black <gabeblack@google.com>
Mon, 25 May 2020 11:27:53 +0000 (04:27 -0700)
committerGabe Black <gabeblack@google.com>
Mon, 25 May 2020 21:21:33 +0000 (21:21 +0000)
This uses a "name()" method which is not defined by the Ticked class,
and isn't a global method. This was probably originally supposed to be
the name() method of the Serializable class that Ticked inherits from,
but a while ago that was removed. It's not clear how this has been
compiling.

Instead, use the name() method of the ClockedObject which is the first
constructor argument.

Change-Id: Icfb71732c58ea9984ef7343bbaa46097a25abf28
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/29406
Reviewed-by: Jason Lowe-Power <power.jg@gmail.com>
Maintainer: Jason Lowe-Power <power.jg@gmail.com>
Tested-by: kokoro <noreply+kokoro@google.com>
src/sim/ticked_object.cc

index c6d1f9817c60115702c42d9cc6d635895f1088ae..7af439cd56010e66e3bb5db8dba2ffa8db44ac54 100644 (file)
@@ -44,7 +44,7 @@ Ticked::Ticked(ClockedObject &object_,
     Stats::Scalar *imported_num_cycles,
     Event::Priority priority) :
     object(object_),
-    event([this]{ processClockEvent(); }, name(), false, priority),
+    event([this]{ processClockEvent(); }, object_.name(), false, priority),
     running(false),
     lastStopped(0),
     /* Allocate numCycles if an external stat wasn't passed in */