Various cleanup. Mostly forcing initialization of some fields...

I noticed that checkpoints dumped from m5.debug and m5.opt had
differences, which is no longer the case (other than addresses
encoded in names).

cpu/full_cpu/smt.hh:
    Get rid of unused SMT_MAX_CPUS.
sim/debug.cc:
    Use AutoDelete for DebugBreakEvent.
sim/serialize.cc:
sim/serialize.hh:
    Move constructor & destructor to header so they can be inlined.

--HG--
extra : convert_revision : bc68c2c9b053b1de0d655ed555734419fafd0b83

3 files changed, 3 insertions, 10 deletions

diff --git a/sim/debug.cc b/sim/debug.cc
index 95187baff..6f3789c96 100644
--- a/sim/debug.cc
+++ b/sim/debug.cc
@@ -66,6 +66,7 @@ class DebugBreakEvent : public Event
 DebugBreakEvent::DebugBreakEvent(EventQueue *q, Tick _when)
     : Event(q)
 {
+    setFlags(AutoDelete);
     schedule(_when, -20000);
 }
 
@@ -76,7 +77,6 @@ void
 DebugBreakEvent::process()
 {
     debug_break();
-    delete this;
 }
 
 
diff --git a/sim/serialize.cc b/sim/serialize.cc
index 1fce6e7b1..9738029c2 100644
--- a/sim/serialize.cc
+++ b/sim/serialize.cc
@@ -50,13 +50,6 @@ using namespace std;
 
 Serializer *Serializeable::serializer = NULL;
 
-Serializeable::Serializeable()
-    : serialized(false)
-{ }
-
-Serializeable::~Serializeable()
-{ }
-
 void
 Serializeable::mark()
 {
diff --git a/sim/serialize.hh b/sim/serialize.hh
index bc40d0ad2..077931d8c 100644
--- a/sim/serialize.hh
+++ b/sim/serialize.hh
@@ -115,8 +115,8 @@ class Serializeable
     void nameOut(std::ostream& os, const std::string &_name);
 
   public:
-    Serializeable();
-    virtual ~Serializeable();
+    Serializeable() : serialized(false) {}
+    virtual ~Serializeable() {}
 
     // manditory virtual function, so objects must provide names
     virtual std::string name() const = 0;