diff options
author | Steve Reinhardt <steve.reinhardt@amd.com> | 2016-01-17 18:27:46 -0800 |
---|---|---|
committer | Steve Reinhardt <steve.reinhardt@amd.com> | 2016-01-17 18:27:46 -0800 |
commit | 707275265f188a514d1d5673ed4c8d6495304962 (patch) | |
tree | c245b55e0efae47759c11e96e703d4c8794c8573 /src/cpu/o3/cpu.hh | |
parent | e595d9cccb3c3a06f915547ecc9b224d65e7d5cc (diff) | |
download | gem5-707275265f188a514d1d5673ed4c8d6495304962.tar.xz |
cpu: remove unnecessary data ptr from O3 internal read() funcs
The read() function merely initiates a memory read operation; the
data doesn't arrive until the access completes and a response packet
is received from the memory system. Thus there's no need to provide
a data pointer; its existence is historical.
Getting this pointer out of this internal o3 interface sets the
stage for similar cleanup in the ExecContext interface. Also
found that we were pointlessly setting the contents at this pointer
on a store forward (the useful memcpy happens just a few lines
below the deleted one).
Diffstat (limited to 'src/cpu/o3/cpu.hh')
-rw-r--r-- | src/cpu/o3/cpu.hh | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/cpu/o3/cpu.hh b/src/cpu/o3/cpu.hh index 09177d404..eed5811cb 100644 --- a/src/cpu/o3/cpu.hh +++ b/src/cpu/o3/cpu.hh @@ -678,10 +678,9 @@ class FullO3CPU : public BaseO3CPU /** CPU read function, forwards read to LSQ. */ Fault read(RequestPtr &req, RequestPtr &sreqLow, RequestPtr &sreqHigh, - uint8_t *data, int load_idx) + int load_idx) { - return this->iew.ldstQueue.read(req, sreqLow, sreqHigh, - data, load_idx); + return this->iew.ldstQueue.read(req, sreqLow, sreqHigh, load_idx); } /** CPU write function, forwards write to LSQ. */ |