changeset 22d08b519cb0 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=22d08b519cb0
description:
        mem, misc: fix building issue with CommMonitor (unused variables)

        Signed-off-by: Pierre-Yves Péneau <[email protected]>
        Reviewed-by: Andreas Hansson <[email protected]>
        Reviewed-by: Jason Lowe-Power <[email protected]>
        Signed-off-by: Jason Lowe-Power <[email protected]>

        Reviewed at http://reviews.gem5.org/r/3801/

diffstat:

 src/mem/comm_monitor.cc |  12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)

diffs (43 lines):

diff -r b9436a4bbbb9 -r 22d08b519cb0 src/mem/comm_monitor.cc
--- a/src/mem/comm_monitor.cc   Wed Feb 15 09:28:44 2017 -0600
+++ b/src/mem/comm_monitor.cc   Wed Feb 15 14:56:54 2017 -0600
@@ -248,8 +248,6 @@
     // or even deleted when sendTiming() is called.
     const ProbePoints::PacketInfo pkt_info(pkt);
 
-    const bool is_read = pkt->isRead();
-    const bool is_write = pkt->isWrite();
     const bool expects_response(pkt->needsResponse() &&
                                 !pkt->cacheResponding());
 
@@ -274,8 +272,8 @@
     }
 
     if (successful) {
-        DPRINTF(CommMonitor, "Forwarded %s request\n",
-                (is_read ? "read" : (is_write ? "write" : "non read/write")));
+        DPRINTF(CommMonitor, "Forwarded %s request\n", pkt->isRead() ? "read" :
+                pkt->isWrite() ? "write" : "non read/write");
         stats.updateReqStats(pkt_info, false, expects_response);
     }
     return successful;
@@ -291,8 +289,6 @@
     // or even deleted when sendTiming() is called.
     const ProbePoints::PacketInfo pkt_info(pkt);
 
-    bool is_read = pkt->isRead();
-    bool is_write = pkt->isWrite();
     Tick latency = 0;
     CommMonitorSenderState* received_state =
         dynamic_cast<CommMonitorSenderState*>(pkt->senderState);
@@ -325,8 +321,8 @@
 
     if (successful) {
         ppPktResp->notify(pkt_info);
-        DPRINTF(CommMonitor, "Received %s response\n",
-                (is_read ? "Read" : (is_write ? "Write" : "non read/write")));
+        DPRINTF(CommMonitor, "Received %s response\n", pkt->isRead() ? "read" :
+                pkt->isWrite() ?  "write" : "non read/write");
         stats.updateRespStats(pkt_info, latency, false);
     }
     return successful;
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to