mem: Compile tracePacket only when TRACING_ON is defined
authorGiacomo Travaglini <giacomo.travaglini@arm.com>
Fri, 7 Dec 2018 11:05:55 +0000 (11:05 +0000)
committerGiacomo Travaglini <giacomo.travaglini@arm.com>
Fri, 7 Dec 2018 13:28:19 +0000 (13:28 +0000)
If TRACING_ON is not defined (e.g. when building gem5.fast), clang
compilations will fail reporting an unused function.

Change-Id: I959dba6e9fcf74b951e16365077939ae4d4ef924
Signed-off-by: Giacomo Travaglini <giacomo.travaglini@arm.com>
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/c/14975
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>

src/mem/abstract_mem.cc

index b05a99db5dae76ba12d675fa388a0de8e8927e99..581fc14cc68805ea18a233686ce89c505cbc2b0c 100644 (file)
@@ -289,6 +289,7 @@ AbstractMemory::checkLockedAddrList(PacketPtr pkt)
     return allowStore;
 }
 
+#if TRACING_ON
 static inline void
 tracePacket(System *sys, const char *label, PacketPtr pkt)
 {
@@ -308,7 +309,6 @@ tracePacket(System *sys, const char *label, PacketPtr pkt)
     DDUMP(MemoryAccess, pkt->getConstPtr<uint8_t>(), pkt->getSize());
 }
 
-#if TRACING_ON
 #   define TRACE_PACKET(A) tracePacket(system(), A, pkt)
 #else
 #   define TRACE_PACKET(A)