Daniel Carvalho has submitted this change and it was merged. (
https://gem5-review.googlesource.com/9301 )
Change subject: mem-cache: Fix MRU rebase
......................................................................
mem-cache: Fix MRU rebase
Rebase of MRU missed a const qualifier, introducing a compilation
error.
Change-Id: Ia25aa30523613a1a87593a353abe439946656f63
Reviewed-on: https://gem5-review.googlesource.com/9301
Reviewed-by: Nikos Nikoleris <[email protected]>
Maintainer: Nikos Nikoleris <[email protected]>
---
M src/mem/cache/replacement_policies/mru_rp.cc
M src/mem/cache/replacement_policies/mru_rp.hh
2 files changed, 2 insertions(+), 2 deletions(-)
Approvals:
Nikos Nikoleris: Looks good to me, approved; Looks good to me, approved
diff --git a/src/mem/cache/replacement_policies/mru_rp.cc
b/src/mem/cache/replacement_policies/mru_rp.cc
index 0a89d43..f4cf014 100644
--- a/src/mem/cache/replacement_policies/mru_rp.cc
+++ b/src/mem/cache/replacement_policies/mru_rp.cc
@@ -56,7 +56,7 @@
}
CacheBlk*
-MRURP::getVictim(ReplacementCandidates& candidates)
+MRURP::getVictim(const ReplacementCandidates& candidates)
{
// There must be at least one replacement candidate
assert(candidates.size() > 0);
diff --git a/src/mem/cache/replacement_policies/mru_rp.hh
b/src/mem/cache/replacement_policies/mru_rp.hh
index d9276e6..d947d7c 100644
--- a/src/mem/cache/replacement_policies/mru_rp.hh
+++ b/src/mem/cache/replacement_policies/mru_rp.hh
@@ -77,7 +77,7 @@
* @param cands Replacement candidates, selected by indexing policy.
* @return Cache block to be replaced.
*/
- CacheBlk* getVictim(ReplacementCandidates& candidates) override;
+ CacheBlk* getVictim(const ReplacementCandidates& candidates) override;
};
#endif // __MEM_CACHE_REPLACEMENT_POLICIES_MRU_RP_HH__
--
To view, visit https://gem5-review.googlesource.com/9301
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: master
Gerrit-Change-Id: Ia25aa30523613a1a87593a353abe439946656f63
Gerrit-Change-Number: 9301
Gerrit-PatchSet: 2
Gerrit-Owner: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Daniel Carvalho <[email protected]>
Gerrit-Reviewer: Nikos Nikoleris <[email protected]>
Gerrit-MessageType: merged
_______________________________________________
gem5-dev mailing list
[email protected]
http://m5sim.org/mailman/listinfo/gem5-dev
