Assert that an mshr has a target in getTarget().

--HG--
extra : convert_revision : 08091670fc319876012ed139fcd2584c364a980c

1 files changed, 7 insertions, 7 deletions

diff --git a/src/mem/cache/miss/mshr.hh b/src/mem/cache/miss/mshr.hh
index 293f290b8..a27f465aa 100644
--- a/src/mem/cache/miss/mshr.hh
+++ b/src/mem/cache/miss/mshr.hh
@@ -182,10 +182,16 @@ 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)