diff options
| author | Daniel R. Carvalho <odanrc@yahoo.com.br> | 2019-08-03 12:04:14 +0200 |
|---|---|---|
| committer | Daniel Carvalho <odanrc@yahoo.com.br> | 2019-08-07 06:49:57 +0000 |
| commit | e33b3aa6692b172f6db5957774a9e0289e81fa5b (patch) | |
| tree | 59a64933b78689ecd8e27552ab8852f3bbebc050 /src/mem/cache/replacement_policies | |
| parent | 598f704d51ddfc7c1abd0afdbb99e4cbafbe8813 (diff) | |
| download | gem5-e33b3aa6692b172f6db5957774a9e0289e81fa5b.tar.xz | |
mem-cache: Fix non-virtual base destructor of Repl Entry
ReplaceableEntry contains a virtual method, yet its destructor
was not virtual, causing errors in some compilers.
Change-Id: I13deec843f4007d9deb924882a8d98ff6a89c84f
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/19808
Reviewed-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Maintainer: Nikos Nikoleris <nikos.nikoleris@arm.com>
Tested-by: kokoro <noreply+kokoro@google.com>
Diffstat (limited to 'src/mem/cache/replacement_policies')
| -rw-r--r-- | src/mem/cache/replacement_policies/replaceable_entry.hh | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/src/mem/cache/replacement_policies/replaceable_entry.hh b/src/mem/cache/replacement_policies/replaceable_entry.hh index dffa4cf10..c558beea1 100644 --- a/src/mem/cache/replacement_policies/replaceable_entry.hh +++ b/src/mem/cache/replacement_policies/replaceable_entry.hh @@ -63,12 +63,15 @@ class ReplaceableEntry */ uint32_t _way; - public: - /** - * Replacement data associated to this entry. - * It must be instantiated by the replacement policy before being used. - */ - std::shared_ptr<ReplacementData> replacementData; + public: + ReplaceableEntry() = default; + virtual ~ReplaceableEntry() = default; + + /** + * Replacement data associated to this entry. + * It must be instantiated by the replacement policy before being used. + */ + std::shared_ptr<ReplacementData> replacementData; /** * Set both the set and way. Should be called only once. |