summaryrefslogtreecommitdiff
path: root/src/cpu
diff options
context:
space:
mode:
authorGabe Black <gabeblack@google.com>2018-01-05 15:33:08 -0800
committerGabe Black <gabeblack@google.com>2018-01-09 03:02:26 +0000
commitb52ea6e98cb27c97aeea1085c4f5fdcbbdf66f0c (patch)
tree5410e78345e98dea06eaef3b250e82ebcb5e7be7 /src/cpu
parent7221a00fdfe6a3f380f573d8bee1063d374b6bdf (diff)
downloadgem5-b52ea6e98cb27c97aeea1085c4f5fdcbbdf66f0c.tar.xz
cpu, power: Get rid of the remnants of the EA computation insts.
Get rid of some remnants of a system which was intended to separate address computation into its own instruction object. Change-Id: I23f9ffd70fcb89a8ea5bbb934507fb00da9a0b7f Reviewed-on: https://gem5-review.googlesource.com/7122 Reviewed-by: Jason Lowe-Power <jason@lowepower.com> Maintainer: Gabe Black <gabeblack@google.com>
Diffstat (limited to 'src/cpu')
-rw-r--r--src/cpu/base_dyn_inst.hh19
-rw-r--r--src/cpu/checker/cpu.hh7
-rw-r--r--src/cpu/exec_context.hh13
-rw-r--r--src/cpu/minor/dyn_inst.hh6
-rw-r--r--src/cpu/minor/exec_context.hh12
-rw-r--r--src/cpu/simple/exec_context.hh16
6 files changed, 1 insertions, 72 deletions
diff --git a/src/cpu/base_dyn_inst.hh b/src/cpu/base_dyn_inst.hh
index d7d32e629..127b0629c 100644
--- a/src/cpu/base_dyn_inst.hh
+++ b/src/cpu/base_dyn_inst.hh
@@ -132,10 +132,6 @@ class BaseDynInst : public ExecContext, public RefCounted
RecordResult,
Predicate,
PredTaken,
- /** Whether or not the effective address calculation is completed.
- * @todo: Consider if this is necessary or not.
- */
- EACalcDone,
IsStrictlyOrdered,
ReqMade,
MemOpDone,
@@ -245,12 +241,6 @@ class BaseDynInst : public ExecContext, public RefCounted
// Need a copy of main request pointer to verify on writes.
RequestPtr reqToVerify;
- private:
- /** Instruction effective address.
- * @todo: Consider if this is necessary or not.
- */
- Addr instEffAddr;
-
protected:
/** Flattened register index of the destination registers of this
* instruction.
@@ -859,15 +849,6 @@ class BaseDynInst : public ExecContext, public RefCounted
ThreadContext *tcBase() { return thread->getTC(); }
public:
- /** Sets the effective address. */
- void setEA(Addr ea) { instEffAddr = ea; instFlags[EACalcDone] = true; }
-
- /** Returns the effective address. */
- Addr getEA() const { return instEffAddr; }
-
- /** Returns whether or not the eff. addr. calculation has been completed. */
- bool doneEACalc() { return instFlags[EACalcDone]; }
-
/** Returns whether or not the eff. addr. source registers are ready. */
bool eaSrcsReady();
diff --git a/src/cpu/checker/cpu.hh b/src/cpu/checker/cpu.hh
index b1a457491..f79aa0864 100644
--- a/src/cpu/checker/cpu.hh
+++ b/src/cpu/checker/cpu.hh
@@ -181,13 +181,6 @@ class CheckerCPU : public BaseCPU, public ExecContext
void serialize(CheckpointOut &cp) const override;
void unserialize(CheckpointIn &cp) override;
- // These functions are only used in CPU models that split
- // effective address computation from the actual memory access.
- void setEA(Addr EA) override
- { panic("CheckerCPU::setEA() not implemented\n"); }
- Addr getEA() const override
- { panic("CheckerCPU::getEA() not implemented\n"); }
-
// The register accessor methods provide the index of the
// instruction's operand (e.g., 0 or 1), not the architectural
// register index, to simplify the implementation of register
diff --git a/src/cpu/exec_context.hh b/src/cpu/exec_context.hh
index e7f5d37ac..59d74146b 100644
--- a/src/cpu/exec_context.hh
+++ b/src/cpu/exec_context.hh
@@ -223,19 +223,6 @@ class ExecContext {
* @name Memory Interface
*/
/**
- * Record the effective address of the instruction.
- *
- * @note Only valid for memory ops.
- */
- virtual void setEA(Addr EA) = 0;
- /**
- * Get the effective address of the instruction.
- *
- * @note Only valid for memory ops.
- */
- virtual Addr getEA() const = 0;
-
- /**
* Perform an atomic memory read operation. Must be overridden
* for exec contexts that support atomic memory mode. Not pure
* virtual since exec contexts that only support timing memory
diff --git a/src/cpu/minor/dyn_inst.hh b/src/cpu/minor/dyn_inst.hh
index 79c9ca4a4..b2decb39b 100644
--- a/src/cpu/minor/dyn_inst.hh
+++ b/src/cpu/minor/dyn_inst.hh
@@ -221,9 +221,6 @@ class MinorDynInst : public RefCounted
* up */
RegId flatDestRegIdx[TheISA::MaxInstDestRegs];
- /** Effective address as set by ExecContext::setEA */
- Addr ea;
-
public:
MinorDynInst(InstId id_ = InstId(), Fault fault_ = NoFault) :
staticInst(NULL), id(id_), traceData(NULL),
@@ -232,8 +229,7 @@ class MinorDynInst : public RefCounted
fuIndex(0), inLSQ(false), inStoreBuffer(false),
canEarlyIssue(false),
instToWaitFor(0), extraCommitDelay(Cycles(0)),
- extraCommitDelayExpr(NULL), minimumCommitCycle(Cycles(0)),
- ea(0)
+ extraCommitDelayExpr(NULL), minimumCommitCycle(Cycles(0))
{ }
public:
diff --git a/src/cpu/minor/exec_context.hh b/src/cpu/minor/exec_context.hh
index 4b3a02fca..6ac0df5f8 100644
--- a/src/cpu/minor/exec_context.hh
+++ b/src/cpu/minor/exec_context.hh
@@ -424,20 +424,8 @@ class ExecContext : public ::ExecContext
thread.getDTBPtr()->demapPage(vaddr, asn);
}
- /* ALPHA/POWER: Effective address storage */
- void setEA(Addr ea) override
- {
- inst->ea = ea;
- }
-
BaseCPU *getCpuPtr() { return &cpu; }
- /* POWER: Effective address storage */
- Addr getEA() const override
- {
- return inst->ea;
- }
-
/* MIPS: other thread register reading/writing */
uint64_t
readRegOtherThread(const RegId& reg, ThreadID tid = InvalidThreadID)
diff --git a/src/cpu/simple/exec_context.hh b/src/cpu/simple/exec_context.hh
index 6d51e5ed9..13c44ac22 100644
--- a/src/cpu/simple/exec_context.hh
+++ b/src/cpu/simple/exec_context.hh
@@ -413,22 +413,6 @@ class SimpleExecContext : public ExecContext {
}
- /**
- * Record the effective address of the instruction.
- *
- * @note Only valid for memory ops.
- */
- void setEA(Addr EA) override
- { panic("BaseSimpleCPU::setEA() not implemented\n"); }
-
- /**
- * Get the effective address of the instruction.
- *
- * @note Only valid for memory ops.
- */
- Addr getEA() const override
- { panic("BaseSimpleCPU::getEA() not implemented\n"); }
-
Fault readMem(Addr addr, uint8_t *data, unsigned int size,
Request::Flags flags) override
{