From 4910d36e765875b6531b13c2a98f958082cc3de0 Mon Sep 17 00:00:00 2001
From: Nikos Nikoleris <nikos.nikoleris@arm.com>
Date: Thu, 1 Feb 2018 17:57:48 +0000
Subject: [PATCH] base: Fix unused function warning

After refactoring the remote gdb interface, break_type is declared as
const function and is only used as a parameter to DPRINTF function
calls. This means that it is seen as unused when compiling
gem5.fast. This changeset fixes the warning.

Change-Id: Iea89b66c53c62341c043d8bd3838ebc27ee333bc
Reviewed-by: Andreas Sandberg <andreas.sandberg@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/7741
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
---
 src/base/remote_gdb.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/base/remote_gdb.cc b/src/base/remote_gdb.cc
index 80dcd137b..37ecef97c 100644
--- a/src/base/remote_gdb.cc
+++ b/src/base/remote_gdb.cc
@@ -290,6 +290,7 @@ enum GdbBreakpointType {
     GdbAccWp = '4',
 };
 
+#ifndef NDEBUG
 const char *
 break_type(char c)
 {
@@ -302,6 +303,7 @@ break_type(char c)
       default: return "unknown breakpoint/watchpoint";
     }
 }
+#endif
 
 std::map<Addr, HardBreakpoint *> hardBreakMap;
 
-- 
2.30.2