From 02cb6b19a782a9e3d87fd5a2e446c77d5521b41d Mon Sep 17 00:00:00 2001 From: Tony Gutierrez Date: Tue, 20 Dec 2016 11:47:30 -0500 Subject: [PATCH] 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. --- src/mem/ruby/system/DMASequencer.cc | 4 +--- 1 file changed, 1 insertion(+), 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); }