From: Daniel R. Carvalho <odanrc@yahoo.com.br>
Date: Wed, 19 Feb 2020 08:27:17 +0000 (+0100)
Subject: base: Make read-only functions const in ScalarBase
X-Git-Tag: develop-gem5-snapshot~136
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=3c915a2e84ff90117cf45461c7c6befd8bebb7a5;p=gem5.git

base: Make read-only functions const in ScalarBase

These functions do not need to modify their storage's contents.

ScalarBase's non-const value() has been removed.

Change-Id: I4dd3899a29a741a7d8cd199ccd254b346d86ae07
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/27084
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Bobby R. Bruce <bbruce@ucdavis.edu>
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
---

diff --git a/src/base/statistics.hh b/src/base/statistics.hh
index 1c3d53df6..b1f27c726 100644
--- a/src/base/statistics.hh
+++ b/src/base/statistics.hh
@@ -522,13 +522,6 @@ class ScalarBase : public DataWrap<Derived, ScalarInfoProxy>
         this->setInit();
     }
 
-  public:
-    /**
-     * Return the current value of this stat as its base type.
-     * @return The current value.
-     */
-    Counter value() const { return data()->value(); }
-
   public:
     ScalarBase(Group *parent = nullptr, const char *name = nullptr,
                const char *desc = nullptr)
@@ -585,13 +578,17 @@ class ScalarBase : public DataWrap<Derived, ScalarInfoProxy>
      */
     size_type size() const { return 1; }
 
-    Counter value() { return data()->value(); }
+    /**
+     * Return the current value of this stat as its base type.
+     * @return The current value.
+     */
+    Counter value() const { return data()->value(); }
 
-    Result result() { return data()->result(); }
+    Result result() const { return data()->result(); }
 
-    Result total() { return result(); }
+    Result total() const { return result(); }
 
-    bool zero() { return result() == 0.0; }
+    bool zero() const { return result() == 0.0; }
 
     void reset() { data()->reset(this->info()); }
     void prepare() { data()->prepare(this->info()); }