I missed a couple of WithEffects, this should do it
authorAli Saidi <saidi@eecs.umich.edu>
Thu, 8 Mar 2007 02:51:44 +0000 (21:51 -0500)
committerAli Saidi <saidi@eecs.umich.edu>
Thu, 8 Mar 2007 02:51:44 +0000 (21:51 -0500)
--HG--
extra : convert_revision : 19fce78a19b27b7ccb5e3653a64b46e6d5292915

src/arch/isa_parser.py
src/arch/sparc/miscregfile.cc
src/arch/sparc/miscregfile.hh
src/arch/sparc/ua2005.cc
src/cpu/o3/alpha/dyn_inst.hh
src/cpu/o3/sparc/dyn_inst.hh
src/cpu/simple/base.hh

index 9b63c88428e76e65e9dcd309d3a5425f682a1e3a..21860a2e17a2c2622b374414b2b78b5400143ef6 100755 (executable)
@@ -1368,7 +1368,7 @@ class ControlRegOperand(Operand):
         bit_select = 0
         if (self.ctype == 'float' or self.ctype == 'double'):
             error(0, 'Attempt to read control register as FP')
-        base = 'xc->readMiscRegOperandWithEffect(this, %s)' % self.src_reg_idx
+        base = 'xc->readMiscRegOperand(this, %s)' % self.src_reg_idx
         if self.size == self.dflt_size:
             return '%s = %s;\n' % (self.base_name, base)
         else:
@@ -1378,7 +1378,7 @@ class ControlRegOperand(Operand):
     def makeWrite(self):
         if (self.ctype == 'float' or self.ctype == 'double'):
             error(0, 'Attempt to write control register as FP')
-        wb = 'xc->setMiscRegOperandWithEffect(this, %s, %s);\n' % \
+        wb = 'xc->setMiscRegOperand(this, %s, %s);\n' % \
              (self.dest_reg_idx, self.base_name)
         wb += 'if (traceData) { traceData->setData(%s); }' % \
               self.base_name
index e7b866e6f8eea201430aa49395d21d36c0d6e27e..20bc4ae5dddcb4d64e58c7bf747229e4f162c50e 100644 (file)
@@ -374,7 +374,7 @@ MiscReg MiscRegFile::readReg(int miscReg, ThreadContext * tc)
       case MISCREG_QUEUE_NRES_ERROR_TAIL:
 #if FULL_SYSTEM
       case MISCREG_HPSTATE:
-        return readFSRegWithEffect(miscReg, tc);
+        return readFSReg(miscReg, tc);
 #else
       case MISCREG_HPSTATE:
         //HPSTATE is special because because sometimes in privilege checks for instructions
