commit 14919c5e5d984690de2e65ac2aa82d7c129b573e
author Tommaso Marinelli <tommarin@ucm.es> Thu Oct 03 10:54:37 2019 +0200
committer Tommaso Marinelli <tommarin@ucm.es> Thu Oct 03 22:51:56 2019 +0000
tree e16a1a8a5cb11c4d6231d5cec7ebed4ed08cdd66
parent 5e1e76efa311f1888eada7df8cae538db51edff0

mem: Remove unused variable

The variable *sys in dram_ctrl.cc was only used in an assert() check,
therefore it has been removed to allow building gem5.fast without
errors. A typo in a comment in abstract_mem.hh has also been corrected.

Change-Id: I2663545449ecfdb5a27c3574b79dd42beb4a49c8
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/21380
Tested-by: kokoro <noreply+kokoro@google.com>
Reviewed-by: Anthony Gutierrez <anthony.gutierrez@amd.com>
Maintainer: Anthony Gutierrez <anthony.gutierrez@amd.com>

src/mem/abstract_mem.hh

diff --git a/src/mem/abstract_mem.hh b/src/mem/abstract_mem.hh
index 655fc01..d92e707 100644
--- a/src/mem/abstract_mem.hh
+++ b/src/mem/abstract_mem.hh
@@ -161,7 +161,7 @@
         }
     }
 
-    /** Pointor to the System object.
+    /** Pointer to the System object.
      * This is used for getting the number of masters in the system which is
      * needed when registering stats
      */

src/mem/dram_ctrl.cc

diff --git a/src/mem/dram_ctrl.cc b/src/mem/dram_ctrl.cc
index 1352dbf..37db27c 100644
--- a/src/mem/dram_ctrl.cc
+++ b/src/mem/dram_ctrl.cc
@@ -2535,8 +2535,7 @@
 {
     using namespace Stats;
 
-    System *sys = dram._system;
-    assert(sys);
+    assert(dram._system);
     const auto max_masters = dram._system->maxMasters();
 
     perBankRdBursts.init(dram.banksPerRank * dram.ranksPerChannel);