diff options
| author | Daniel R. Carvalho <odanrc@yahoo.com.br> | 2018-04-12 13:03:55 +0200 |
|---|---|---|
| committer | Daniel Carvalho <odanrc@yahoo.com.br> | 2018-10-10 18:17:42 +0000 |
| commit | 79eccfbe674d3cc8ebd2d9ed6787893f981de43a (patch) | |
| tree | 0655b0d6317778fc7eb8876f042070c022357f18 /src/mem/cache/tags/base_set_assoc.hh | |
| parent | f32882d4fc78af3747f81375dfd1ec3e37596c2b (diff) | |
| download | gem5-79eccfbe674d3cc8ebd2d9ed6787893f981de43a.tar.xz | |
mem-cache: Remove CacheSet.hh
Replacement policies aren't aware of cache sets and do not
organize blocks based on replacement data. Block search is
independent of block placement.
Besides, indexing policies have their own way of addressing
the sets, therefore there is no need to use this class anymore.
BlkType has been removed, as it wasn't being used.
Change-Id: Ia79c2a491e59f295c8d60a0466c317eb0e2bdab9
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/9782
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Reviewed-by: Jason Lowe-Power <jason@lowepower.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Diffstat (limited to 'src/mem/cache/tags/base_set_assoc.hh')
| -rw-r--r-- | src/mem/cache/tags/base_set_assoc.hh | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/mem/cache/tags/base_set_assoc.hh b/src/mem/cache/tags/base_set_assoc.hh index e06417224..01bfa05b2 100644 --- a/src/mem/cache/tags/base_set_assoc.hh +++ b/src/mem/cache/tags/base_set_assoc.hh @@ -71,16 +71,12 @@ */ class BaseSetAssoc : public BaseTags { - public: - /** Typedef the block type used in this tag store. */ - typedef CacheBlk BlkType; - protected: /** The allocatable associativity of the cache (alloc mask). */ unsigned allocAssoc; /** The cache blocks. */ - std::vector<BlkType> blks; + std::vector<CacheBlk> blks; /** Whether tags and data are accessed sequentially. */ const bool sequentialAccess; @@ -129,7 +125,7 @@ class BaseSetAssoc : public BaseTags */ CacheBlk* accessBlock(Addr addr, bool is_secure, Cycles &lat) override { - BlkType *blk = findBlock(addr, is_secure); + CacheBlk *blk = findBlock(addr, is_secure); // Access all tags in parallel, hence one in each way. The data side // either accesses all blocks in parallel, or one block sequentially on |