summaryrefslogtreecommitdiff
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
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.
-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);
}