changeset 17b2781e1482 in /z/repo/gem5
details: http://repo.gem5.org/gem5?cmd=changeset;node=17b2781e1482
description:
        inorder: clear reg. dep entry after removing from list
        this will safeguard future code from trying to remove
        from the list twice. That code wouldnt break but would
        waste time.

diffstat:

 src/cpu/inorder/inorder_dyn_inst.hh |  3 +++
 src/cpu/inorder/reg_dep_map.cc      |  2 ++
 2 files changed, 5 insertions(+), 0 deletions(-)

diffs (25 lines):

diff -r a6aacf190f14 -r 17b2781e1482 src/cpu/inorder/inorder_dyn_inst.hh
--- a/src/cpu/inorder/inorder_dyn_inst.hh       Sun Jun 19 21:43:42 2011 -0400
+++ b/src/cpu/inorder/inorder_dyn_inst.hh       Sun Jun 19 21:43:42 2011 -0400
@@ -943,6 +943,9 @@
     /** Sets this instruction as entered on the CPU Reg Dep Map */
     void setRegDepEntry() { status.set(RegDepMapEntry); }
 
+    /** Unsets this instruction as entered on the CPU Reg Dep Map */
+    void clearRegDepEntry() { status.reset(RegDepMapEntry); }
+
     /** Returns whether or not the entry is on the CPU Reg Dep Map */
     bool isRegDepEntry() const { return status[RegDepMapEntry]; }
 
diff -r a6aacf190f14 -r 17b2781e1482 src/cpu/inorder/reg_dep_map.cc
--- a/src/cpu/inorder/reg_dep_map.cc    Sun Jun 19 21:43:42 2011 -0400
+++ b/src/cpu/inorder/reg_dep_map.cc    Sun Jun 19 21:43:42 2011 -0400
@@ -157,6 +157,8 @@
 
             remove(reg_type, flat_idx, inst);
         }
+
+        inst->clearRegDepEntry();
     }
 }
 
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev

Reply via email to