Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/52483 )
Change subject: base: Set enableUnits in ScalarPrint declared in
DistPrint::operator().
......................................................................
base: Set enableUnits in ScalarPrint declared in DistPrint::operator().
The setup() method of ScalarPrint, inherited from BasePrint, is not
called by its constructor, and is not called on the ScalarPrint "print"
in DistPrint::operator(). This sets most of the values in BasePrint.
Instead, this method sets these values itself based on the values of the
DistPrint itself.
Unfortunately it looks like this method forgot to set enableUnits, which
ends up otherwise uninitialized when the printUnits method is called.
This change fixes that.
Change-Id: Ib1ea78796539f6d9222d19dad597a4e64aa69808
---
M src/base/stats/text.cc
1 file changed, 20 insertions(+), 0 deletions(-)
diff --git a/src/base/stats/text.cc b/src/base/stats/text.cc
index 828f551..db5743a 100644
--- a/src/base/stats/text.cc
+++ b/src/base/stats/text.cc
@@ -465,6 +465,7 @@
print.flags = flags;
print.descriptions = descriptions;
print.desc = desc;
+ print.enableUnits = enableUnits;
print.unitStr = unitStr;
print.pdf = Nan;
print.cdf = Nan;
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/52483
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Ib1ea78796539f6d9222d19dad597a4e64aa69808
Gerrit-Change-Number: 52483
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
