Assert that an mshr has a target in getTarget().
authorSteve Reinhardt <stever@eecs.umich.edu>
Tue, 17 Jul 2007 13:23:11 +0000 (06:23 -0700)
committerSteve Reinhardt <stever@eecs.umich.edu>
Tue, 17 Jul 2007 13:23:11 +0000 (06:23 -0700)
--HG--
extra : convert_revision : 08091670fc319876012ed139fcd2584c364a980c

src/mem/cache/miss/mshr.hh

index 293f290b83a56a199aa4b355191dabf5df36a4aa..a27f465aad0412d994f78dc8556a56539a68a45f 100644 (file)
@@ -181,11 +181,17 @@ public:
      */
     TargetList* getTargetList() { return &targets; }
 
+    /**
+     * Returns true if there are targets left.
+     * @return true if there are targets
+     */
+    bool hasTargets() { return !targets.empty(); }
+
     /**
      * Returns a reference to the first target.
      * @return A pointer to the first target.
      */
-    Target *getTarget() { return &targets.front(); }
+    Target *getTarget() { assert(hasTargets());  return &targets.front(); }
 
     /**
      * Pop first target.
@@ -196,12 +202,6 @@ public:
         targets.pop_front();
     }
 
-    /**
-     * Returns true if there are targets left.
-     * @return true if there are targets
-     */
-    bool hasTargets() { return !targets.empty(); }
-
     bool isSimpleForward()
     {
         if (getNumTargets() != 1)