Hello Andreas Sandberg,
I'd like you to do a code review. Please visit
https://gem5-review.googlesource.com/7741
to review the following change.
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 <[email protected]>
---
M src/base/remote_gdb.cc
1 file changed, 2 insertions(+), 0 deletions(-)
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: newchange
Gerrit-Change-Id: Iea89b66c53c62341c043d8bd3838ebc27ee333bc
Gerrit-Change-Number: 7741
Gerrit-PatchSet: 1
Gerrit-Owner: Nikos Nikoleris <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
