diff options
| author | Andreas Hansson <andreas.hansson@arm.com> | 2014-12-02 06:07:36 -0500 |
|---|---|---|
| committer | Andreas Hansson <andreas.hansson@arm.com> | 2014-12-02 06:07:36 -0500 |
| commit | 9779ba2e37a753df407b976fc4b299d936ea62b8 (patch) | |
| tree | e25e0cf44834427975767e88bf3dcfc6359ed4a8 /src/cpu/minor | |
| parent | 25bfc249998b26403d50587eb66e6ee5e6de5b58 (diff) | |
| download | gem5-9779ba2e37a753df407b976fc4b299d936ea62b8.tar.xz | |
mem: Add const getters for write packet data
This patch takes a first step in tightening up how we use the data
pointer in write packets. A const getter is added for the pointer
itself (getConstPtr), and a number of member functions are also made
const accordingly. In a range of places throughout the memory system
the new member is used.
The patch also removes the unused isReadWrite function.
Diffstat (limited to 'src/cpu/minor')
| -rw-r--r-- | src/cpu/minor/execute.cc | 2 | ||||
| -rw-r--r-- | src/cpu/minor/lsq.cc | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/cpu/minor/execute.cc b/src/cpu/minor/execute.cc index 123128358..69cb9a239 100644 --- a/src/cpu/minor/execute.cc +++ b/src/cpu/minor/execute.cc @@ -355,7 +355,7 @@ Execute::handleMemResponse(MinorDynInstPtr inst, if (is_load && packet->getSize() > 0) { DPRINTF(MinorMem, "Memory data[0]: 0x%x\n", - static_cast<unsigned int>(packet->getPtr<uint8_t>()[0])); + static_cast<unsigned int>(packet->getConstPtr<uint8_t>()[0])); } /* Complete the memory access instruction */ diff --git a/src/cpu/minor/lsq.cc b/src/cpu/minor/lsq.cc index cae0d3666..fca580085 100644 --- a/src/cpu/minor/lsq.cc +++ b/src/cpu/minor/lsq.cc @@ -560,7 +560,7 @@ LSQ::SplitDataRequest::retireResponse(PacketPtr response) * by the response fragment */ std::memcpy( data + (response->req->getVaddr() - request.getVaddr()), - response->getPtr<uint8_t>(), + response->getConstPtr<uint8_t>(), response->req->getSize()); } } |