From: Tony Gutierrez <anthony.gutierrez@amd.com>
Date: Tue, 20 Dec 2016 16:47:30 +0000 (-0500)
Subject: ruby: fix issue with unused var in DMASequencer
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=02cb6b19a782a9e3d87fd5a2e446c77d5521b41d;p=gem5.git

ruby: fix issue with unused var in DMASequencer

the iterator declared in DMASequencer::ackCallback() is only used in an
assert, this causes clang to fail when building fast. here we move
the find call on the request table directly into the assert.
---

diff --git a/src/mem/ruby/system/DMASequencer.cc b/src/mem/ruby/system/DMASequencer.cc
index 4bee19b52..9ffd28cec 100644
--- a/src/mem/ruby/system/DMASequencer.cc
+++ b/src/mem/ruby/system/DMASequencer.cc
@@ -191,9 +191,7 @@ DMASequencer::dataCallback(const DataBlock & dblk, const Addr& address)
 void
 DMASequencer::ackCallback(const Addr& address)
 {
-    RequestTable::iterator i = m_RequestTable.find(address);
-    assert(i != m_RequestTable.end());
-
+    assert(m_RequestTable.find(addres) != m_RequestTable.end());
     issueNext(address);
 }