diff options
| author | Andreas Hansson <andreas.hansson@arm.com> | 2014-09-19 10:35:18 -0400 |
|---|---|---|
| committer | Andreas Hansson <andreas.hansson@arm.com> | 2014-09-19 10:35:18 -0400 |
| commit | 41fc8a573ea61b2463606a0714a9e563494da329 (patch) | |
| tree | c038491b91eb89fa487781bca6ba5b6b1ba65ec3 /src/cpu/translation.hh | |
| parent | 619c5519fe214250d537527ec95191a9b3d6fad2 (diff) | |
| download | gem5-41fc8a573ea61b2463606a0714a9e563494da329.tar.xz | |
arch: Pass faults by const reference where possible
This patch changes how faults are passed between methods in an attempt
to copy as few reference-counting pointer instances as possible. This
should avoid unecessary copies being created, contributing to the
increment/decrement of the reference counters.
Diffstat (limited to 'src/cpu/translation.hh')
| -rw-r--r-- | src/cpu/translation.hh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/cpu/translation.hh b/src/cpu/translation.hh index 46c96a0d4..f870a9c11 100644 --- a/src/cpu/translation.hh +++ b/src/cpu/translation.hh @@ -111,7 +111,7 @@ class WholeTranslationState * request to make it easier to access them later on. */ bool - finish(Fault fault, int index) + finish(const Fault &fault, int index) { assert(outstanding); faults[index] = fault; @@ -249,7 +249,7 @@ class DataTranslation : public BaseTLB::Translation * translation is complete if the state says so. */ void - finish(Fault fault, RequestPtr req, ThreadContext *tc, + finish(const Fault &fault, RequestPtr req, ThreadContext *tc, BaseTLB::Mode mode) { assert(state); |