Gabe Black has submitted this change. ( https://gem5-review.googlesource.com/c/public/gem5/+/44030 )

Change subject: base: Get rid of the unused "debuggers" vector in the remote GDB stub.
......................................................................

base: Get rid of the unused "debuggers" vector in the remote GDB stub.

This was presumably there for debuggers attached to gem5 itself to
examine, but doesn't provide much value and adds a small amount of
complexity.

Change-Id: Iaed08b63beafe8bf05f1496f6341a7feadc350ce
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/44030
Reviewed-by: Boris Shingarov <[email protected]>
Reviewed-by: Daniel Carvalho <[email protected]>
Maintainer: Gabe Black <[email protected]>
Tested-by: kokoro <[email protected]>
---
M src/base/remote_gdb.cc
1 file changed, 0 insertions(+), 3 deletions(-)

Approvals:
  Boris Shingarov: Looks good to me, approved
  Daniel Carvalho: Looks good to me, but someone else must approve
  Gabe Black: Looks good to me, approved
  kokoro: Regressions pass



diff --git a/src/base/remote_gdb.cc b/src/base/remote_gdb.cc
index 38ddb75..5822309 100644
--- a/src/base/remote_gdb.cc
+++ b/src/base/remote_gdb.cc
@@ -155,8 +155,6 @@
 static const char GDBGoodP = '+';
 static const char GDBBadP = '-';

-std::vector<BaseRemoteGDB *> debuggers;
-
 class HardBreakpoint : public PCEvent
 {
   private:
@@ -316,7 +314,6 @@
         active(false), attached(false), sys(_system), tc(c),
         trapEvent(this), singleStepEvent(*this)
 {
-    debuggers.push_back(this);
 }

 BaseRemoteGDB::~BaseRemoteGDB()



1 is the latest approved patch-set.
No files were changed between the latest approved patch-set and the submitted one.
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/44030
To unsubscribe, or for help writing mail filters, visit https://gem5-review.googlesource.com/settings

Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Iaed08b63beafe8bf05f1496f6341a7feadc350ce
Gerrit-Change-Number: 44030
Gerrit-PatchSet: 3
Gerrit-Owner: Gabe Black <[email protected]>
Gerrit-Reviewer: Andreas Sandberg <[email protected]>
Gerrit-Reviewer: Bobby R. Bruce <[email protected]>
Gerrit-Reviewer: Boris Shingarov <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Gabe Black <[email protected]>
Gerrit-Reviewer: kokoro <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to