@@ -682,7 +682,7 @@ void MiscRegFile::setReg(int miscReg,
       case MISCREG_QUEUE_NRES_ERROR_TAIL:
 #if FULL_SYSTEM
       case MISCREG_HPSTATE:
-        setFSRegWithEffect(miscReg, val, tc);
+        setFSReg(miscReg, val, tc);
         return;
 #else
       case MISCREG_HPSTATE:
index cb46eb2aab1fcaee7b0736d420175582cfbb55f9..6063c21c8ecfee54ee2cc02712f333fc0b157852 100644 (file)
@@ -257,9 +257,8 @@ namespace SparcISA
         // These need to check the int_dis field and if 0 then
         // set appropriate bit in softint and checkinterrutps on the cpu
 #if FULL_SYSTEM
-        void  setFSRegWithEffect(int miscReg, const MiscReg &val,
-                ThreadContext *tc);
-        MiscReg readFSRegWithEffect(int miscReg, ThreadContext * tc);
+        void  setFSReg(int miscReg, const MiscReg &val, ThreadContext *tc);
+        MiscReg readFSReg(int miscReg, ThreadContext * tc);
 
         // Update interrupt state on softint or pil change
         void checkSoftInt(ThreadContext *tc);
index 7a16dc3528ead9ce4dd03399204f74e86319e53b..6c8a987fea3ab3b6a1cf6fc0e770e848441654da 100644 (file)
@@ -59,8 +59,7 @@ MiscRegFile::checkSoftInt(ThreadContext *tc)
 
 
 void
-MiscRegFile::setFSRegWithEffect(int miscReg, const MiscReg &val,
-                                ThreadContext *tc)
+MiscRegFile::setFSReg(int miscReg, const MiscReg &val, ThreadContext *tc)
 {
     int64_t time;
     switch (miscReg) {
@@ -196,7 +195,7 @@ MiscRegFile::setFSRegWithEffect(int miscReg, const MiscReg &val,
 }
 
 MiscReg
-MiscRegFile::readFSRegWithEffect(int miscReg, ThreadContext * tc)
+MiscRegFile::readFSReg(int miscReg, ThreadContext * tc)
 {
     switch (miscReg) {
         /* Privileged registers. */
index 6c27e890a8c4598eea784ff159723195f1e85f40..20759d8498c8d5d543e78d75b3ec8bbe5d146ca2 100644 (file)
@@ -125,7 +125,7 @@ class AlphaDynInst : public BaseDynInst<Impl>
     }
 
     /** Reads a miscellaneous register. */
-    TheISA::MiscReg readMiscRegOperand(const StaticInst *si, int idx)
+    TheISA::MiscReg readMiscRegOperandNoEffect(const StaticInst *si, int idx)
     {
         return this->cpu->readMiscRegNoEffect(
                 si->srcRegIdx(idx) - TheISA::Ctrl_Base_DepTag,
@@ -135,7 +135,7 @@ class AlphaDynInst : public BaseDynInst<Impl>
     /** Reads a misc. register, including any side-effects the read
      * might have as defined by the architecture.
      */
-    TheISA::MiscReg readMiscRegOperandWithEffect(const StaticInst *si, int idx)
+    TheISA::MiscReg readMiscRegOperand(const StaticInst *si, int idx)
     {
         return this->cpu->readMiscReg(
                 si->srcRegIdx(idx) - TheISA::Ctrl_Base_DepTag,
@@ -143,7 +143,7 @@ class AlphaDynInst : public BaseDynInst<Impl>
     }
 
     /** Sets a misc. register. */
-    void setMiscRegOperand(const StaticInst * si, int idx, const MiscReg &val)
+    void setMiscRegOperandNoEffect(const StaticInst * si, int idx, const MiscReg &val)
     {
         this->instResult.integer = val;
         return this->cpu->setMiscRegNoEffect(
@@ -154,7 +154,7 @@ class AlphaDynInst : public BaseDynInst<Impl>
     /** Sets a misc. register, including any side-effects the write
      * might have as defined by the architecture.
      */
-    void setMiscRegOperandWithEffect(const StaticInst *si, int idx,
+    void setMiscRegOperand(const StaticInst *si, int idx,
                                      const MiscReg &val)
     {
         return this->cpu->setMiscReg(
index bd61b0384f5e7df47c8961e220020568123a9456..72242b161eb8d8d4404f7750b2ae531707258e7d 100644 (file)
@@ -107,7 +107,7 @@ class SparcDynInst : public BaseDynInst<Impl>
     }
 
     /** Reads a miscellaneous register. */
-    TheISA::MiscReg readMiscRegOperand(const StaticInst *si, int idx)
+    TheISA::MiscReg readMiscRegOperandNoEffect(const StaticInst *si, int idx)
     {
         return this->cpu->readMiscRegNoEffect(
                 si->srcRegIdx(idx) - TheISA::Ctrl_Base_DepTag,
@@ -117,7 +117,7 @@ class SparcDynInst : public BaseDynInst<Impl>
     /** Reads a misc. register, including any side-effects the read
      * might have as defined by the architecture.
      */
-    TheISA::MiscReg readMiscRegOperandWithEffect(const StaticInst *si, int idx)
+    TheISA::MiscReg readMiscRegOperand(const StaticInst *si, int idx)
     {
         return this->cpu->readMiscReg(
                 si->srcRegIdx(idx) - TheISA::Ctrl_Base_DepTag,
@@ -125,7 +125,7 @@ class SparcDynInst : public BaseDynInst<Impl>
     }
 
     /** Sets a misc. register. */
-    void setMiscRegOperand(const StaticInst * si,
+    void setMiscRegOperandNoEffect(const StaticInst * si,
             int idx, const TheISA::MiscReg &val)
     {
         this->instResult.integer = val;
@@ -137,7 +137,7 @@ class SparcDynInst : public BaseDynInst<Impl>
     /** Sets a misc. register, including any side-effects the write
      * might have as defined by the architecture.
      */
-    void setMiscRegOperandWithEffect(
+    void setMiscRegOperand(
             const StaticInst *si, int idx, const TheISA::MiscReg &val)
     {
         return this->cpu->setMiscReg(
index 980ea2f96251e25af828b7ad4163bc47c1fa37dc..eae24014bb2a5460971ff12c217fe8acbdd18d66 100644 (file)
@@ -304,25 +304,25 @@ class BaseSimpleCPU : public BaseCPU
         return thread->setMiscReg(misc_reg, val);
     }
 
-    MiscReg readMiscRegOperand(const StaticInst *si, int idx)
+    MiscReg readMiscRegOperandNoEffect(const StaticInst *si, int idx)
     {
         int reg_idx = si->srcRegIdx(idx) - TheISA::Ctrl_Base_DepTag;
         return thread->readMiscRegNoEffect(reg_idx);
     }
 
-    MiscReg readMiscRegOperandWithEffect(const StaticInst *si, int idx)
+    MiscReg readMiscRegOperand(const StaticInst *si, int idx)
     {
         int reg_idx = si->srcRegIdx(idx) - TheISA::Ctrl_Base_DepTag;
         return thread->readMiscReg(reg_idx);
     }
 
-    void setMiscRegOperand(const StaticInst *si, int idx, const MiscReg &val)
+    void setMiscRegOperandNoEffect(const StaticInst *si, int idx, const MiscReg &val)
     {
         int reg_idx = si->destRegIdx(idx) - TheISA::Ctrl_Base_DepTag;
         return thread->setMiscRegNoEffect(reg_idx, val);
     }
 
-    void setMiscRegOperandWithEffect(
+    void setMiscRegOperand(
             const StaticInst *si, int idx, const MiscReg &val)
     {
         int reg_idx = si->destRegIdx(idx) - TheISA::Ctrl_Base_DepTag;