Nikos Nikoleris has submitted this change and it was merged. ( https://gem5-review.googlesource.com/7741 )

Change subject: base: Fix unused function warning
......................................................................

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.sandb...@arm.com>
Reviewed-on: https://gem5-review.googlesource.com/7741
Reviewed-by: Gabe Black <gabebl...@google.com>
Maintainer: Nikos Nikoleris <nikos.nikole...@arm.com>
---
M src/base/remote_gdb.cc
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Gabe Black: Looks good to me, approved
  Nikos Nikoleris: Looks good to me, approved



diff --git a/src/base/remote_gdb.cc b/src/base/remote_gdb.cc
index 80dcd13..37ecef9 100644
--- a/src/base/remote_gdb.cc
+++ b/src/base/remote_gdb.cc
@@ -290,6 +290,7 @@
     GdbAccWp = '4',
 };

+#ifndef NDEBUG
 const char *
 break_type(char c)
 {
@@ -302,6 +303,7 @@
       default: return "unknown breakpoint/watchpoint";
     }
 }
+#endif

 std::map<Addr, HardBreakpoint *> hardBreakMap;


--
To view, visit https://gem5-review.googlesource.com/7741
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Iea89b66c53c62341c043d8bd3838ebc27ee333bc
Gerrit-Change-Number: 7741
Gerrit-PatchSet: 2
Gerrit-Owner: Nikos Nikoleris <nikos.nikole...@arm.com>
Gerrit-Reviewer: Andreas Sandberg <andreas.sandb...@arm.com>
Gerrit-Reviewer: Gabe Black <gabebl...@google.com>
Gerrit-Reviewer: Nikos Nikoleris <nikos.nikole...@arm.com>
_______________________________________________
gem5-dev mailing list
gem5-dev@gem5.org
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to