summaryrefslogtreecommitdiff
path: root/src/mem/ruby/system
diff options
context:
space:
mode:
authorTony Gutierrez <anthony.gutierrez@amd.com>2016-12-20 11:47:30 -0500
committerTony Gutierrez <anthony.gutierrez@amd.com>2016-12-20 11:47:30 -0500
commit02cb6b19a782a9e3d87fd5a2e446c77d5521b41d (patch)
tree00eb0f030253ae5f448976102d500ddcb36d8261 /src/mem/ruby/system
parentb7d072b2353038ea9d00d4ea536449295ab10839 (diff)
downloadgem5-02cb6b19a782a9e3d87fd5a2e446c77d5521b41d.tar.xz
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.
Diffstat (limited to 'src/mem/ruby/system')
-rw-r--r--src/mem/ruby/system/DMASequencer.cc4
1 files changed, 1 insertions, 3 deletions
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);
}